-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace netstat parsers with procfs netstat parsers #2336
Comments
@discordianfish Can you please assign this to me, i will start working on it asap. |
@discordianfish @SuperQ I think we should consider refactoring the types of the metrics on the procfs side here, here and here from What do you think about this? |
I think this is reasonable |
Any progress on getting this resolved? |
@jcpunk Yes, I will work on it during the next days. |
Hey @nikakis, any chance I can take this issue? I would like to give this a shot. |
Go for it. |
hi, i've been exploring and using prometheus exporters as of late and would love to contribute some fixes etc to this project (learning stuff along the way). I think no one's actively working on this issue right now and would like to take this up. I have a question about this issue tho: I've looked into the relevant parts of both the codebases and read up about the p.s. this is my first time learning about these concepts and contributing to Golang / cloud projects in general so there are chances I might've missed something |
I looked into the code and procfs more and figured out what I was missing but in doing so I realised, a lot of work done in #2360 holds up and can be proceeded with now that prometheus/procfs#464 is merged. I wonder if you'd be available for this, @nikakis? I'd be available to take over otherwise, let me know! |
Yes go ahead please. I don't have time to work on this right now. |
Replace this parsers in node exporter with this ones from procfs.
The text was updated successfully, but these errors were encountered: