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.

Sign up for GitHub

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

Jump to bottom

[Mellanox] enhance the reboot cause handling on 201811 #3253

Merged
merged 3 commits into from
Aug 2, 2019
Merged

[Mellanox] enhance the reboot cause handling on 201811 #3253

merged 3 commits into from
Aug 2, 2019

Conversation

stephenxs
Copy link
Collaborator

@stephenxs stephenxs commented Aug 1, 2019 •
edited
Loading

- What I did

  1. Enable platform-api to be loaded as a whole so that other modules can import it simply by "import sonic_platform". Currently, only process-reboot-cause imports this module. Originally it uses "import sonic_platform.platform", which can fail when "platform" is not supported.
  2. Support currently hw-management currently running on 201811.

- How I did it

  1. add specific initialization to init.py for the module.
  2. adaptively find reboot cause file by searching for a series of possible dirs.

- How to verify it
test whether the reboot cause can be successfully parsed.

- Description for the changelog
[Mellanox] enable new platform api to be loaded as a whole.

- A picture of a cute animal (not mandatory but encouraged)

@stephenxs stephenxs changed the title [Mellanox] enable new platform api to be loaded as a whole [Mellanox] enhance the reboot cause handling on 201811 Aug 1, 2019
@lguohan lguohan requested a review from jleveque August 2, 2019 04:28
@jleveque jleveque merged commit ca38147 into sonic-net:201811 Aug 2, 2019
@stephenxs stephenxs deleted the enable-load-mlnx-platform-api-as-a-whole branch August 2, 2019 21:36
mssonicbld added a commit that referenced this pull request Jun 6, 2024
@mssonicbld
[submodule] Update submodule sonic-utilities to the latest HEAD autom…
dadf0dd 
…atically (#19225)

#### Why I did it
src/sonic-utilities
```
* 1ebd0993 - (HEAD -> master, origin/master, origin/HEAD) Fix show fabric monitor capacity command when the feature is disabled. (#3347) (8 hours ago) [jfeng-arista]
* 6829ded4 - Add W-ECMP CLI (#3253) (10 hours ago) [Nazarii Hnydyn]
* 4133ef54 - [chassis][voq] Added support for Voq Counters(SAI_SWITCH_STAT_PACKET_INTEGRITY_DROP,SAI_QUEUE_STAT_CREDIT_WD_DELETED_PACKETS) for Voq/Fabric switches (#3322) (25 hours ago) [saksarav-nokia]
```
#### How I did it
#### How to verify it
#### Description for the changelog
arun1355492 pushed a commit to arun1355492/sonic-buildimage that referenced this pull request Jul 26, 2024
@mssonicbld @arun1355492
[submodule] Update submodule sonic-utilities to the latest HEAD autom…
5728f54 
…atically (sonic-net#19225)

#### Why I did it
src/sonic-utilities
```
* 1ebd0993 - (HEAD -> master, origin/master, origin/HEAD) Fix show fabric monitor capacity command when the feature is disabled. (sonic-net#3347) (8 hours ago) [jfeng-arista]
* 6829ded4 - Add W-ECMP CLI (sonic-net#3253) (10 hours ago) [Nazarii Hnydyn]
* 4133ef54 - [chassis][voq] Added support for Voq Counters(SAI_SWITCH_STAT_PACKET_INTEGRITY_DROP,SAI_QUEUE_STAT_CREDIT_WD_DELETED_PACKETS) for Voq/Fabric switches (sonic-net#3322) (25 hours ago) [saksarav-nokia]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jleveque jleveque jleveque approved these changes

Assignees
No one assigned
Labels
Enhancement ➕ Included in 201811 Branch Platform: Mellanox Request for 201811 Branch :shipit:
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@stephenxs @jleveque @yxieca @andriymoroz-mlnx