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

Fixes invalid namespace issues that would occur when making sfputil C… #2318

Closed
wants to merge 2 commits into from

Conversation

bmridul
Copy link
Contributor

@bmridul bmridul commented Aug 17, 2022

What I did

Fixed invalid namespace issues that would occur when making sfputil CLI calls.

How I did it

Made a call to load_db_config() in load_port_config()

How to verify it

sfputil show presence

Previous command output (if the output of a command-line utility has changed)

root@sonic:/home/cisco# sfputil show presence
Error reading port info (:- validateNamespace: Initialize global DB config using API SonicDBConfig::initializeGlobalConfig)

New command output (if the output of a command-line utility has changed)

root@sonic:/home/cisco# sfputil show presence
Port        Presence
----------  -----------
Ethernet18  Not present
Ethernet38  Not present
Ethernet45  Not present
Ethernet41  Not present
Ethernet44  Not present
Ethernet47  Not present
Ethernet28  Not present
Ethernet36  Not present
Ethernet7   Present
Ethernet19  Not present
Ethernet16  Not present
Ethernet25  Not present
Ethernet29  Not present
Ethernet43  Not present
Ethernet32  Not present
Ethernet33  Not present
Ethernet11  Not present
Ethernet2   Not present
Ethernet10  Not present
Ethernet6   Present
Ethernet22  Not present
Ethernet42  Not present
Ethernet37  Not present
Ethernet5   Present
Ethernet30  Not present
Ethernet9   Not present
Ethernet14  Not present
Ethernet26  Not present
Ethernet46  Not present
Ethernet31  Not present
Ethernet35  Not present
Ethernet17  Not present
Ethernet4   Not present
Ethernet8   Present
Ethernet1   Not present
Ethernet20  Not present
Ethernet24  Not present
Ethernet23  Not present
Ethernet27  Not present
Ethernet40  Not present
Ethernet3   Not present
Ethernet12  Not present
Ethernet21  Not present
Ethernet0   Present
Ethernet39  Not present
Ethernet34  Not present
Ethernet13  Not present
Ethernet15  Not present

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 17, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: bmridul / name: Mridul Bajpai (cfdbf82)

@bmridul
Copy link
Contributor Author

bmridul commented Sep 12, 2022

@abdosi @rlhui
Pls assign someone to review.

@abdosi
Copy link
Contributor

abdosi commented Sep 26, 2022

@mlok-nokia can you please help review this

@mlok-nokia
Copy link
Contributor

mlok-nokia commented Sep 26, 2022

@mlok-nokia can you please help review this
This is the similar to the issue which I have created the PR for. But My PR only addresses "sfputil show error-status", Not address "sfputil show presence". Review is done
#2373

@mlok-nokia
Copy link
Contributor

@abdosi The reason why I separated the "sudo sfputil show presence/eeprom" and "sudo sfputil show error-status" is: There is no problem with ""sudo sfputil show presence/eeprom"" command on msft-2205 branch, this two command only occurs on Master branch. The msft-2205 branch only has the "sudo sfputil show error-status" issue.

@bmridul bmridul closed this Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants