)]}'
{"id":"openstack%2Fneutron~986652","triplet_id":"openstack%2Fneutron~master~I70e62eecae6928c93f789a36e6b1258529b802da","project":"openstack/neutron","branch":"master","topic":"combine_lrp_commands","hashtags":[],"change_id":"I70e62eecae6928c93f789a36e6b1258529b802da","subject":"WIP \u003d\u003d DNM \u003d\u003d Combine AddLRouterPort with SetLRouterPortInLSwitchPort commands","status":"NEW","created":"2026-04-29 13:39:44.000000000","updated":"2026-04-29 17:08:14.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":false,"submittable":false,"total_comment_count":0,"unresolved_comment_count":0,"work_in_progress":true,"has_review_started":true,"meta_rev_id":"be630014eaf02efab323f7ac36a47c703b13de81","_number":986652,"virtual_id_number":986652,"owner":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2026-04-29 15:46:25.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},"Review-Priority":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"CC":[{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"}]},"pending_reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"CC":[{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"}]},"reviewer_updates":[{"updated":"2026-04-29 14:42:46.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":"CC"},{"updated":"2026-04-29 15:46:25.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":"2026-04-29 17:08:14.000000000","updated_by":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"reviewer":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"state":"CC"}],"messages":[{"id":"e003d93da7f3b0d3179ae63a5a97ded343ea2c19","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2026-04-29 13:39:44.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"828bd82e3cb1e6617653497f2d26830b72de81ef","tag":"autogenerated:gerrit:setWorkInProgress","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2026-04-29 13:40:47.000000000","message":"Set Work In Progress","accounts_in_message":[],"_revision_number":1},{"id":"3292e6eeb5ca96a19325818640645f9ffb670caa","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-04-29 14:42:46.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/a44e9ddfa3e04d409e62b133bda04bb2\n\n- openstack-tox-py310-arm64 https://zuul.opendev.org/t/openstack/build/13b276659e174b18b684e2e99b528534 : SUCCESS in 36m 56s (non-voting)\n- openstack-tox-py313-arm64 https://zuul.opendev.org/t/openstack/build/b206b3a55ee64865840fec5ad0e1948d : SUCCESS in 24m 54s (non-voting)\n- openstack-tox-py314-arm64 https://zuul.opendev.org/t/openstack/build/765849eddf4049b48a7cd2502ca5b773 : SUCCESS in 32m 49s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"4362ac86faf6e82514b87de58918ab12039bf812","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-04-29 15:46:25.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/e34fc55e741d4b3f8e9ec55bd030f17a\n\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/49b52218a813404ca176aa3428207128 : SUCCESS in 1h 34m 07s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/b63a555dfec8488b92baa98d93c9e4db : SUCCESS in 1h 11m 07s\n- neutron-tempest-plugin-ovn https://zuul.opendev.org/t/openstack/build/e85553ce342d4e358ce8bece888d75be : SUCCESS in 2h 05m 34s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/7062d4ab7f2f400685b43cf2ae36f163 : SUCCESS in 1h 08m 03s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4f12c5f375de46eaafad44da0e684ec2 : SUCCESS in 12m 18s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/55ad4bcbb2ad44f4b3411b97eb153c55 : SUCCESS in 42m 02s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/0187cfe5981f475a8e936544fd15ea2f : SUCCESS in 35m 23s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/b90244ba256448a6826d70b227af90fa : SUCCESS in 16m 13s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/8836a46c6e1d4a7b93727b7b176224ea : SUCCESS in 22m 14s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/1f3f3488349046aabfec39e5aac472ff : SUCCESS in 11m 31s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/e4695a314a7a48d48ed2117974a98534 : SUCCESS in 42m 55s\n- neutron-ovn-grenade-multinode https://zuul.opendev.org/t/openstack/build/2b63f3ca978e4c6aa78bb4f3993d3e92 : SUCCESS in 1h 25m 25s\n- neutron-ovn-tempest-ipv6-only-ovs-release https://zuul.opendev.org/t/openstack/build/35ff23252d2b42a097d6230f02866d88 : SUCCESS in 52m 55s\n- neutron-ovn-rally-task https://zuul.opendev.org/t/openstack/build/820d44252b7647e89e59d38777169671 : SUCCESS in 1h 23m 44s\n- neutron-ovn-bgp-tempest-multinode https://zuul.opendev.org/t/openstack/build/4f09903191e745239bcc07bedb20403a : SUCCESS in 1h 25m 14s (non-voting)","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"0f85c78809473530b37c812032357348bb3d9a3d","revisions":{"0f85c78809473530b37c812032357348bb3d9a3d":{"kind":"REWORK","_number":1,"created":"2026-04-29 13:39:44.000000000","uploader":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"ref":"refs/changes/52/986652/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/52/986652/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/52/986652/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/52/986652/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/52/986652/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/52/986652/1"}}},"commit":{"parents":[{"commit":"b85e35d6a6f89d454525962b3023350afa2da3e9","subject":"Merge \"Enable \"filter_validation\" extension by default\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/b85e35d6a6f89d454525962b3023350afa2da3e9"}]}],"author":{"name":"Rodolfo Alonso Hernandez","email":"ralonsoh@redhat.com","date":"2026-04-29 13:38:55.000000000","tz":120},"committer":{"name":"Rodolfo Alonso Hernandez","email":"ralonsoh@redhat.com","date":"2026-04-29 13:39:24.000000000","tz":120},"subject":"WIP \u003d\u003d DNM \u003d\u003d Combine AddLRouterPort with SetLRouterPortInLSwitchPort commands","message":"WIP \u003d\u003d DNM \u003d\u003d Combine AddLRouterPort with SetLRouterPortInLSwitchPort commands\n\nWhen creating a logical router port, the corresponding logical switch\nport is always configured in the same transaction. Previously these\nwere two separate commands, each performing an independent OVSDB table\nlookup. This merges the LSP configuration into AddLRouterPortCommand\nso that a single run_idl() call handles both, eliminating the redundant\nLogical_Switch_Port lookup on every router port creation.\n\nSetLRouterPortInLSwitchPortCommand is preserved for the update path\nwhere only the LSP needs reconfiguring without creating a new LRP.\n\nBUG: to be defined...\nSigned-off-by: Rodolfo Alonso Hernandez \u003cralonsoh@redhat.com\u003e\nChange-Id: I70e62eecae6928c93f789a36e6b1258529b802da\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/0f85c78809473530b37c812032357348bb3d9a3d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/0f85c78809473530b37c812032357348bb3d9a3d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY"},{"label":"Workflow","status":"MAY"},{"label":"Review-Priority","status":"MAY"}]}],"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":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"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":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Review-Priority","description":"Review priority","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"-label:Review-Priority\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":[],"failing_atoms":["label:Review-Priority\u003dMIN"],"atom_explanations":{"label:Review-Priority\u003dMIN":""}}},{"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":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}
