Skip to content

Fix ssdhealth failure on multi-asic VS platform#3743

Merged
kperumalbfn merged 1 commit intosonic-net:masterfrom
vivekrnv:fix_ssdhealth_cli
Jan 31, 2025
Merged

Fix ssdhealth failure on multi-asic VS platform#3743
kperumalbfn merged 1 commit intosonic-net:masterfrom
vivekrnv:fix_ssdhealth_cli

Conversation

@vivekrnv
Copy link
Copy Markdown
Contributor

What I did

show platform ssdhealth is failing on multi asic vs platform

E sys.exit(cli())
E ^^^^^
E File "/usr/local/lib/python3.11/dist-packages/click/core.py", line 764, in call
E return self.main(*args, **kwargs)
E ^^^^^^^^^^^^^^^^^^^^^^^^^^
E File "/usr/local/lib/python3.11/dist-packages/click/core.py", line 717, in main
E rv = self.invoke(ctx)
E ^^^^^^^^^^^^^^^^
E File "/usr/local/lib/python3.11/dist-packages/click/core.py", line 1137, in invoke
E return _process_result(sub_ctx.command.invoke(sub_ctx))
E ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
E File "/usr/local/lib/python3.11/dist-packages/click/core.py", line 1137, in invoke
E return _process_result(sub_ctx.command.invoke(sub_ctx))
E ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
E File "/usr/local/lib/python3.11/dist-packages/click/core.py", line 956, in invoke
E return ctx.invoke(self.callback, **ctx.params)
E ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
E File "/usr/local/lib/python3.11/dist-packages/click/core.py", line 555, in invoke
E return callback(*args, **kwargs)
E ^^^^^^^^^^^^^^^^^^^^^^^^^
E File "/usr/local/lib/python3.11/dist-packages/show/platform.py", line 117, in ssdhealth
E device = platform_data.get("chassis", {}).get("disk", {}).get("device", None)
E ^^^^^^^^^^^^^^^^^
E AttributeError: 'NoneType' object has no attribute 'get'

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@kperumalbfn kperumalbfn merged commit 7f3957c into sonic-net:master Jan 31, 2025
6 checks passed
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202411: #3744

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.

5 participants