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.

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

DellEMC S6000 : Platform2.0 API implementation [PSU, Thermal] #3357

Merged
merged 7 commits into from
Sep 18, 2019
Merged

DellEMC S6000 : Platform2.0 API implementation [PSU, Thermal] #3357

merged 7 commits into from
Sep 18, 2019

Conversation

ArunSaravananBalachandran
Copy link
Contributor

- What I did

Implement PSU and Thermal platform2.0 API for DellEMC S6000 platform

- How I did it

Added new files psu.py and thermal.py for PSU and thermal platform2.0 API implementation

- How to verify it

Wrote python script to load Chassis class and then call PSU and Thermal APIs accordingly.
UT logs : UT-logs_PSU.txt , UT-logs_thermal.txt

- Description for the changelog

DellEMC S6000 : Platform2.0 API implementation [PSU, Thermal]

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

not
"""
# Hardware not supported
return False
Copy link
Contributor

Choose a reason for hiding this comment

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

The S6000 has status LEDs on each PSU and also one power status LED on the front panel. How are these controlled?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In S6000, the firmware running in the PSU controls the individual PSU's status LED.
The power status LED on the front panel is controlled via CPLD.

Copy link
Contributor

Choose a reason for hiding this comment

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

So the CPLD will manage the power status LED on the front panel without software assistance?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The front panel power status LED needs to be controlled by software via CPLD.

Copy link
Contributor

Choose a reason for hiding this comment

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

OK. Do you intend to control the power status LED somewhere other than here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Currently, the front panel power status LED is not controlled here. If required we can take this as separate work-item.

@lguohan lguohan requested a review from sujinmkang August 29, 2019 04:47
@sujinmkang
Copy link
Collaborator

@ArunSaravananBalachandran can you please resolve the conflicts?

@ArunSaravananBalachandran
Copy link
Contributor Author

@ArunSaravananBalachandran can you please resolve the conflicts?

Merge conflicts have been resolved.

@sujinmkang sujinmkang merged commit 6ca0d7b into sonic-net:master Sep 18, 2019
@ArunSaravananBalachandran ArunSaravananBalachandran deleted the platfrom2.0_s6000 branch September 19, 2019 02:55
mssonicbld added a commit that referenced this pull request Jun 11, 2024
…atically (#19274)

#### Why I did it
src/sonic-utilities
```
* c2370f88 - (HEAD -> master, origin/master, origin/HEAD) [DPB]Fixing return code of breakout command on failure (#3357) (8 hours ago) [Sudharsan Dhamal Gopalarathnam]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this pull request Jun 15, 2024
…atically (#19284)

#### Why I did it
src/sonic-utilities
```
* ea9497a3 - (HEAD -> 202311, origin/202311) [consutil] Fix consule CLI and enhance unittest (#3360) (3 days ago) [Zhijian Li]
* 68f89d64 - [DPB]Fixing return code of breakout command on failure (#3357) (3 days ago) [Sudharsan Dhamal Gopalarathnam]
```
#### 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
…atically (sonic-net#19274)

#### Why I did it
src/sonic-utilities
```
* c2370f88 - (HEAD -> master, origin/master, origin/HEAD) [DPB]Fixing return code of breakout command on failure (sonic-net#3357) (8 hours ago) [Sudharsan Dhamal Gopalarathnam]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this pull request Aug 3, 2024
…atically (#19806)

#### Why I did it
src/sonic-utilities
```
* cee3e8e0 - (HEAD -> 202405, origin/202405) Enable show ip bgp on sup and -n all for show ip bgp network (#3417) (4 hours ago) [Changrong Wu]
* 26f5d757 - Add the definition of `log` in `script decode-syseeprom` (#3383) (4 hours ago) [Yutong Zhang]
* af18c89c - Improve load_mingraph to wait eth0 restart before exit (#3365) (4 hours ago) [Hua Liu]
* 73e12564 - [chassis][mutli-asic][lldp] "show lldp table/neighbors" displays error message in output on multi-asis linecard. (#3358) (4 hours ago) [Marty Y. Lok]
* 69dab7ac - [pbh]: Fix show PBH counters when cache is partial (#3356) (4 hours ago) [Nazarii Hnydyn]
* ae20defd - [fast-reboot] Backup database after syncd/swss stopped (#3342) (4 hours ago) [Stepan Blyshchak]
* a9876ee6 - Add Checkpoint and Rollback for Multi ASIC. (#3299) (4 hours ago) [Xincun Li]
* 7643ac3c - [DPB]Fixing return code of breakout command on failure (#3357) (4 hours ago) [Sudharsan Dhamal Gopalarathnam]
```
#### 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants