Skip to content

Remove fan LED initialization from led_control plugins for Arista 7050-QX32, 7050-QX-32S#1733

Merged
jleveque merged 1 commit intosonic-net:masterfrom
jleveque:arista_led_fix
May 24, 2018
Merged

Remove fan LED initialization from led_control plugins for Arista 7050-QX32, 7050-QX-32S#1733
jleveque merged 1 commit intosonic-net:masterfrom
jleveque:arista_led_fix

Conversation

@jleveque
Copy link
Copy Markdown
Contributor

@jleveque jleveque commented May 22, 2018

In aristanetworks/sonic@6e35bf5, the mapping of color values changed for the 7050-QX, causing the values in led_control.py to set the LEDs to yellow/orange rather than green as they had previously. However, also in this commit, Arista also began initializing the fan LEDs to green rather than off, as they had previously. Therefore, there is no longer a need to initialize the fan LEDs to green in led_control.py.

Similar changes were implemented for the 7050-QX-32-S here: aristanetworks/sonic@561f785, however there was no remapping of colors, so these devices were not impacted. I am also removing the fan LED initialization for the SKU because it is now also redundant. #

@jleveque jleveque self-assigned this May 22, 2018
@jleveque jleveque requested a review from lguohan May 22, 2018 22:10
@jleveque jleveque merged commit 6fc38af into sonic-net:master May 24, 2018
@jleveque jleveque deleted the arista_led_fix branch May 24, 2018 00:57
lguohan pushed a commit that referenced this pull request May 26, 2018
BasimShalata added a commit to BasimShalata/sonic-buildimage that referenced this pull request Aug 4, 2021
This PR contains the following commits:
54b74a2 2021-08-04 [LLDP] Fix lldpshow script to enable display multiple MAC addresses on the same remote physical interface (sonic-net#1657)
0d53b7a 2021-08-03 [sonic_installer] don't print errors when installing an image not supporting app ext (sonic-net#1719)
394e2fb 2021-08-03 Implement script null_route_helper (sonic-net#1737)
dd01b56 2021-08-02 disk_check updates: (sonic-net#1736)
8a74d03 2021-07-30 [CLI][show][bgp] Fix the show ip bgp network command (sonic-net#1733)
679a4ba 2021-07-30 [MACsec]: Allow upgrade-docker for macsec container (sonic-net#1716)
e9c73e8 2021-07-28 [CLI][MPLS][Show] Added multi ASIC support for 'show mpls command'.


Signed-off-by: Basim Shalata <basims@nvidia.com>
lguohan pushed a commit that referenced this pull request Aug 6, 2021
This PR contains the following commits:
54b74a2 2021-08-04 [LLDP] Fix lldpshow script to enable display multiple MAC addresses on the same remote physical interface (#1657)
0d53b7a 2021-08-03 [sonic_installer] don't print errors when installing an image not supporting app ext (#1719)
394e2fb 2021-08-03 Implement script null_route_helper (#1737)
dd01b56 2021-08-02 disk_check updates: (#1736)
8a74d03 2021-07-30 [CLI][show][bgp] Fix the show ip bgp network command (#1733)
679a4ba 2021-07-30 [MACsec]: Allow upgrade-docker for macsec container (#1716)
e9c73e8 2021-07-28 [CLI][MPLS][Show] Added multi ASIC support for 'show mpls command'.


Signed-off-by: Basim Shalata <basims@nvidia.com>
judyjoseph added a commit that referenced this pull request Aug 7, 2021
8b149a3 Load the  database global_db only once for show cli  (#1712)
cd0e560 [config][interface][speed] Fixed the config interface speed in multiasic issue (#1739)
b595ba6 [fast-reboot] revert the change of disabling counter polling before fast-reboot (#1744)
8518820 [minigraph] Donot enable PFC watchdog for MgmtTsToR (#1734)
2213774 [CLI][show][bgp] Fix the show ip bgp network command (#1733)
3526507 [configlet] Python3 compatible syntax for extracting a key from the dict (#1721)
5b56b97 [sonic_installer] don't print errors when installing an image not supporting app ext (#1719)
a581955 [LLDP] Fix lldpshow script to enable display multiple MAC addresses on the same remote physical interface (#1657)
theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this pull request Feb 5, 2022
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this pull request Apr 28, 2022
Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
noaOrMlnx pushed a commit to noaOrMlnx/sonic-buildimage that referenced this pull request Nov 24, 2025
<!--
Please make sure you've read and understood our contributing guidelines:
     https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

** Make sure all your commits include a signature generated with `git
commit -s` **

If this is a bug fix, make sure your description includes "fixes #xxxx",
or
     "closes #xxxx" or "resolves #xxxx"

     Please provide the following information:
-->

#### Why I did it
This is a manual cherry-pick of
sonic-net#24289
Upgrade the xgs SAI version to 13.2.1.10 to include the following
changes:
- OCP SAI spec Version upgrade from 1.16.3 -> 1.16.4
- TH_Regular : Memory leak is observed during warm shut and cold shut
during warm boot.
- Uplift SDKLT-51968 to 13.2.1: Drop counter is increasing even if port
is Oper down
- On TH4GT: Need to disable TunnelMpIp4ip6 tests as per Jira
SONIC-108933
- Updated PORT attribute PRIORITY_FLOW_CONTROL handling for TH5 with a
custom value 2 for config property sai_pfc_dlr_init_capability
##### Work item tracking
- Microsoft ADO **(number only)**: 35635641

#### How I did it
Upgrade the SAI version in the sai.mk file.

#### How to verify it


Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>

Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants