Add gbsyncd container for broncos #11154
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6 tasks
jimmyzhai
reviewed
Jun 29, 2022
jimmyzhai
reviewed
Jun 29, 2022
jimmyzhai
reviewed
Jun 29, 2022
jimmyzhai
reviewed
Jun 29, 2022
7 tasks
lguohan
approved these changes
Jul 6, 2022
jimmyzhai
reviewed
Jul 7, 2022
|
|
||
| $(DOCKER_GBSYNCD_BRONCOS)_FILES += $(SUPERVISOR_PROC_EXIT_LISTENER_SCRIPT) | ||
|
|
||
| $(DOCKER_GBSYNCD_BRONCOS)_LOAD_DOCKERS += $(DOCKER_CONFIG_ENGINE_BUSTER) |
There was a problem hiding this comment.
Please update base docker from buster to bullseye like #11288.
jimmyzhai
reviewed
Jul 7, 2022
| @@ -0,0 +1,40 @@ | |||
| FROM docker-config-engine-buster | |||
There was a problem hiding this comment.
Please update base docker from buster to bullseye like #11288.
jimmyzhai
approved these changes
Jul 8, 2022
Blueve
pushed a commit
that referenced
this pull request
Jul 20, 2022
* Why I did it Followup to #10656. This change adds the remaining configs for the 720DT-48S platform. * How I did it Adds the following: gearbox_config.json and other gearbox-related config files, to enable traffic on external PHY ports (Ethernet0-23) sensors.conf pcie.yaml Also add missing facts in platform.json * How to verify it show interfaces status shows links up on interfaces Ethernet0-23 traffic flows with no errors on interfaces Ethernet0-23 Note: above testing depends on Add gbsyncd container for broncos #11154 and [orchagent]: Enhance initSaiPhyApi sonic-swss#2367, as well as having the appropriate PAI driver. Co-authored-by: Samuel Angebault <staphylo@arista.com>
yxieca
pushed a commit
that referenced
this pull request
Jul 28, 2022
* Add docker-gbsyncd-broncos support * Address review comments * Add socket to gbsyncd * Upgrade gbsyncd-broncos to bullseye
skbarista
pushed a commit
to skbarista/sonic-buildimage
that referenced
this pull request
Aug 17, 2022
* Add docker-gbsyncd-broncos support * Address review comments * Add socket to gbsyncd * Upgrade gbsyncd-broncos to bullseye
andywongarista
added a commit
to andywongarista/sonic-buildimage
that referenced
this pull request
Aug 30, 2022
* Why I did it Followup to sonic-net#10656. This change adds the remaining configs for the 720DT-48S platform. * How I did it Adds the following: gearbox_config.json and other gearbox-related config files, to enable traffic on external PHY ports (Ethernet0-23) sensors.conf pcie.yaml Also add missing facts in platform.json * How to verify it show interfaces status shows links up on interfaces Ethernet0-23 traffic flows with no errors on interfaces Ethernet0-23 Note: above testing depends on Add gbsyncd container for broncos sonic-net#11154 and [orchagent]: Enhance initSaiPhyApi sonic-swss#2367, as well as having the appropriate PAI driver. Co-authored-by: Samuel Angebault <staphylo@arista.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why I did it
This change is to support platforms that use the Broncos (BCM54182) external PHY chips.
How I did it
The change introduces a new docker-gbsyncd-broncos container, similar to the container for Credo added in #8144
The gbsyncd.ini file is added for determining which gbsyncd container a platform needs.
By default, the container will not be included in the built image unless the (LIBSAI_BRONCOS)_URL is defined (pointing to a deb package containing the broncos driver).
How to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)