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

[broadcom]: fix delay to load led proc init #1940

Merged
merged 1 commit into from
Aug 17, 2018
Merged

Conversation

lguohan
Copy link
Collaborator

@lguohan lguohan commented Aug 17, 2018

led start bcm cmd triggers link state notifications.
This is quick fix to avoid such fake link state notifications.

Signed-off-by: Guohan Lu gulv@microsoft.com

- What I did
led start bcm cmd triggers link state notifications.
This is quick fix to avoid such fake link state notifications.

- How I did it
change the sleep time to 3 seconds

- How to verify it
10 times reload swss to verify led status are loaded while no start.sh error return on multiple DUT.

- Description for the changelog

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

led start bcm cmd triggers link state notifications.
This is quick fix to avoid such fake link state notifications.

Signed-off-by: Guohan Lu <gulv@microsoft.com>
@jleveque
Copy link
Contributor

Retest this please

@lguohan lguohan merged commit ec8a7f5 into sonic-net:master Aug 17, 2018
lguohan added a commit that referenced this pull request Aug 17, 2018
led start bcm cmd triggers link state notifications.
This is quick fix to avoid such fake link state notifications.

Signed-off-by: Guohan Lu <gulv@microsoft.com>
abdosi pushed a commit that referenced this pull request Jan 21, 2022
Update the sonic-swss submodule. The following are new commits in the submodule:

6cb43ee [p4orch] Fix handlePortStatusChangeNotification status deserialize (#2111)
863f0f1 [azp]: Enable PR diff coverage (#2083)
bf4cd4a Fix the unsafe usage of strncpy in portsorch.cpp (#2110)
c1b4b40 support port isolation group in BFN platform (#1940)


Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>
theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this pull request Feb 5, 2022
Signed-off-by: Anton <antonx.novikau@intel.com>
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.

None yet

3 participants