-
Notifications
You must be signed in to change notification settings - Fork 23
GT-1561: Add basic Dell SONiC driver #1982
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
Conversation
Basic support for Dell SONiC switches for Arcus. Build: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/19330333096/job/55291529233 Patch: stackhpc/networking-generic-switch#130
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request introduces basic support for Dell SONiC switches by adding a new release note entry. It also updates the Kolla image tags for the neutron service across different operating system distributions. The changes appear to be straightforward and correctly implemented, aligning with the stated purpose of the pull request. No issues of medium, high, or critical severity were identified in the code changes.
Alex-Welsh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but is there an equivalent patch for 2025.1? I'm a bit concerned we'll forget about it, upgrade, and break things
Thanks Alex - I am trying to attack it from the other side for later releases since there is a Dell Sonic driver upstream. I'll leave the ticket open until I've resolved it one way or another. |
Basic support for Dell SONiC switches for Arcus.
Build: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/19330333096/job/55291529233
Patch: stackhpc/networking-generic-switch#130