Skip to content
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 float64 with *float64 for proc_netstat, proc_snmp and proc_snmp6 #464

Conversation

nikakis
Copy link
Contributor

@nikakis nikakis commented Sep 4, 2022

This PR is related to this issue #434 which is needed for prometheus/node_exporter#2336

Signed-off-by: Nikos Kakavas nikakis@protonmail.com

@SuperQ
Copy link
Member

SuperQ commented Sep 4, 2022

It seems like your procfs branch is out of date. Also, you need to setup a global gitignore.

Use git config --global core.excludesFile.

@SuperQ
Copy link
Member

SuperQ commented Sep 4, 2022

Please also either link to an issue, or include a description about why you are making changes.

@nikakis nikakis force-pushed the replace-float64-with-pointer-for-proc-netstat-snmp-and-snmp6 branch 4 times, most recently from 1109ce9 to b13676a Compare September 4, 2022 11:07
Signed-off-by: Nikos Kakavas <nikakis@protonmail.com>
@nikakis nikakis force-pushed the replace-float64-with-pointer-for-proc-netstat-snmp-and-snmp6 branch from b13676a to cf0ef0d Compare September 4, 2022 11:23
@nikakis
Copy link
Contributor Author

nikakis commented Sep 4, 2022

@SuperQ Any idea about how i should proceed with the codespell error?

@SuperQ
Copy link
Member

SuperQ commented Sep 4, 2022

Looks like the codespell issues are unrelated. They can be ignored.

@nikakis
Copy link
Contributor Author

nikakis commented Sep 4, 2022

Looks like the codespell issues are unrelated. They can be ignored.

Ok, so I am waiting for an approval from your side, right?

Copy link
Member

@discordianfish discordianfish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nikakis
Copy link
Contributor Author

nikakis commented Sep 10, 2022

@SuperQ Does this looks good to you? Can we merge it?

@SuperQ SuperQ merged commit 6f02dd1 into prometheus:master Sep 24, 2022
remijouannet pushed a commit to remijouannet/procfs that referenced this pull request Oct 20, 2022
…mp6 (prometheus#464)

Signed-off-by: Nikos Kakavas <nikakis@protonmail.com>

Signed-off-by: Nikos Kakavas <nikakis@protonmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants