Skip to content

[DellEMC] S6000 - Thermal support - Last Reboot Reason#4097

Merged
lguohan merged 2 commits intosonic-net:masterfrom
santhosh-kt:s6000-thermal
Feb 6, 2020
Merged

[DellEMC] S6000 - Thermal support - Last Reboot Reason#4097
lguohan merged 2 commits intosonic-net:masterfrom
santhosh-kt:s6000-thermal

Conversation

@santhosh-kt
Copy link
Contributor

@santhosh-kt santhosh-kt commented Feb 3, 2020 •
edited
Loading

- What I did

  • Added support for Thermal event in Last Reboot Reason "show reboot-cause" command.
  • Added support for sending log message in case of thermal shutdown.
    sonic NOTICE root: Shutting down due to over temperature (40 degree, 30 degree, 34 degree)

- How I did it
On branch s6000-thermal
Changes to be committed:
(use "git reset HEAD ..." to unstage)

    modified:   platform/broadcom/sonic-platform-modules-dell/s6000/scripts/fancontrol.sh
    modified:   platform/broadcom/sonic-platform-modules-dell/s6000/sonic_platform/chassis.py

- How to verify it
Added the Unit Test Results.
Unit test - S6000.txt.txt
Thermal_reboot_loop_logs.txt.txt

For Testing, we instrumented the software to do thermal shutdown at 40 degree and temperature max hysteresis to 38 degree(Using sensors.conf).

- Description for the changelog
[DellEMC] S6000 - Thermal support - Last Reboot Reason

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

jleveque
jleveque approved these changes Feb 3, 2020
View reviewed changes
Copy link
Contributor

@jleveque jleveque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. @sujinmkang to review, as well.

@jleveque jleveque requested a review from sujinmkang February 3, 2020 19:06
@sujinmkang
Copy link
Collaborator

sujinmkang commented Feb 6, 2020

retest vsimage please

@lguohan lguohan merged commit 74e28ad into sonic-net:master Feb 6, 2020
abdosi pushed a commit that referenced this pull request Apr 27, 2020
@santhosh-kt @abdosi
[DellEMC] S6000 - Thermal support - Last Reboot Reason (#4097)
214541b 
- Added support for Thermal event in Last Reboot Reason "show reboot-cause" command.
- Added support for sending log message in case of thermal shutdown.

sonic NOTICE root: Shutting down due to over temperature (40 degree, 30 degree, 34 degree)
mssonicbld added a commit that referenced this pull request Feb 12, 2026
@mssonicbld
[submodule] Update submodule sonic-swss to the latest HEAD automatica…
a4e6620 
…lly (#25406)

#### Why I did it
src/sonic-swss
```
* 8eb38142 - (HEAD -> master, origin/master, origin/HEAD) Use the Inband Port's alias in getNeighborEntry if the nexthop's alias is Inband port (#4157) (2 hours ago) [saksarav-nokia]
* c79d2ecc - Add PORT_PHY_ATTR flex counter support (#3957) (9 hours ago) [Dhanasekar Rathinavel]
* ee45787b - Merge pull request #4098 from divyagayathri-hcl/13_multicast (24 hours ago) [StephenWangGoogle]
|\ 
| failure_prs.log skip_prs.log e2005cd2 - [P4Orch] Implement functions to verify state for multicast replication table entries. (32 hours ago) [mint570]
|/ 
* 0ad109f7 - Fix cargo bench compiling error (#4204) (33 hours ago) [Ze Gan]
* c269be5e - Merge pull request #4097 from divyagayathri-hcl/12_multicast (2 days ago) [StephenWangGoogle]
|\ 
| failure_prs.log skip_prs.log 2f27ddfe - [P4Orch] Implement functions to drain multicast replication table entries. (2 days ago) [mint570]
|/ 
* 35c247d5 - [macsecmgr]: Fix MACsec unconfigure failures due to time out (#4064) (2 days ago) [rajshekhar-nexthop]
* 07cb2217 - [tests] Add unit tests for CounterNameMapUpdater (#3972) (2 days ago) [Prasoon Saurav]
* cd68f864 - Merge pull request #4095 from divyagayathri-hcl/11_multicast (2 days ago) [StephenWangGoogle]
* 5ae47564 - [P4Orch] Implement functions for updating multicast replication entries. (5 days ago) [mint570]
```
#### How I did it
#### How to verify it
#### Description for the changelog
FengPan-Frank pushed a commit to FengPan-Frank/sonic-buildimage that referenced this pull request Mar 6, 2026
@mssonicbld @FengPan-Frank
[submodule] Update submodule sonic-swss to the latest HEAD automatica…
ae464b6 
…lly (sonic-net#25406)

#### Why I did it
src/sonic-swss
```
* 8eb38142 - (HEAD -> master, origin/master, origin/HEAD) Use the Inband Port's alias in getNeighborEntry if the nexthop's alias is Inband port (sonic-net#4157) (2 hours ago) [saksarav-nokia]
* c79d2ecc - Add PORT_PHY_ATTR flex counter support (sonic-net#3957) (9 hours ago) [Dhanasekar Rathinavel]
* ee45787b - Merge pull request sonic-net#4098 from divyagayathri-hcl/13_multicast (24 hours ago) [StephenWangGoogle]
|\
| failure_prs.log skip_prs.log e2005cd2 - [P4Orch] Implement functions to verify state for multicast replication table entries. (32 hours ago) [mint570]
|/
* 0ad109f7 - Fix cargo bench compiling error (sonic-net#4204) (33 hours ago) [Ze Gan]
* c269be5e - Merge pull request sonic-net#4097 from divyagayathri-hcl/12_multicast (2 days ago) [StephenWangGoogle]
|\
| failure_prs.log skip_prs.log 2f27ddfe - [P4Orch] Implement functions to drain multicast replication table entries. (2 days ago) [mint570]
|/
* 35c247d5 - [macsecmgr]: Fix MACsec unconfigure failures due to time out (sonic-net#4064) (2 days ago) [rajshekhar-nexthop]
* 07cb2217 - [tests] Add unit tests for CounterNameMapUpdater (sonic-net#3972) (2 days ago) [Prasoon Saurav]
* cd68f864 - Merge pull request sonic-net#4095 from divyagayathri-hcl/11_multicast (2 days ago) [StephenWangGoogle]
* 5ae47564 - [P4Orch] Implement functions for updating multicast replication entries. (5 days ago) [mint570]
```
#### How I did it
#### How to verify it
#### Description for the changelog

Signed-off-by: Feng Pan <[email protected]>
dprital pushed a commit that referenced this pull request Mar 19, 2026
@mssonicbld @dprital
[submodule] Update submodule sonic-swss to the latest HEAD automatica…
d3c76bc 
…lly (#25406)

#### Why I did it
src/sonic-swss
```
* 8eb38142 - (HEAD -> master, origin/master, origin/HEAD) Use the Inband Port's alias in getNeighborEntry if the nexthop's alias is Inband port (#4157) (2 hours ago) [saksarav-nokia]
* c79d2ecc - Add PORT_PHY_ATTR flex counter support (#3957) (9 hours ago) [Dhanasekar Rathinavel]
* ee45787b - Merge pull request #4098 from divyagayathri-hcl/13_multicast (24 hours ago) [StephenWangGoogle]
|\
| failure_prs.log skip_prs.log e2005cd2 - [P4Orch] Implement functions to verify state for multicast replication table entries. (32 hours ago) [mint570]
|/
* 0ad109f7 - Fix cargo bench compiling error (#4204) (33 hours ago) [Ze Gan]
* c269be5e - Merge pull request #4097 from divyagayathri-hcl/12_multicast (2 days ago) [StephenWangGoogle]
|\
| failure_prs.log skip_prs.log 2f27ddfe - [P4Orch] Implement functions to drain multicast replication table entries. (2 days ago) [mint570]
|/
* 35c247d5 - [macsecmgr]: Fix MACsec unconfigure failures due to time out (#4064) (2 days ago) [rajshekhar-nexthop]
* 07cb2217 - [tests] Add unit tests for CounterNameMapUpdater (#3972) (2 days ago) [Prasoon Saurav]
* cd68f864 - Merge pull request #4095 from divyagayathri-hcl/11_multicast (2 days ago) [StephenWangGoogle]
* 5ae47564 - [P4Orch] Implement functions for updating multicast replication entries. (5 days ago) [mint570]
```
#### How I did it
#### How to verify it
#### Description for the changelog

Signed-off-by: dprital <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@jleveque jleveque jleveque approved these changes

@sujinmkang sujinmkang sujinmkang approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Enhancement ➕ Included in 201911 Branch Platform: DellEMC Request for 201911 Branch :shipit:

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@santhosh-kt @sujinmkang @jleveque @lguohan @abdosi