)]}'
{"id":"openstack%2Fneutron~952583","triplet_id":"openstack%2Fneutron~stable%2F2025.1~I00fa8332fdebc958ddb8f28c638670c75a70e0c5","project":"openstack/neutron","branch":"stable/2025.1","attention_set":{},"removed_from_attention_set":{"32586":{"account":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"last_update":"2025-06-23 14:11:05.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"I00fa8332fdebc958ddb8f28c638670c75a70e0c5","subject":"Consider logging options when using OVNdbsync","status":"MERGED","created":"2025-06-13 14:54:51.000000000","updated":"2025-06-23 14:13:17.000000000","submitted":"2025-06-23 14:11:05.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"952583","meta_rev_id":"fd2dde6d6e0049094860512dab3531522d98b619","_number":952583,"virtual_id_number":952583,"owner":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2025-06-23 14:11:05.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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"all":[{"value":2,"date":"2025-06-23 11:22:34.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":2,"date":"2025-06-13 15:49:01.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_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":{"approved":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"all":[{"value":1,"date":"2025-06-23 11:22:34.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_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":[{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_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":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-06-13 15:49:01.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":"REVIEWER"},{"updated":"2025-06-13 16:12:40.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":"2025-06-13 17:36:18.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-06-23 11:22:34.000000000","updated_by":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"reviewer":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"state":"REVIEWER"}],"messages":[{"id":"aa5360b36b8c6cc53e2f42e6aec970976f65b012","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"date":"2025-06-13 14:54:51.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"f906d36ed8e21f723b1294862410f3df35b4cd08","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"date":"2025-06-13 14:57:05.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"701bc97b1dfaa201606ffeba03ed354eac92fcc6","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2025-06-13 15:49:01.000000000","message":"Patch Set 2: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"2eb05ec54e133747f09bc9e1f8085dc36d726239","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-13 16:12:40.000000000","message":"Patch Set 2:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/a0095fe6754848e3b1882f94b987c458\n\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/a1de76ccf6bc481abce518c3b3d0cc07 : SUCCESS in 39m 24s (non-voting)\n- openstack-tox-py312-arm64 https://zuul.opendev.org/t/openstack/build/7ede378167e8400090a6ffb893b41f49 : SUCCESS in 37m 47s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"bb97bd80c340dc1e7bb9757278ed7c74264f1729","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-13 17:36:18.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/79be22b6b4ce4288b4c3a87d57b09540\n\n- neutron-tempest-plugin-ovn-2025-1 https://zuul.opendev.org/t/openstack/build/b1a425f2f4294d418e60a0b36ba84879 : SUCCESS in 2h 36m 57s\n- neutron-tempest-plugin-designate-scenario-2025-1 https://zuul.opendev.org/t/openstack/build/eff77e8e1fdb4e51be98b775a22c74d7 : SUCCESS in 49m 17s\n- neutron-tempest-plugin-ovn-ubuntu-jammy-2025-1 https://zuul.opendev.org/t/openstack/build/bda22a2b37ef41639c78194c7492874f : SUCCESS in 1h 02m 54s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/f98ea7a984364b0c8c4a623308d520e1 : SUCCESS in 32m 30s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/31b806ce37bf49adaa0805196fcad505 : SUCCESS in 6m 41s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/277d20f3f1544c3d908c4f4299969a46 : SUCCESS in 22m 29s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/8ed094284aaa4087a3e9b8dadf68a0b5 : SUCCESS in 18m 31s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d27002912258476a8dd25d47328a68f6 : SUCCESS in 12m 50s\n- neutron-ovn-grenade-multinode-skip-level https://zuul.opendev.org/t/openstack/build/ee8848016bd94f23bad50c640e6b035a : SUCCESS in 1h 00m 57s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/73051a04b06143cba9978a87d7d7ea44 : SUCCESS in 1h 58m 06s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/e97a235f688d43a4ac3f052f225d22e4 : SUCCESS in 1h 49m 21s\n- neutron-ovn-grenade-multinode https://zuul.opendev.org/t/openstack/build/8ffb818dfa414dbaa85ca0259e2677f6 : SUCCESS in 1h 06m 56s\n- neutron-ovn-tempest-ipv6-only-ovs-release https://zuul.opendev.org/t/openstack/build/c470060ec0884b9e849d149f1a33092a : SUCCESS in 45m 28s\n- neutron-ovn-rally-task https://zuul.opendev.org/t/openstack/build/2d0678ba3f3f49e7a457544fbfebc60a : SUCCESS in 1h 14m 07s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/a7252d5fbaed4c8194128fd52ebf4253 : SUCCESS in 1h 16m 44s","accounts_in_message":[],"_revision_number":2},{"id":"c74a1fa8a1a2ebcd4049cf274026ceb1bde5e451","author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"date":"2025-06-23 11:22:34.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"1d484b4de2b88c29451ae5abfecb5dd1304632ab","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-23 11:22:55.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"48e8c897965c497ce9bd91c8ecac81613bb58899","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-23 14:11:05.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/6988692dbb4447799df37a5ec1e21905\n\n- neutron-tempest-plugin-ovn-2025-1 https://zuul.opendev.org/t/openstack/build/67d2660efb44401991a26c91904b9a88 : SUCCESS in 1h 53m 52s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/3b2fa0392e4b47b1ae00bfccfe0891b6 : SUCCESS in 11m 25s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/25ba5d9024484b1bbc57e8e0f2b34bd7 : SUCCESS in 22m 26s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/300fc032d74a45edb9ddf020f8434b7b : SUCCESS in 21m 03s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a672fd19cfd6409a943af6923bc87f2c : SUCCESS in 12m 00s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/57770fc0155f4de7838c066d02502568 : SUCCESS in 1h 35m 36s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/b9d3f5de8d954d66af805b20d281eb39 : SUCCESS in 1h 39m 43s\n- neutron-ovn-grenade-multinode https://zuul.opendev.org/t/openstack/build/e2df934bc58b429595a09aa957782b6e : SUCCESS in 1h 05m 30s\n- neutron-ovn-tempest-ipv6-only-ovs-release https://zuul.opendev.org/t/openstack/build/d15cd52be71741b4b755af6525e2ac5d : SUCCESS in 1h 02m 26s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/e75b24ada85c4f7eae2d9eddb8543995 : SUCCESS in 1h 15m 17s","accounts_in_message":[],"_revision_number":2},{"id":"2554f90b413d38fc2b7a3a7ca110764df97f3cec","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-23 14:11:05.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":2},{"id":"fd2dde6d6e0049094860512dab3531522d98b619","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-23 14:13:17.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/327fe8c12a744e26962f877ffa714d6b\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/4fc992490ac14b5bb7c11fd8827396f8 : SUCCESS in 1m 57s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"fa93909b0a81dc9ebcdc9f3d973cf9922b1df1fe","revisions":{"5281028072a90baf0b084daffaebcb6ee79e34fb":{"kind":"REWORK","_number":1,"created":"2025-06-13 14:54:51.000000000","uploader":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"ref":"refs/changes/83/952583/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/83/952583/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/83/952583/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/83/952583/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/83/952583/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/83/952583/1"}}},"commit":{"parents":[{"commit":"ecdb6ed1e508cba177c199af960a5369ea36d400","subject":"Merge \"Fix nova notification on port unplug\" into stable/2025.1","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/ecdb6ed1e508cba177c199af960a5369ea36d400"}]}],"author":{"name":"Elvira","email":"egarciar@redhat.com","date":"2025-05-27 19:11:46.000000000","tz":0},"committer":{"name":"Elvira Garcia","email":"egarciar@redhat.com","date":"2025-06-13 14:54:26.000000000","tz":120},"subject":"Consider logging options when using OVNdbsync","message":"Consider logging options when using OVNdbsync\n\nPreviously, OVN db sync would erase an ACL if any unexpected property\nappeared on it and not recreate it again. This happened because of the\norder of deletion and creation of the ACLS: the new ACL was first\ncreated and then deleted just the moment after that. This meant that\neven crucial ACLs like the ones bounded to the pg_drop port group, which\nare used to reject all the traffic by default on ML2/OVN environments,\nwould dissapear. The order of the ACL deletion and creation has been\ninverted to avoid this.\n\nFurthermore, security group logging was not supported on the\novn_db_sync script, which would also cause the logging parameters to\ndissapear. Now, the logging options are considered when doing a sync.\n\nCloses-Bug: #2107925\nChange-Id: I00fa8332fdebc958ddb8f28c638670c75a70e0c5\nSigned-off-by: Elvira Garcia \u003cegarciar@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/5281028072a90baf0b084daffaebcb6ee79e34fb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/5281028072a90baf0b084daffaebcb6ee79e34fb"}]},"branch":"refs/heads/stable/2025.1"},"fa93909b0a81dc9ebcdc9f3d973cf9922b1df1fe":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2025-06-13 14:57:05.000000000","uploader":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"ref":"refs/changes/83/952583/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/83/952583/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/83/952583/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/83/952583/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/83/952583/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/83/952583/2"}}},"commit":{"parents":[{"commit":"ecdb6ed1e508cba177c199af960a5369ea36d400","subject":"Merge \"Fix nova notification on port unplug\" into stable/2025.1","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/ecdb6ed1e508cba177c199af960a5369ea36d400"}]}],"author":{"name":"Elvira","email":"egarciar@redhat.com","date":"2025-05-27 19:11:46.000000000","tz":0},"committer":{"name":"Elvira Garcia","email":"egarciar@redhat.com","date":"2025-06-13 14:55:33.000000000","tz":120},"subject":"Consider logging options when using OVNdbsync","message":"Consider logging options when using OVNdbsync\n\nPreviously, OVN db sync would erase an ACL if any unexpected property\nappeared on it and not recreate it again. This happened because of the\norder of deletion and creation of the ACLS: the new ACL was first\ncreated and then deleted just the moment after that. This meant that\neven crucial ACLs like the ones bounded to the pg_drop port group, which\nare used to reject all the traffic by default on ML2/OVN environments,\nwould dissapear. The order of the ACL deletion and creation has been\ninverted to avoid this.\n\nFurthermore, security group logging was not supported on the\novn_db_sync script, which would also cause the logging parameters to\ndissapear. Now, the logging options are considered when doing a sync.\n\nConflicts:\n  neutron/common/ovn/acl.py\n\nCloses-Bug: #2107925\nChange-Id: I00fa8332fdebc958ddb8f28c638670c75a70e0c5\nSigned-off-by: Elvira Garcia \u003cegarciar@redhat.com\u003e\n(cherry picked from commit 1cf5b6de7c964b17913d4ade9a8c7485aff8a2db)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/fa93909b0a81dc9ebcdc9f3d973cf9922b1df1fe"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/fa93909b0a81dc9ebcdc9f3d973cf9922b1df1fe"}]},"branch":"refs/heads/stable/2025.1"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"CLOSED","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"}},{"label":"Workflow","status":"MAY","applied_by":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"}},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dMAX"],"failing_atoms":["label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dMAX"],"failing_atoms":["label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"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":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dMAX"],"failing_atoms":["label:Workflow\u003dMIN"],"atom_explanations":{}}}]}
