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

[Juniper][QFX5210] Workaround for orchagent crash #3458

Merged
merged 7 commits into from
Sep 20, 2019
Merged

[Juniper][QFX5210] Workaround for orchagent crash #3458

merged 7 commits into from
Sep 20, 2019

Conversation

ciju-juniper
Copy link
Contributor

Orchagent is crashing on the latest SONiC images. Issue #458 is raised to track the problem. Dynamic port breakout commit 6f40933 introduced this regression. One of the recommendation that we received was to disable the loopback and mgmt ports in bcm config. It was helpful in fixing the issue but not the ideal solution. It works on qfx5210 as the loopback and management configurations are not applicable.

While the orchagent crash is being debugged, we need this commit to fix the issue in qfx5210 platform. This patch can be reversed once the correct fix for the orchagent is identified.

- How to verify it

  1. 'docker ps' is listing all the docker containers up and running. No crashes seen
  2. 'show interfaces status' lists the functional ports correctly

This switch has 64 QSFP28 (40G/100G) ports, 2 SFP+ (1G/10G) ports
on Broadcom Tomahawk II chipset. CPU used in QFX5210-64C-S is
Intel Broadwell-DE. The machine has Redundant and hot-swappable
Power Supply (1+1) and also has Redundant and hot swappable fans (3+1).

Signed-off-by: Ciju Rajan K <[email protected]>
Signed-off-by: Ciju Rajan K <[email protected]>
This commit implements the following changes
 - Moving to optoe driver for sfp management
 - Removing the old sfp driver
 - Updating the port-config.ini to add the index field
 - Correction in sfputil.py to incorporate optoe driver
 - Platform support for 'poweroff' command

Signed-off-by: Ciju Rajan K <[email protected]>
Orchagent is crashing on the latest SONiC images. Issue #458
is raised to track the problem. Dynamic port breakout
commit 6f40933 introduced
this regression. One of the recommendation that we received
was to disable the loopback and mgmt ports in bcm config.
It was helpful in fixing the issue but not the ideal solution.
It works on qfx5210 as the loopback and management configurations
are not applicable.

While the orchagent crash is being debugged, we need this commit
to fix the issue in qfx5210 platform. This patch can be reversed
once the correct fix for the orchagent is identified.

Signed-off-by: Ciju Rajan K <[email protected]>
@ciju-juniper
Copy link
Contributor Author

@lguohan Please merge this patch

@ciju-juniper
Copy link
Contributor Author

@lguohan Could you take this patch? This will enable us to test the daily SONiC builds and arrive at a good label.

@lguohan lguohan merged commit 766b15d into sonic-net:master Sep 20, 2019
mssonicbld added a commit that referenced this pull request Jul 31, 2024
…atically (#19742)

#### Why I did it
src/sonic-utilities
```
* ff2c73f8 - (HEAD -> master, origin/master, origin/HEAD) Add namespace check for multiasic (#3458) (4 hours ago) [Xincun Li]
```
#### 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants