[Nokia][snmpwalk] Fix the snmpwalk cefcFruPowerStatusTable on Nokia LC platform #19733
Conversation
…C platform Signed-off-by: mlok <[email protected]>
|
/azp run |
|
Commenter does not have sufficient privileges for PR 19733 in repo sonic-net/sonic-buildimage |
SuvarnaMeenakshi
added
Request for 202405 Branch
Chassis for 202205 branch
|
@judyjoseph @rlhui can you help approve and merge this PR. |
ms_Checker failing |
|
/ms_checker |
|
/azpw ms_checker -f |
|
@qiluo-msft : please help with merge of this. |
|
MS ADO : 27241572 |
|
@SuvarnaMeenakshi @qiluo-msft Please don't cherry-pick this PR to 202205 branch. Takes this change to 202205 branch will require other PR to be cherry-pick to the Nokia 202205 sonic_platform submodule |
@mlok-nokia do you know what is the list of PRs that is required in Nokia 202205 sonic_platform submodule? |
@mlok-nokia Can you help share list of PRs that is required in 202205 to ensure that this can be cherry-picked for 202205? |
Cherry-pick this PR to 202205 requires to merge PR Azure/sonic-buildimage-msft#449 |
Thank you @mlok-nokia , I see that the above change is already present in 202205 branch. So we should be good, as long as this is the only dependent PR. |
This is the only dependency. PR Azure/sonic-buildimage-msft#449 is to update the submodule on 202205 branch to pick the change to work with this PR. |
…C platform (sonic-net#19733) 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" Nokia LC - Nokia_IXR7250E_36x400G will get error in the syslog. This PR will enable Psud in LC to address this issue. Fixes sonic-net#19535 Signed-off-by: mlok <[email protected]>
|
Cherry-pick PR to 202405: #20181 |
…C platform (#19733) 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" Nokia LC - Nokia_IXR7250E_36x400G will get error in the syslog. This PR will enable Psud in LC to address this issue. Fixes #19535 Signed-off-by: mlok <[email protected]>
mssonicbld
added
Included in 202405 Branch
and removed
Created PR to 202405 Branch
labels
…C platform (sonic-net#19733) 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" Nokia LC - Nokia_IXR7250E_36x400G will get error in the syslog. This PR will enable Psud in LC to address this issue. Fixes sonic-net#19535 Signed-off-by: mlok <[email protected]>
Why I did it
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" Nokia LC - Nokia_IXR7250E_36x400G will get the following error in the syslog:
Jul 30 02:03:21.836386 ixre-egl-board9 ERR snmp#snmp-subagent [sonic_ax_impl] ERROR: PowerStatusHandler._get_psu_index() caught an unexpected exception during _get_num_psus()#012Traceback (most recent call last):#12 File "/usr/local/lib/python3.9/dist-packages/sonic_ax_impl/mibs/vendor/cisco/ciscoEntityFruControlMIB.py", line 101, in _get_psu_index#012 num_psus = self._get_num_psus()#12 File "/usr/local/lib/python3.9/dist-packages/sonic_ax_impl/mibs/vendor/cisco/ciscoEntityFruControlMIB.py", line 66, in _get_num_psus#012 return int(num_psus[0])#012ValueError: invalid literal for int() with base 10: ''
This PR will enable to Psud in LC to address this issue. Fixes #19535
Work item tracking
How I did it
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
Which release branch to backport (provide reason below if selected)
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)