)]}'
{"id":"openstack%2Fnetworking-generic-switch~965373","triplet_id":"openstack%2Fnetworking-generic-switch~master~Ia79de43a33c19f7b80bd0ef29badee464cd6fc0c","project":"openstack/networking-generic-switch","branch":"master","topic":"feature/upstream-dell-sonic-cli","attention_set":{},"removed_from_attention_set":{"17669":{"account":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"last_update":"2025-11-13 10:27:24.000000000","reason":"\u003cGERRIT_ACCOUNT_17669\u003e replied on the change","reason_account":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"}},"4571":{"account":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"last_update":"2025-11-13 10:28:11.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"Ia79de43a33c19f7b80bd0ef29badee464cd6fc0c","subject":"Add Dell Enterprise SONiC CLI driver","status":"ABANDONED","created":"2025-10-30 12:52:34.000000000","updated":"2025-11-13 10:28:11.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f8d3d34cf7f897bb1db3a514b33f65b326334f37","_number":965373,"virtual_id_number":965373,"owner":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2025-10-30 20:09:46.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Do Not Backport"," 0":"Backport Review Needed","+1":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"}],"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-10-30 15:38:56.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2025-10-30 23:28:38.000000000","updated_by":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"reviewer":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"state":"CC"}],"messages":[{"id":"056c0b69b38ca53c45872bf7a63622cd733408a4","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"date":"2025-10-30 12:52:34.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"de924cebe73c7771f32cfe1c2454469a98890456","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"date":"2025-10-30 13:07:40.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"347ccdfad2bbc9b4c133d1dc3eb7d0c3d57ad84f","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-10-30 15:38:56.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/7aed1cc1ed4046a7bba6bce1f973fde0\n\n- test-release-openstack https://zuul.opendev.org/t/openstack/build/cc3325fd865644fca0982dc23cfc0c13 : SUCCESS in 2m 22s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e7b23fea19fd426fad832218901faaef : SUCCESS in 6m 46s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/01ebfba865574c70a23444fce774069d : SUCCESS in 4m 06s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/7dd8251a9ae44ef8a49301fb8b4b5160 : SUCCESS in 7m 15s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/00e63830078d4104a31ff0f5bbae3f71 : SUCCESS in 6m 20s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/e23d5c9a21c747e59007b63ca6cd64fa : SUCCESS in 3m 27s\n- networking-generic-switch-tempest-dlm https://zuul.opendev.org/t/openstack/build/18e3055d1aac4f16b0cbe3765841ecc7 : SUCCESS in 14m 42s\n- ironic-tempest-ipa-wholedisk-direct-multinode https://zuul.opendev.org/t/openstack/build/7688c11a14b4475881cb34ec97375817 : SUCCESS in 2h 20m 32s\n- networking-generic-switch-tox-codespell https://zuul.opendev.org/t/openstack/build/2bf0c2f3cb3b4ef4a0f8e4f78f4ca253 : FAILURE in 1m 53s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"38db04fbfb38262aa006d455afd3d4e50623078b","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"date":"2025-10-30 17:35:58.000000000","message":"Uploaded patch set 3.\n\nOutdated Votes:\n* Verified+1\n","accounts_in_message":[],"_revision_number":3},{"id":"07deda7b6cf7c818dc8401585ce0f2273824003e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-10-30 20:09:46.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/e6de08a6c94d48ce94485fbaa3b53954\n\n- test-release-openstack https://zuul.opendev.org/t/openstack/build/a1709b79c28b4fe381905a813481f1e1 : SUCCESS in 2m 40s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5f7e1360a8cb40e292d9c1528afceef4 : SUCCESS in 4m 26s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/5835833d32fc48dfbc1701b7c59f9cd0 : SUCCESS in 5m 10s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/c7ce1ce63a874fea94bda24e0c3c6186 : SUCCESS in 7m 55s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/64ce7aea8d664976ae87673fc0766a11 : SUCCESS in 11m 21s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/551f1ef8dbd942d4ba154e5c5ab73f8c : SUCCESS in 5m 31s\n- networking-generic-switch-tempest-dlm https://zuul.opendev.org/t/openstack/build/68216b7c22944459b591c5cee400f56d : SUCCESS in 13m 51s\n- ironic-tempest-ipa-wholedisk-direct-multinode https://zuul.opendev.org/t/openstack/build/5975e731c7c64ca19a73265433837a96 : FAILURE in 2h 28m 41s\n- networking-generic-switch-tox-codespell https://zuul.opendev.org/t/openstack/build/ea00ca688285456381058c05df80036c : FAILURE in 3m 32s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"667566ef3fd099653d3386d6adcdf86d5b23ec3f","author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"date":"2025-10-30 23:28:38.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"7363e5d63d73037acbaa2d8f7a28c0bbc5b95cd0","author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"date":"2025-11-13 10:27:24.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"f8d3d34cf7f897bb1db3a514b33f65b326334f37","tag":"autogenerated:gerrit:abandon","author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"date":"2025-11-13 10:28:11.000000000","message":"Abandoned\n\nAbandoned because a way forward using the native Linux shell was identified.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"8e425504dc7e0a7ae9eccb2bcde666b86cfbcf55","revisions":{"af6d9c8fcb5458b66143a8a9cd0b9232fce29629":{"kind":"REWORK","_number":1,"created":"2025-10-30 12:52:34.000000000","uploader":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"ref":"refs/changes/73/965373/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/networking-generic-switch","ref":"refs/changes/73/965373/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/1"}}},"commit":{"parents":[{"commit":"8c0f1ec30bd0d765cd539de068a0c02cd9c8d699","subject":"Merge \"etcd: Discover api version automatically\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/8c0f1ec30bd0d765cd539de068a0c02cd9c8d699"}]}],"author":{"name":"Doug Szumski","email":"doug@stackhpc.com","date":"2025-10-29 16:19:20.000000000","tz":0},"committer":{"name":"Doug Szumski","email":"doug@stackhpc.com","date":"2025-10-30 12:50:47.000000000","tz":0},"subject":"Add Dell Enterprise Sonic-CLI driver","message":"Add Dell Enterprise Sonic-CLI driver\n\nThis driver uses the sonic-cli rather than Linux userspace to\nconfigure the switch. This is because LLDP advertises switch\nports based on the naming from the sonic-cli, which is not the\nsame as Linux userspace.\n\nThere remains an exercise to push this upstream. Note the existing\nDell Sonic driver on main is using the Linux namespace netmiko\ndriver, which is not suitable [1].\n\n[1] https://github.com/openstack/networking-generic-switch/tree/master/networking_generic_switch/devices/netmiko_devices\n\nChange-Id: Ia79de43a33c19f7b80bd0ef29badee464cd6fc0c\nSigned-off-by: Doug Szumski \u003cdoug@stackhpc.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/af6d9c8fcb5458b66143a8a9cd0b9232fce29629"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/af6d9c8fcb5458b66143a8a9cd0b9232fce29629"}]},"branch":"refs/heads/master"},"d1eed04a1bbeefab27d94f99e6a2b90d5abfa9a7":{"kind":"REWORK","_number":2,"created":"2025-10-30 13:07:40.000000000","uploader":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"ref":"refs/changes/73/965373/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/networking-generic-switch","ref":"refs/changes/73/965373/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/2"}}},"commit":{"parents":[{"commit":"8c0f1ec30bd0d765cd539de068a0c02cd9c8d699","subject":"Merge \"etcd: Discover api version automatically\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/8c0f1ec30bd0d765cd539de068a0c02cd9c8d699"}]}],"author":{"name":"Doug Szumski","email":"doug@stackhpc.com","date":"2025-10-29 16:19:20.000000000","tz":0},"committer":{"name":"Doug Szumski","email":"doug@stackhpc.com","date":"2025-10-30 13:05:55.000000000","tz":0},"subject":"Add Dell Enterprise SONiC CLI driver","message":"Add Dell Enterprise SONiC CLI driver\n\nThis driver uses the `sonic-cli` rather than Linux userspace to\nconfigure the switch. This is because LLDP advertises switch\nports based on the naming from the `sonic-cli`, which is not the\nsame as Linux userspace.\n\nNote the existing Dell SONiC driver is using the Linux namespace\nnetmiko driver, which is not suitable.\n\nChange-Id: Ia79de43a33c19f7b80bd0ef29badee464cd6fc0c\nSigned-off-by: Doug Szumski \u003cdoug@stackhpc.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/d1eed04a1bbeefab27d94f99e6a2b90d5abfa9a7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/d1eed04a1bbeefab27d94f99e6a2b90d5abfa9a7"}]},"branch":"refs/heads/master"},"8e425504dc7e0a7ae9eccb2bcde666b86cfbcf55":{"kind":"REWORK","_number":3,"created":"2025-10-30 17:35:58.000000000","uploader":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"ref":"refs/changes/73/965373/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/networking-generic-switch","ref":"refs/changes/73/965373/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/networking-generic-switch refs/changes/73/965373/3"}}},"commit":{"parents":[{"commit":"8c0f1ec30bd0d765cd539de068a0c02cd9c8d699","subject":"Merge \"etcd: Discover api version automatically\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/8c0f1ec30bd0d765cd539de068a0c02cd9c8d699"}]}],"author":{"name":"Doug Szumski","email":"doug@stackhpc.com","date":"2025-10-29 16:19:20.000000000","tz":0},"committer":{"name":"Doug Szumski","email":"doug@stackhpc.com","date":"2025-10-30 17:33:55.000000000","tz":0},"subject":"Add Dell Enterprise SONiC CLI driver","message":"Add Dell Enterprise SONiC CLI driver\n\nThis driver uses the `sonic-cli` rather than Linux userspace to\nconfigure the switch. This is because LLDP advertises switch\nports based on the naming from the `sonic-cli`, which is not the\nsame as Linux userspace.\n\nNote the existing Dell SONiC driver is using the Linux namespace\nnetmiko driver, which is not suitable.\n\nChange-Id: Ia79de43a33c19f7b80bd0ef29badee464cd6fc0c\nSigned-off-by: Doug Szumski \u003cdoug@stackhpc.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/8e425504dc7e0a7ae9eccb2bcde666b86cfbcf55"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/8e425504dc7e0a7ae9eccb2bcde666b86cfbcf55"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Backport-Candidate","description":"Backport candidate status","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}