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

Cherry pick: [Nokia][snmpwalk] Fix the snmpwalk cefcFruPowerStatusTable on Nokia LC platform #19733 #451

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

mlok-nokia
Copy link
Contributor

Why I did it

Cherry-pick PR sonic-net/sonic-buildimage#19733 to MSFT Repo 202205 branch

This PR will enable to Psud in LC to address this issue. Fixes sonic-net/sonic-buildimage#19535

Work item tracking
  • Microsoft ADO (number only):

How I did it

Cherry-pick PR sonic-net/sonic-buildimage#19733 to MSFT Repo 202205 branch.

Based on the snmp_agent common handling, it requires to enable the Pusd on the LC to set the "psu_num": 0 (as in the below entry) to allow the mibwalk handle the return value correctly. Enable the Psud on the LC although there is no PSU in LC.

How to verify it

  1. Executes "docker exec -it snmp snmpwalk -v2c -c 127.0.0.1 1.3.6.1.4.1.9.9.117.1.1.2.1"
  2. Check the syslog, there should not any error in the syslog which is related to this MIBwalk.

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

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

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

…C platform

Signed-off-by: mlok <[email protected]>
(cherry picked from commit fb6302e062dae1ee5cb24e639369c39fdb200808)
@mlok-nokia
Copy link
Contributor Author

@judyjoseph This is the cherry-pick for PR sonic-net/sonic-buildimage#19733

@BYGX-wcr
Copy link

BYGX-wcr commented Sep 8, 2024

@gechiang , please merge this PR

@BYGX-wcr BYGX-wcr merged commit 595d526 into Azure:202205 Sep 9, 2024
3 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.

3 participants