)]}'
{"id":"openstack%2Fneutron~836321","triplet_id":"openstack%2Fneutron~stable%2Fussuri~I0df376a80140ead7ff1fbf7f5ffef08a999dbe0b","project":"openstack/neutron","branch":"stable/ussuri","topic":"bug/1963899-stable/ussuri","attention_set":{},"removed_from_attention_set":{"11975":{"account":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"last_update":"2022-04-07 17:05:49.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"I0df376a80140ead7ff1fbf7f5ffef08a999dbe0b","subject":"Ensure that re_added ports are DOWN before set back to UP","status":"MERGED","created":"2022-04-04 12:34:22.000000000","updated":"2022-04-07 17:07:31.000000000","submitted":"2022-04-07 17:05:49.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"836321-bug/1963899-stable/ussuri","meta_rev_id":"23e8b6c648953a4c7b5070a79d9fc4be1798083c","_number":836321,"virtual_id_number":836321,"owner":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2022-04-07 17:05:48.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":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"all":[{"value":2,"date":"2022-04-07 14:50:25.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"value":2,"date":"2022-04-05 11:28:39.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"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":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"all":[{"value":1,"date":"2022-04-07 14:50:25.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"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":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"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":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"_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":"2022-04-04 16:43:00.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":"2022-04-05 11:28:39.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"},{"updated":"2022-04-07 14:50:25.000000000","updated_by":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"reviewer":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"state":"REVIEWER"}],"messages":[{"id":"1712e0a23dc2070ed2b200fbcf629a4382a22a48","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2022-04-04 12:34:22.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b6aa7259927f304a11890cd2fd654c73e529f5d9","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-04-04 16:43:00.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- neutron-tempest-plugin-api-ussuri https://zuul.opendev.org/t/openstack/build/ae94e14af8404d168a2adf77a09fe0c4 : SUCCESS in 1h 06m 35s\n- neutron-tempest-plugin-scenario-openvswitch-ussuri https://zuul.opendev.org/t/openstack/build/fa7b5f7c10f44ab1bc655006f7ba31d9 : SUCCESS in 1h 43m 15s\n- neutron-tempest-plugin-scenario-openvswitch-iptables_hybrid-ussuri https://zuul.opendev.org/t/openstack/build/9a600caeb0fd49f3ad3f737459e19c29 : SUCCESS in 1h 38m 53s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/90c16ed78fda4118b852ab8ca0c3f80d : SUCCESS in 33m 27s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/62cac763ab4a412ea1dbf8edd03f2aa1 : SUCCESS in 13m 47s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a821985050624fef97755bb4a3f90a6e : SUCCESS in 26m 05s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/f0bf9bb623204c47846e7f4cf15c0f68 : SUCCESS in 28m 34s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/f68383fed0f84011bfe91d4c87f4009d : SUCCESS in 25m 22s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/35cf2793c6cc443a9e3183c1158e9a18 : SUCCESS in 9m 24s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/67913c881bba459bb7cf4542dae04ca0 : SUCCESS in 53m 48s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/ce6b5f2dde3c417298de87ff6a9441d0 : SUCCESS in 1h 20m 59s\n- neutron-fullstack-with-uwsgi https://zuul.opendev.org/t/openstack/build/1448de04cad14c8ba365442e13b4ab4c : SUCCESS in 1h 20m 45s\n- neutron-rally-task https://zuul.opendev.org/t/openstack/build/6743bdae8ebc4cf7a1917f70e4315f61 : SUCCESS in 1h 27m 37s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/778f6fe8f21b44c3a0778726a3e1e954 : SUCCESS in 1h 13m 13s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/47bb01ad862d47aa8ba95d5da4a4e074 : SUCCESS in 1h 14m 21s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/a14948a4236d431ea6ce313fd9ac10c8 : SUCCESS in 1h 11m 11s\n- neutron-tempest-with-uwsgi https://zuul.opendev.org/t/openstack/build/1b6f535029bf48508f3eb47beca26182 : SUCCESS in 1h 08m 00s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/19279a36d58e443ea193c29f035ebb4d : SUCCESS in 1h 08m 05s\n- tempest-multinode-full-py3 https://zuul.opendev.org/t/openstack/build/1ddfd2d6b4d44250879f73cf679a8fae : SUCCESS in 1h 39m 27s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full https://zuul.opendev.org/t/openstack/build/076cac8038d540bab0c3bd74a47a0b55 : SUCCESS in 1h 21m 58s (non-voting)\n- neutron-tempest-iptables_hybrid https://zuul.opendev.org/t/openstack/build/92dbeeb13c2d4fa080ac720d3b10eb4b : SUCCESS in 1h 21m 21s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/0e7d64a2523149f49d158d01725eb779 : FAILURE in 1h 07m 25s (non-voting)\n- neutron-tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/9a00e1ce3f6a40109f3ef9ed4ff4ae73 : SUCCESS in 1h 36m 21s\n- neutron-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/f0cce22358134c19bb0672e4f4c28846 : SUCCESS in 1h 06m 32s\n- neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/739d577a9bf941ea9492fbac42f72728 : SUCCESS in 1h 08m 40s\n- neutron-ovn-tempest-ovs-release-ipv6-only https://zuul.opendev.org/t/openstack/build/72f7bd9bb69d412f8d42dfa36b56022c : SUCCESS in 42m 04s (non-voting)\n- openstacksdk-functional-devstack-networking https://zuul.opendev.org/t/openstack/build/2f02979cc5734401bbc6b1d02444a59c : SUCCESS in 1h 23m 40s (non-voting)\n- neutron-functional-with-uwsgi https://zuul.opendev.org/t/openstack/build/21ab4b2e97ca4f9e96c7dbf41da16819 : SUCCESS in 56m 31s (non-voting)\n- tripleo-ci-centos-8-content-provider https://zuul.opendev.org/t/openstack/build/34a0446ad9f84260963eeeee599689b1 : SUCCESS in 4h 06m 41s (non-voting)\n- neutron-centos-8-tripleo-standalone https://zuul.opendev.org/t/openstack/build/6dc9ca77d9444149be5d7ae88bd59f63 : SUCCESS in 1h 26m 35s (non-voting)\n- neutron-ovn-rally-task https://zuul.opendev.org/t/openstack/build/3cc09b591f854eb2ac594054ecb83177 : SUCCESS in 1h 41m 54s (non-voting)\n- neutron-ovn-tripleo-ci-centos-8-containers-multinode https://zuul.opendev.org/t/openstack/build/d8b2205a9cc041feb90080a5308770ab : SUCCESS in 2h 11m 16s (non-voting)\n- neutron-ovn-tempest-slow https://zuul.opendev.org/t/openstack/build/542dc513ea834690a113e5e03735a087 : FAILURE in 1h 32m 29s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"dc161a9fc03ed94e0bdd359c3aa18ab75ae1c9bd","author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"date":"2022-04-05 11:28:39.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"1945968e921e2de91398d8402f6f1ad50eea5782","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2022-04-07 14:50:25.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"c8548ffd3b14c9ad76961ab1301b60bbbaaae562","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-04-07 14:51:25.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"4f13fc2a86a73a28285705f143174a6e3920e8c0","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-04-07 17:05:48.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- neutron-tempest-plugin-api-ussuri https://zuul.opendev.org/t/openstack/build/f7e1acf0d6b64337a5ed8d6eec1b4bf3 : SUCCESS in 41m 46s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ec2272e9ab3a420f96e734ed60166092 : SUCCESS in 12m 05s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/de96e63baa584fc6a73954884fa68637 : SUCCESS in 21m 22s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/8fe30529942e41d29173160a18aa24ad : SUCCESS in 22m 28s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ae5f08f32f1443a0a8af902ca690935d : SUCCESS in 9m 36s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/87fc6b0518884dc6abdf7843fd8b71fb : SUCCESS in 48m 58s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/76756c29aebf4efd8d81e2d62a72c187 : SUCCESS in 1h 14m 44s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/6650bb2197274a6298dac8282300ab41 : SUCCESS in 1h 09m 04s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/93b4a7af696c47f48071b7dd227e061a : SUCCESS in 1h 09m 06s\n- neutron-tempest-iptables_hybrid https://zuul.opendev.org/t/openstack/build/cb5af1f8f49149638f912e43d904b233 : SUCCESS in 1h 10m 24s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/a019adb89d9f42e38ad8889654df4138 : SUCCESS in 1h 13m 18s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/0ba1acd05a1f4f488ddeb0e086abfcc0 : SUCCESS in 1h 11m 18s\n- neutron-tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/aaeaa8b531384987ad85988076ca3cb6 : SUCCESS in 1h 30m 51s\n- neutron-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9d193cfd7b144955accc8b71bfbbc246 : SUCCESS in 1h 04m 08s\n- neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/9a8ef48b2bb642788704be5929f6c5c9 : SUCCESS in 1h 08m 53s","accounts_in_message":[],"_revision_number":1},{"id":"2b529bb786a2d42fa1a92bc7e6d9203540d2c0b3","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-04-07 17:05:49.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":1},{"id":"23e8b6c648953a4c7b5070a79d9fc4be1798083c","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-04-07 17:07:31.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d1b1a86bc4ae468181a39843a3b93809 : SUCCESS in 1m 20s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"04453cd161b172f1cdbacd0a034f5e6d21fd6e34","revisions":{"04453cd161b172f1cdbacd0a034f5e6d21fd6e34":{"kind":"REWORK","_number":1,"created":"2022-04-04 12:34:22.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/21/836321/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/21/836321/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/21/836321/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/21/836321/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/21/836321/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/21/836321/1"}}},"commit":{"parents":[{"commit":"9743fbd75e67283fb54ab02027176fe2f6a5003b","subject":"Merge \"Log request IDs for matched Nova external events\" into stable/ussuri","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/9743fbd75e67283fb54ab02027176fe2f6a5003b"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2022-03-23 11:25:26.000000000","tz":60},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2022-04-04 12:33:59.000000000","tz":120},"subject":"Ensure that re_added ports are DOWN before set back to UP","message":"Ensure that re_added ports are DOWN before set back to UP\n\nDuring e.g. rebuild of the server by Nova, ports plugged to such server\nare quickly removed and added again into br-int. In such case, ports are\nin the \"re_added\" ports set in the neutron-ovs-agent.\nBut it seems that in some cases it may happen that such port isn\u0027t\nswitched to be DOWN first and then, when neutron-ovs-agent treats port\nas added/updated and reports to the server that port is UP, there is no\nnotification to nova-compute send (because port\u0027s status was UP and new\nstatus is still UP in the Neutron DB).\nAs Nova waits for the notification from Neutron in such case server\ncould ends up in the ERROR state.\n\nTo avoid such issue, all ports which are treated as \"re_added\" by the\nneutron-ovs-agent are now first switched to be DOWN on the server side.\nThat way, when those ports are treated as added/updated in the same\nrpc_loop iteration, switching their status to UP will for sure trigger\nnotification to nova.\n\nConflicts:\n    neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_ovs_neutron_agent.py\n\nCloses-Bug: #1963899\nChange-Id: I0df376a80140ead7ff1fbf7f5ffef08a999dbe0b\n(cherry picked from commit e7edcec2608fe1cb30d7458a882eb81e058eb76e)\n(cherry picked from commit 0842d28232949acd456aefbcd5ca714638103a50)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/04453cd161b172f1cdbacd0a034f5e6d21fd6e34"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/04453cd161b172f1cdbacd0a034f5e6d21fd6e34"}]},"branch":"refs/heads/stable/ussuri"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"}},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}