Skip to content

[202311][SNMP] Fix config template issue when setting snmpagentaddress#20154

Open
brholmes1 wants to merge 1 commit intosonic-net:202311from
brholmes1:202311_snmpagentaddress
Open

[202311][SNMP] Fix config template issue when setting snmpagentaddress#20154
brholmes1 wants to merge 1 commit intosonic-net:202311from
brholmes1:202311_snmpagentaddress

Conversation

@brholmes1
Copy link
Contributor

@brholmes1 brholmes1 commented Sep 5, 2024

Related to PR#16187
Cherry-pick PR#20152

Why I did it

When you configure an SNMP agent address with a VRF, the snmpd service will fail to properly start after the configuration has been applied. The snmpd service shows a status of "FATAL."

sudo config snmpagentaddress add 1.1.1.1 -p 161 -v mgmt

docker exec -it snmp supervisorctl status | awk '{print $1, $2}'
dependent-startup RUNNING
rsyslogd RUNNING
snmp-subagent STOPPED
snmpd FATAL
start EXITED
supervisor-proc-exit-listener RUNNING
Work item tracking
  • Microsoft ADO (number only):

How I did it

Corrected syntax in the jinja template used to generate the snmpd.conf within the SNMP docker container.

How to verify it

sudo config snmpagentaddress add 1.1.1.1 -p 161 -v mgmt

docker exec -it snmp supervisorctl status | awk '{print $1, $2}'
dependent-startup EXITED
rsyslogd RUNNING
snmp-subagent RUNNING
snmpd RUNNING
start EXITED
supervisor-proc-exit-listener RUNNING

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305
  • 202311
  • 202405

Tested branch (Please provide the tested image version)

  • SONiC.master.634739-2ff111cb9
  • SONiC.202405.565256-4d6d808b8
  • SONiC-OS-202311.634628-3cc3e1a11

Description for the changelog

Link to config_db schema for YANG module changes

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

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 5, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@brholmes1 brholmes1 force-pushed the 202311_snmpagentaddress branch 4 times, most recently from 9bbdea0 to 629cf46 Compare September 6, 2024 15:42
@brholmes1 brholmes1 force-pushed the 202311_snmpagentaddress branch from 629cf46 to 5ad1b81 Compare September 6, 2024 18:30
@brholmes1 brholmes1 marked this pull request as ready for review September 6, 2024 18:30
@brholmes1
Copy link
Contributor Author

@qiluo-msft Can you please help review? Thanks!

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.

1 participant