-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Fix snmp v2 walk failure #37823
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
base: main
Are you sure you want to change the base?
Fix snmp v2 walk failure #37823
Conversation
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.
Some feedback for your review re: spelling and grammar.
- | | ||
List upgrade notes here, or remove this section. | ||
Upgrade notes should be rare: only list known/potential breaking changes, | ||
or major behaviorial changes that require user action before the upgrade. |
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.
or major behaviorial changes that require user action before the upgrade. | |
or major behavioral changes that require user action before the upgrade. |
List known issues here, or remove this section. | ||
deprecations: | ||
- | | ||
List deprecations notes here, or remove this section. |
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.
List deprecations notes here, or remove this section. | |
List deprecation notes here, or remove this section. |
other: | ||
- | | ||
Add here every other information you want in the CHANGELOG that | ||
don't fit in any other section. This section should rarely be |
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.
don't fit in any other section. This section should rarely be | |
doesn't fit in any other section. This section should rarely be |
Fixes for SNMP v2 compatibility with the `snmpwalk` command. | ||
other: | ||
- | | ||
Add here every other information you want in the CHANGELOG that |
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.
Add here every other information you want in the CHANGELOG that | |
Add any other information you want in the CHANGELOG that |
- | | ||
List enhancements (new behavior that is too small to be | ||
considered a new feature), or remove this section. | ||
issues: |
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.
should this be known issues or limitations?
What does this PR do?
snmp walk
subcommand to pullsnmp_version: 2
from configuration to be used for troubleshootingMotivation
snmp walk
subcommand currently only supports SNMP version 2c, but snmp integration configuration only allowssnmp_version: 2
Describe how you validated your changes
Possible Drawbacks / Trade-offs
Additional Notes