Update show platform syseeprom test#17601
Merged
yxieca merged 1 commit intosonic-net:masterfrom Jun 10, 2025
Merged
Conversation
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
SavchukRomanLv
force-pushed
the
show_platfrom_syseeprom
branch
from
156980a to
9f336c0
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
@SavchukRomanLv Could you give more information on how the read of plugins dir file content stopped? Is there any relevant PR? |
Contributor
If we look at the code of the CLI scripts, we can see that it has already been adapted to calling the platform API instead of using the plugin, https://github.com/sonic-net/sonic-utilities/blob/6ce5257459b6d36496bc98a282b5d78c5be5b530/scripts/decode-syseeprom#L31-L40 , relevant PR is here: sonic-net/sonic-utilities#1435 |
Collaborator
|
@prgeor kindly reminder to review and merge |
Contributor
Thank you for providing the info. changes LGTM. Thank you |
vvolam
approved these changes
Jun 10, 2025
r12f
approved these changes
Jun 10, 2025
Collaborator
|
Cherry-pick PR to msft-202412: Azure/sonic-mgmt.msft#386 |
mssonicbld
added
Included in msft-202412 Branch
and removed
Created PR to msft-202412 Branch
labels
mssonicbld
pushed a commit
to mssonicbld/sonic-mgmt
that referenced
this pull request
Jun 11, 2025
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
Collaborator
|
Cherry-pick PR to 202505: #18924 |
11 tasks
mssonicbld
pushed a commit
that referenced
this pull request
Jun 11, 2025
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
mssonicbld
added
Included in 202505 branch
and removed
Created PR to 202505 branch
labels
nissampa
pushed a commit
to nissampa/sonic-mgmt_dpu_test
that referenced
this pull request
Aug 7, 2025
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
opcoder0
pushed a commit
to opcoder0/sonic-mgmt
that referenced
this pull request
Dec 8, 2025
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
AharonMalkin
pushed a commit
to AharonMalkin/sonic-mgmt
that referenced
this pull request
Dec 16, 2025
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2
pushed a commit
to gshemesh2/sonic-mgmt
that referenced
this pull request
Dec 21, 2025
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop
pushed a commit
to venu-nexthop/sonic-mgmt
that referenced
this pull request
Jan 13, 2026
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
gshemesh2
pushed a commit
to gshemesh2/sonic-mgmt
that referenced
this pull request
Jan 26, 2026
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Feb 2, 2026
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
Signed-off-by: Yael Tzur <ytzur@nvidia.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
Summary:
SONiC stop read from /usr/share/sonic/device/{}/plugins/eeprom.py starting from 2020x release
Introduce new comparison between cli and db
Type of change
Back port request
Approach
What is the motivation for this PR?
Avoid TC failure if file removed
How did you do it?
Update test_show_platform_syseeprom.
Introduce new comparison between cli and db
How did you verify/test it?
Run test, TC passed
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation