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

Limit SNMP communities #35

Merged
merged 1 commit into from
Jul 12, 2024
Merged

Limit SNMP communities #35

merged 1 commit into from
Jul 12, 2024

Conversation

cpaillet
Copy link
Collaborator

@cpaillet cpaillet commented Jul 10, 2024

As Sonic devices do not accept more than one SNMP community, we prefer to limit to the lowest common denominator.

When you try to add more than 1 community, you have this message

Screenshot 2024-07-10 at 09-45-57 Add a new snmp NetBox

@cpaillet cpaillet force-pushed the snmp branch 2 times, most recently from db196cb to 4b1b4dd Compare July 10, 2024 07:47
@cpaillet cpaillet force-pushed the snmp branch 2 times, most recently from 75124c7 to e0edf15 Compare July 10, 2024 08:21
As Sonic devices do not accept more than one SNMP community, we prefer to limit to the lowest
common denominator.
@cpaillet cpaillet requested a review from a team July 10, 2024 09:02
@cpaillet cpaillet merged commit be064c4 into criteo:main Jul 12, 2024
2 checks passed
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.

2 participants