Skip to content

[Mellanox] Implement new fan platform API#2747

Merged
lguohan merged 5 commits intosonic-net:masterfrom
keboliu:new-fan-api
Apr 21, 2019
Merged

[Mellanox] Implement new fan platform API#2747
lguohan merged 5 commits intosonic-net:masterfrom
keboliu:new-fan-api

Conversation

@keboliu
Copy link
Collaborator

@keboliu keboliu commented Apr 4, 2019

- What I did

  1. Implement new fan platform API for Mellanox platform
  2. Add fan object to PSU class and init the psu fan
  3. Init fan list in Chassis class.

- How I did it
add new file for fan API implemetation.

- How to verify it
write a python script to load chassis class then call the fan/psu API accordingly.

- Description for the changelog

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

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.

As comments

Copy link
Collaborator

@lguohan lguohan left a comment

Choose a reason for hiding this comment

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

:shipit:

@lguohan lguohan merged commit 818ba43 into sonic-net:master Apr 21, 2019
@keboliu keboliu deleted the new-fan-api branch May 24, 2019 07:01
MichelMoriniaux pushed a commit to criteo-forks/sonic-buildimage that referenced this pull request May 28, 2019
prsunny pushed a commit that referenced this pull request May 18, 2023
Updated for commit:

f141880 - 2023-04-26 : [bugfix] vnet ping missing with secondary endpoints empty in priority routes. (#2736) (#2747) [siqbal1986]
qiluo-msft pushed a commit that referenced this pull request Aug 10, 2023
…lly (#14865)

#### Why I did it
src/sonic-swss
```
* f141880 - (HEAD -> 202012, origin/202012) [bugfix] vnet ping missing with secondary endpoints empty in priority routes. (#2736) (#2747) (3 weeks ago) [siqbal1986]
```
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.

3 participants