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] Updating platform README #3746

Merged
merged 21 commits into from
Nov 13, 2019
Merged

[Juniper][QFX5210] Updating platform README #3746

merged 21 commits into from
Nov 13, 2019

Conversation

ciju-juniper
Copy link
Contributor

Modified the platform README to include the 'FEC' mode configuration for 100G optics

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 <crajank@juniper.net>
Signed-off-by: Ciju Rajan K <crajank@juniper.net>
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 <crajank@juniper.net>
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 <crajank@juniper.net>
This patch does the following:
 - Storing eeprom contents including the vendor extensions
   into a text file
 - Implement the get routines for reading various fields
   from eeprom

Signed-off-by: Ciju Rajan K <crajank@juniper.net>
This patch implements the following:
 - Added logging facility in juniper_qfx5210_util file
 - Changed the path of juniper_qfx5210_monitor.log to /var/log
 - Invoke "poweroff" command when temperature reaches fire threshold
 - Display warning messages in the console before system shutdown

Signed-off-by: Ciju Rajan K <crajank@juniper.net>
There are primary and secondary bios in qfx5210 platform.
There is a problem which prevents the OS booting from the
secondary bios when the OS was installed using primary bios.
Secondary bios fails to detect the UEFI partition. Right now
the workaround is to have a folder structure /EFI/BOOT/BOOT64x.efi

Signed-off-by: Ciju Rajan K <crajank@juniper.net>
There is a cpld swizzle logic which will reboot qfx5210
platform if the swizzle logic was not disabled in five minutes.

Signed-off-by: Ciju Rajan K <crajank@juniper.net>
This patch set implements the following:
 - Adding the cpld soft reset as the platform reboot handler
 - Fixing the cpld access issues

Signed-off-by: Ciju Rajan K <crajank@juniper.net>
Signed-off-by: Ciju Rajan K <crajank@juniper.net>
Added the documentation for the platform poweroff/reboot

Signed-off-by: Ciju Rajan K <crajank@juniper.net>
This patch adds the preemphasis values for the various supported
optics for qfx5210 platform

Signed-off-by: Ciju Rajan K <crajank@juniper.net>
Added the steps for configuring FEC mode

Signed-off-by: Ciju Rajan K <crajank@juniper.net>
@lguohan lguohan merged commit 7372122 into sonic-net:master Nov 13, 2019
@lguohan
Copy link
Collaborator

lguohan commented Nov 13, 2019

can you follow PR convention here. Rebase your PR on top of master branch. It is challenging to merge your PR with correct comments since your branch diverges a lot from master.

https://github.com/edx/edx-platform/wiki/How-to-Rebase-a-Pull-Request

@ciju-juniper
Copy link
Contributor Author

Done. My fork is now even with Azure master branch. Thanks!

zhenggen-xu pushed a commit to zhenggen-xu/sonic-buildimage that referenced this pull request Jan 10, 2020
* Updating the platform README

Added the steps for configuring FEC mode

Signed-off-by: Ciju Rajan K <crajank@juniper.net>
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