Skip to content

[sonic_fan]Add constant for fan direction#76

Merged
jleveque merged 1 commit intosonic-net:masterfrom
stephenxs:plugin_fan_dir
Jan 22, 2020
Merged

[sonic_fan]Add constant for fan direction#76
jleveque merged 1 commit intosonic-net:masterfrom
stephenxs:plugin_fan_dir

Conversation

@stephenxs
Copy link
Collaborator

- What I did
Add string constants for fan directions
- How I did it

@liat-grozovik
Copy link
Collaborator

@jleveque can you please review?

@jleveque jleveque merged commit b11ab3d into sonic-net:master Jan 22, 2020
@stephenxs stephenxs deleted the plugin_fan_dir branch January 26, 2020 07:54
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
…ic-net#78)

Merge of QSFP-DD support via PR# sonic-net#66
is making xcvrd to bail out on platforms that do not support
new key/attribute.

Fix is to ignore new key/attribute if not present/supported yet.

Fix takes changes done as part of PR#sonic-net#76 which can be closed
once this gets merged

Verified manually xcvrd running fine after above changes.
also command show interface transceiver is showing correct o/p

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-platform-common that referenced this pull request May 7, 2025
Code sync sonic-net/sonic-platform-common:202411 => 202412

```
*   d151d69 (HEAD -> code-sync-202412, origin/code-sync-202412) r12f 250417:0530 - Merge remote-tracking branch 'base/202411' into code-sync-202412
|\  
| * f679a67 (base/202411) mssonicbld 250304:0402 - Restore optoe page to default 0 for active cables (sonic-net#550)
| * c735073 Aryeh Feigin 250212:1628 - [202411][cmis] Fix cmis.get_error_description speed for passive module (sonic-net#538)
| * b7e75d8 mssonicbld 250207:0102 - Add 800G innolight PNs (sonic-net#540)
```
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.

4 participants