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

SNMP Default Contact addition to configDB #13469

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

praveen-ramamoorthi
Copy link

Why I did it

SNMP Default contact value is added as sysContact Azure Cloud Switch vteam <linuxnetdev@microsoft.com> in snmpd.conf during container bringup , but not updated in configDB . This creates a discrepancy between snmpget and show runningconfiguration snmp output. Hence added common default SNMP contact value.

How I did it

Added the default SNMP Contact value in snmp.yml and parsed it via snmp_yml_to_configdb.py to add it to configDB

How to verify it

Check show runningconfiguration snmp , snmpget and /etc/snmp/snmpd.conf in snmp container for default values

admin@sonic:~$ show runningconfiguration snmp
Location
----------
public


SNMP_CONTACT    SNMP_CONTACT_EMAIL
--------------  -------------------------------
SONiC-team      <sonicproject@googlegroups.com>


Community String    Community Type
------------------  ----------------
public              RO


User    Permission Type    Type    Auth Type    Auth Password    Encryption Type    Encryption Password
------  -----------------  ------  -----------  ---------------  -----------------  ---------------------
admin@sonic:~$

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

SNMP Default Contact addition to configDB via snmp.yml

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

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