-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Hugepages input plugin #1447
Hugepages input plugin #1447
Conversation
A plugin that is described in #1176 |
) | ||
|
||
var hugepagesSampleConfig = ` | ||
## |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this line and 2-space indentation please :)
looks good, thanks @maksadbek, just the config formatting fix and I'll merge |
Ok, thanks |
for k, v := range numaNodeMetrics { | ||
metrics := make(map[string]interface{}) | ||
tags := map[string]string{ | ||
"node": k, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's the possible size of the number of unique occurences of "k"? (ie, is there a cardinality concern here?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Theoretically one might have up to 1023 (max number of NUMA nodes in Linux kernels these days). In the real world it will probably always be well under 10.
@maksadbek what's the status of this PR? if you rebase your changes it should clear up the test failure |
closing for inactivity, please feel free to open a new PR when it's ready |
Required for all PRs: