Add Dell Enterprise Sonic-CLI driver#129
Add Dell Enterprise Sonic-CLI driver#129dougszumski wants to merge 2 commits intostackhpc/2024.1from
Conversation
|
pep8 fail is unrelated |
a855505 to
f810370
Compare
This patch is required urgently for Cambridge. It is unlikely that we can wait for the upstream patch to merge: https://review.opendev.org/c/openstack/networking-generic-switch/+/965373 This driver uses the `sonic-cli` rather than Linux userspace to configure the switch. This is because LLDP advertises switch ports based on the naming from the `sonic-cli`, which is not the same as Linux userspace. Note the existing Dell SONiC driver is using the Linux namespace netmiko driver, which is not suitable. Change-Id: Ia79de43a33c19f7b80bd0ef29badee464cd6fc0c Signed-off-by: Doug Szumski <doug@stackhpc.com>
f810370 to
249fd80
Compare
|
Is there a reason it doesn't support trunk? |
Good questions. They don't want to configure trunk ports, they would just turn off that feature. I could add it if we have these switches somewhere else. I was hoping to backport the upstream patch if I can get it merged, but this could also go to 2025.1 first if you'd prefer. |
Ok then - let's try to update the upstream patch with trunk (since trunking is normally supported since 2025.1) - and merge this one in 2024.1. If you need any help in adding the trunk commands or gaining traction from the Ironic community - please let me know :-) |
|
After more investigation, I'm going to abandon this. It turns out that you can convert Linux userspace to use a different naming convention with: It appears than sonic-cli is not that widely used, so Linux userspace is preferred. Dell also: |
This patch is required urgently for Cambridge. It is unlikely
that we can wait for the upstream patch to merge:
https://review.opendev.org/c/openstack/networking-generic-switch/+/965373
This driver uses the
sonic-clirather than Linux userspace toconfigure the switch. This is because LLDP advertises switch
ports based on the naming from the
sonic-cli, which is not thesame as Linux userspace.
Note the existing Dell SONiC driver is using the Linux namespace
netmiko driver, which is not suitable.