-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[Celestica] Add Celestica Questone2/2A platform on master branch #15990
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Rancho Zhang <zhwwan@gmail.com>
…ng used per program Signed-off-by: Rancho Zhang <zhwwan@gmail.com>
I am also interested in support for this switch, hope to see this merged! |
I Hope this commit can be merged |
Hi, I have tried to run this on two of my D3030's without success. One try on master and one on 202211. Has anyone successfully got this to run on Questone 2A? The error I am getting is that syncd refuses to start with an "Invalid unit" error. EDIT: Seems our switch was a Questone 2 and not 2A, with a few extra fixes we got it to work. See kamelnetworks@05a12d9 for our patches. |
Note that this PR is probably no longer needed as #18191 has been merged. |
Probably need a new PR, it does not appear to include recognition of the HP SKUs here: https://github.com/jemifdo/sonic-buildimage/blob/537f376fd0464f3dc3fb3a2dd3faba26766245fd/platform/broadcom/sonic-platform-modules-cel/debian/platform-modules-questone2.init |
Why I did it
Add new Celestica device Questone2/2A of network switch platform with Broadcom ASIC on master branch.
Work item tracking
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Add Celestica device platform of Questone2/2A on master branch.
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)