)]}'
{"id":"openstack%2Fneutron~737700","triplet_id":"openstack%2Fneutron~stable%2Ftrain~I515c310f221e7d9ae3be59a26260538d1bc591c2","project":"openstack/neutron","branch":"stable/train","topic":"bug/1883730-stable/ussuri-stable/train","hashtags":[],"change_id":"I515c310f221e7d9ae3be59a26260538d1bc591c2","subject":"Make _ensure_default_security_group method atomic","status":"MERGED","created":"2020-06-24 07:51:45.000000000","updated":"2020-06-26 17:44:10.000000000","submitted":"2020-06-26 17:40:26.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":"737700-1593193226854-0147cf3e","meta_rev_id":"4176c39cf1b575ce5299d0c9369027ec737b46c2","_number":737700,"virtual_id_number":737700,"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":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-06-26 17:40:26.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2020-06-24 09:47:30.000000000","_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"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":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"all":[{"value":2,"date":"2020-06-24 22:30:50.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"value":2,"date":"2020-06-26 14:13:12.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"]},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"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":0,"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"value":1,"date":"2020-06-26 14:13:12.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"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"]},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"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":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-06-24 09:47:30.000000000","updated_by":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"reviewer":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"state":"REVIEWER"},{"updated":"2020-06-24 22:30:50.000000000","updated_by":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"reviewer":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"state":"REVIEWER"},{"updated":"2020-06-26 14:13:12.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":"2020-06-26 17:40:26.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"}],"messages":[{"id":"fff1e17ec01134aaecc148a49464cb8d76ff3f94","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2020-06-24 07:51:45.000000000","message":"Patch Set 1: Cherry Picked from branch stable/ussuri.","accounts_in_message":[],"_revision_number":1},{"id":"bcbe0a443acd879ee3c855739a5bbab602a5efd3","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2020-06-24 09:47:30.000000000","message":"Patch Set 1:\n\nBuild succeeded.\n\n- neutron http://cloudbase-ci.com/neutron/737700/1 : SUCCESS in 1h 55m 02s","accounts_in_message":[],"_revision_number":1},{"id":"a77a608d036b6b4550e519022733c1758192685f","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-24 11:37:53.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- neutron-tempest-plugin-api-train https://zuul.opendev.org/t/openstack/build/b67bf0248e4d450693d857da8306c934 : SUCCESS in 1h 10m 27s\n- neutron-tempest-plugin-designate-scenario-train https://zuul.opendev.org/t/openstack/build/d973d53731514be58ecbff21d8266bb2 : SUCCESS in 47m 00s\n- neutron-tempest-plugin-scenario-linuxbridge-train https://zuul.opendev.org/t/openstack/build/2f6d81de67014e859c657afedaaf2c4a : SUCCESS in 1h 24m 28s\n- neutron-tempest-plugin-scenario-openvswitch-train https://zuul.opendev.org/t/openstack/build/6b069ae58a324a94aae3456533e4e177 : SUCCESS in 1h 18m 18s\n- neutron-tempest-plugin-scenario-openvswitch-iptables_hybrid-train https://zuul.opendev.org/t/openstack/build/8e4f3aaed48a4cbc8212faa3d40b135f : SUCCESS in 1h 06m 03s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/947c2deda2a34f71a056e6e2a392975e : SUCCESS in 27m 43s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/39d5b5119f9c467c8f0f3ae1797e507a : SUCCESS in 20m 35s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/70c5ea7bd2bc416ca54139bd6d14cfc9 : SUCCESS in 9m 20s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/f4a82ffd6b7840afaed26514be543a6b : SUCCESS in 23m 48s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/410f9e852dc6463cb1004cb8714131ba : SUCCESS in 23m 46s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/e128d387a1d84612a01174095f690d76 : SUCCESS in 16m 58s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e3e8e90ae036458da39960f47a208167 : SUCCESS in 8m 24s\n- grenade https://zuul.opendev.org/t/openstack/build/05cb2c45d6db415aa583ceda166ae799 : SUCCESS in 1h 08m 51s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/d75893c5b8b54ff9a10f6de5bd08973e : SUCCESS in 54m 56s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/77ef6501418443349f62f2882446913f : SUCCESS in 42m 14s\n- neutron-functional-python27 https://zuul.opendev.org/t/openstack/build/054fe1ab1a0144fea5e10c73caff6c65 : SUCCESS in 43m 12s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/bcd9638eb5da498eadb490b2bbf590b9 : SUCCESS in 1h 07m 19s\n- neutron-rally-task https://zuul.opendev.org/t/openstack/build/6d4b47e23a0544bdbac3fa885b5a70a6 : SUCCESS in 1h 54m 31s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/26fe4dc815bf4b8e873565bd6c882e2b : SUCCESS in 1h 20m 07s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/47cc2461d9004d8c97944ef342f2b6f2 : SUCCESS in 1h 07m 10s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/d5ead4cfdcaf42808dc48262693a2ddb : SUCCESS in 1h 32m 52s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/f877638adf9749a7b5d61884c48210d2 : SUCCESS in 1h 22m 13s\n- tempest-multinode-full-py3 https://zuul.opendev.org/t/openstack/build/c9e15b2ffd8642138f85df82ea577d86 : SUCCESS in 1h 40m 37s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full https://zuul.opendev.org/t/openstack/build/9ff5456ccf684fc4be3fd5c486e42326 : SUCCESS in 1h 49m 58s (non-voting)\n- neutron-tempest-iptables_hybrid https://zuul.opendev.org/t/openstack/build/388f9fe073d54b30b3740474237b82e7 : SUCCESS in 55m 48s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/8a453d0e68b6403b8c860690fa532c9d : SUCCESS in 1h 05m 29s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/3938efdaa7994ea3a3aab3145d51c8fd : SUCCESS in 59m 33s (non-voting)\n- tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/d89dba3a0ff44abea8a9be7c8cd2db90 : SUCCESS in 2h 02m 09s\n- networking-ovn-tempest-dsvm-ovs-release https://zuul.opendev.org/t/openstack/build/f4dc3666a82e4b1db49b184640388d23 : SUCCESS in 1h 20m 24s\n- networking-midonet-tempest-aio-ml2-centos-7 https://zuul.opendev.org/t/openstack/build/679cd3c47b164aeeb5a423c82afed796 : FAILURE in 18m 50s (non-voting)\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/1cde56c9dcf94a4d9fdb1c7d37cc49e7 : SUCCESS in 41m 53s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/f567d4a324214ff2943f2e304a04044e : SUCCESS in 57m 31s\n- openstacksdk-functional-devstack-networking https://zuul.opendev.org/t/openstack/build/216d60d5d812428c8ea4c7611980713b : SUCCESS in 1h 21m 45s (non-voting)\n- neutron-functional-with-uwsgi https://zuul.opendev.org/t/openstack/build/892b2ecb63e843949250e6aac24573f6 : SUCCESS in 38m 26s (non-voting)\n- neutron-fullstack-with-uwsgi https://zuul.opendev.org/t/openstack/build/a14fd3103bbd412a9dd4f2cfdd8a8348 : SUCCESS in 51m 35s (non-voting)\n- neutron-tempest-with-uwsgi https://zuul.opendev.org/t/openstack/build/c4bccd43dd8f44f48dd8290c1888057e : SUCCESS in 54m 55s (non-voting)\n- neutron-centos-7-tripleo-standalone https://zuul.opendev.org/t/openstack/build/23f393d167f3425dbbb3125689faad95 : FAILURE in 1h 28m 09s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"063356a05884086a3c3685e247b5dfb0281e293b","author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"date":"2020-06-24 22:30:50.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"ebfd299aeeb338e8db623bdb9270edcc93894010","author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"date":"2020-06-26 14:13:12.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"8232d2b4ea31efce5265cb1a8f29400546cec546","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-26 14:15:32.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"dfd17c338e69153b2da6d7e0b220bc8cfb9738d4","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-26 17:40:26.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"309770af58160e0ff2685bf93b21b356621b4edd","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-26 17:40:26.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- neutron-tempest-plugin-api-train https://zuul.opendev.org/t/openstack/build/4d755ea00c28473dae20a8e30d46cd27 : SUCCESS in 48m 49s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/a338f826ccbd463eb3d738e8cd585606 : SUCCESS in 21m 19s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/0cd0439019234e8aacfa4951ad56ea49 : SUCCESS in 13m 47s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/068c2e82eefa438b914a3db5de5f056a : SUCCESS in 22m 49s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/7169858cc2154d0c976e084f09e84ceb : SUCCESS in 22m 25s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/dce02e27c1954667bef022a669b33e87 : SUCCESS in 22m 11s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/17813c055f944f65932b34c7700681f3 : SUCCESS in 10m 04s\n- grenade https://zuul.opendev.org/t/openstack/build/71da2d5729024e0eb8c324e5d600fa22 : SUCCESS in 1h 09m 24s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/a97988ae07c74c079b027fdc1477d202 : SUCCESS in 1h 12m 58s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/929c740332124dfb96e966f5725f8cbe : SUCCESS in 45m 06s\n- neutron-functional-python27 https://zuul.opendev.org/t/openstack/build/c24798782ce847648e6314f7378698b4 : SUCCESS in 44m 18s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/36cd9284c8b34cd7aa21d4ce1cbeac87 : SUCCESS in 1h 04m 11s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/0fe277c735954ed284dc5b6f0bba2ec0 : SUCCESS in 1h 22m 58s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/4d3a3ca55cb24bcb9484b324d6c9057d : SUCCESS in 1h 14m 50s\n- neutron-tempest-iptables_hybrid https://zuul.opendev.org/t/openstack/build/75647378a73144819d65d74a2345cb91 : SUCCESS in 1h 10m 14s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/4f02e49792f4487c8f4f757cb4eb25d5 : SUCCESS in 1h 10m 55s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/77abfea93c514216b8e8fa237b4c6550 : SUCCESS in 1h 17m 19s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/0b8821d3a20a479182f86cad93d1fd1f : SUCCESS in 58m 52s\n- tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/e162fd5b5b0041c2857a7f75fd5306bf : SUCCESS in 2h 08m 25s\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/e91a2d2d2d1844199e78b1f717c0b3aa : SUCCESS in 1h 14m 56s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/51d58219cd1b44b9ba85afa6d170f481 : SUCCESS in 1h 08m 38s","accounts_in_message":[],"_revision_number":1},{"id":"65e49538301665be51e2739248d7ae28294de526","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-26 17:44:10.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/bf1bdba43ad743b19892b8a5dc8d41f2 : SUCCESS in 2m 54s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"eed9bcbd7be62aa94ff1c3a3543835295be06f04","revisions":{"eed9bcbd7be62aa94ff1c3a3543835295be06f04":{"kind":"REWORK","_number":1,"created":"2020-06-24 07:51:45.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/00/737700/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/00/737700/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/00/737700/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/00/737700/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/00/737700/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/00/737700/1"}}},"commit":{"parents":[{"commit":"1a30c1a587ac7f23a45fb14c6336ec335b79d847","subject":"Merge \"[DVR] Reconfigure re-created physical bridges for dvr routers\" into stable/train","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/1a30c1a587ac7f23a45fb14c6336ec335b79d847"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2020-06-18 21:18:07.000000000","tz":120},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2020-06-24 07:51:45.000000000","tz":0},"subject":"Make _ensure_default_security_group method atomic","message":"Make _ensure_default_security_group method atomic\n\nMethod _ensure_default_security_group wasn\u0027t atomic as it first tries to get\ndefault SG and if that not exists in DB, it tries to create it.\n\nIt may happend, like e.g. in Calico plugin that between\nget_default_sg_id method and create_security_group method, this default\nSG will be created by other neutron worker. And in such case there will\nbe Duplicate entry exception raised.\n\nSo this patch is adding handling of such exception.\n\nChange-Id: I515c310f221e7d9ae3be59a26260538d1bc591c2\nCloses-Bug: #1883730\n(cherry picked from commit 7019c5cf50bc4ad97b302220664a26efaf81a7fd)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/eed9bcbd7be62aa94ff1c3a3543835295be06f04"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/eed9bcbd7be62aa94ff1c3a3543835295be06f04"}]},"branch":"refs/heads/stable/train"}},"requirements":[],"submit_records":[],"submit_requirements":[]}