-
Notifications
You must be signed in to change notification settings - Fork 66
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
Update cisco-catalyst.yml #771
Conversation
@thezackm made the requested changes. thankyou |
@thezackm Please roll this back, it broke the interface utilization for all catalyst devices. |
For more context, the Metric utilization metric (https://docs.newrelic.com/docs/network-performance-monitoring/troubleshooting/understanding-snmp-utilization-calculations/#interface-utilization) is calculated using the sysUptime (1.3.6.1.2.1.1.3) which is in 100ths of a second. snmpEngineTime (1.3.6.1.6.3.10.2.1.3) is in seconds and breaks the utilization metric. Given that sysUptime is defined in the ktranslate image (https://github.com/kentik/ktranslate/blob/65e861175b50a96ecffa85ce7659611ed737092c/pkg/inputs/snmp/mibs/json/SNMPv2-MIB.json#L103) as a more or less golden metric, this kind of change should be made in the configuration yaml at the customer level instead. |
This reverts commit bbefaa7.
thanks for the details @araddas @al3kh - can you please submit a feature request on the ktranslate repo so we can review other potential solutions for this? |
Hi, added new Uptime OID. @thezackm can you please approve. TIA