Skip to content

Coderabbit test#14

Open
dprital wants to merge 2 commits intomasterfrom
coderabbit_test
Open

Coderabbit test#14
dprital wants to merge 2 commits intomasterfrom
coderabbit_test

Conversation

@dprital
Copy link
Owner

@dprital dprital commented Sep 4, 2025

What I did

How I did it

How to verify it

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

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

tirupatihemanth and others added 2 commits July 30, 2025 17:38
…t#3988)

Why I did it
When running the test_fwutil_update_current test in https://github.com/sonic-net/sonic-mgmt/blob/master/tests/platform_tests/fwutil/test_fwutil.py we encounter the following issue when we run the command

admin@sonic:$ sudo fwutil update chassis component ONIE fw -y

Error: Failed to parse "platform_components.json": invalid platform schema: "module" key hasn't been found. Aborting...

Aborted!
This is because platform_components.json for smartswitch doesn't have module key in it and this should be ignored for smartswitch.

What I did
Avoid any modular chassis operations on smart-switch.

How I did it
Added a check to verify the device is not a smart switch before treating it as a modular chassis.
@dprital
Copy link
Owner Author

dprital commented Sep 9, 2025

bugbot run

@dprital
Copy link
Owner Author

dprital commented Sep 9, 2025

/run bugbot

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.

2 participants