-
Notifications
You must be signed in to change notification settings - Fork 634
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
Override help string #1105
Comments
SuperQ
added a commit
that referenced
this issue
Feb 4, 2024
Allow override of the metric HELP text in the generator. Allows for altering generated MIB documentation. * Update example help text for `st4TempSensorValue` to match scale override. Fixes: #1105 Signed-off-by: SuperQ <[email protected]>
SuperQ
added a commit
that referenced
this issue
Feb 4, 2024
Allow override of the metric HELP text in the generator. Allows for altering generated MIB documentation. * Update example help text for `st4TempSensorValue` to match scale override. Fixes: #1105 Signed-off-by: SuperQ <[email protected]>
SuperQ
added a commit
that referenced
this issue
Feb 28, 2024
Allow override of the metric HELP text in the generator. Allows for altering generated MIB documentation. * Update example help text for `st4TempSensorValue` to match scale override. Fixes: #1105 Signed-off-by: SuperQ <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Discussed in #1086
Originally posted by llamafilm January 3, 2024
It would be helpful to override the help text, for MIBs that are either lacking or inaccurate.
The help string is defined here:
snmp_exporter/generator/tree.go
Line 353 in 9c42d6c
I propose leaving that format as-is and allowing the user to override
description
in the generator.This is especially useful in conjunction with the recent addition of scaling factor in #1026. For example, this Cyberpower UPS returns all sensor values as integers, so after scaling the voltage to the correct value, the help text is wrong. This can be fixed in
snmp.yml
but I think we need a way to do it in the generator.The text was updated successfully, but these errors were encountered: