)]}'
{"id":"openstack%2Fneutron~837726","triplet_id":"openstack%2Fneutron~master~Ibfd91832a26c18c8047e026138546eb7ecd43e5c","project":"openstack/neutron","branch":"master","attention_set":{},"removed_from_attention_set":{"30463":{"account":{"_account_id":30463,"name":"Krzysztof Tomaszewski","display_name":"Krzysztof Tomaszewski","email":"ktomaszewski@cloudferro.com","username":"labedz"},"last_update":"2022-06-15 14:58:25.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"Ibfd91832a26c18c8047e026138546eb7ecd43e5c","subject":"Ensure to remove conntrack after applying deferred openflows","status":"ABANDONED","created":"2022-04-13 12:45:51.000000000","updated":"2022-06-15 14:58:25.000000000","total_comment_count":4,"unresolved_comment_count":3,"work_in_progress":true,"has_review_started":true,"meta_rev_id":"01df25779dd59bb723715cacf804bcd91799c764","_number":837726,"virtual_id_number":837726,"owner":{"_account_id":30463,"name":"Krzysztof Tomaszewski","display_name":"Krzysztof Tomaszewski","email":"ktomaszewski@cloudferro.com","username":"labedz"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"tag":"autogenerated:zuul:check","value":-1,"date":"2022-04-13 14:28:28.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,"date":"2022-04-13 13:44:10.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"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":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"_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,"permitted_voting_range":{"min":-1,"max":2},"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"_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":{"CC":[{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"REVIEWER":[{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{"CC":[{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"REVIEWER":[{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"reviewer_updates":[{"updated":"2022-04-13 12:55:07.000000000","updated_by":{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2022-04-13 13:12: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":"CC"},{"updated":"2022-04-13 13:16:07.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":"CC"},{"updated":"2022-04-13 13:17:06.000000000","updated_by":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2022-04-13 13:19:20.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":"CC"},{"updated":"2022-04-13 13:24:09.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"},{"updated":"2022-04-13 14:28:28.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-06-15 14:58:24.000000000","updated_by":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"reviewer":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"state":"CC"}],"messages":[{"id":"e09c7ae7ba95d6698d476a165a30f7b252f3ebb8","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30463,"name":"Krzysztof Tomaszewski","display_name":"Krzysztof Tomaszewski","email":"ktomaszewski@cloudferro.com","username":"labedz"},"date":"2022-04-13 12:45:51.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3c52abfab4cb2f6901fa91b2e0ba658e48ed1c80","tag":"autogenerated:gerrit:setWorkInProgress","author":{"_account_id":30463,"name":"Krzysztof Tomaszewski","display_name":"Krzysztof Tomaszewski","email":"ktomaszewski@cloudferro.com","username":"labedz"},"date":"2022-04-13 12:46:11.000000000","message":"Set Work In Progress","accounts_in_message":[],"_revision_number":1},{"id":"38f247680f2a858e2c40ead17dea28736cff3303","author":{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},"date":"2022-04-13 12:55:07.000000000","message":"Patch Set 1:\n\nArista third party testing FAILED [ https://arista-openstack-ci.arista.com:8000/April-2022/Arista_Tempest_Testrefs/changes/26/837726/1 ]\n\nFor testing details or any questions please check https://wiki.openstack.org/wiki/Arista-third-party-testing","accounts_in_message":[],"_revision_number":1},{"id":"0c1763fa4cb55d7274509259957f81b7c0881e72","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-04-13 13:12:26.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/4be4306604224c4a895831be223785fd : RETRY_LIMIT in 4m 11s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/20d176017ede44259e87157363490fd6 : RETRY_LIMIT in 3m 44s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"0f9977fe4e11ab2cd963c25ed7b9b8245c010927","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2022-04-13 13:16:07.000000000","message":"Patch Set 1:\n\nFor rechecking only on the Cloudbase Neutron Hyper-V CI, add a review comment with run-Cloudbase Neutron Hyper-V CI\n\n- neutron http://cloudbase-ci.com/neutron/837726/1 : FAILURE in 29m 41s","accounts_in_message":[],"_revision_number":1},{"id":"3741a645a5b6c6f45ab89720edd882c4fedbecc5","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2022-04-13 13:17:06.000000000","message":"Patch Set 1:\n\nBuild Failed \n\n* openstack-sriov-neutron-master-direct http://13.74.249.42/refs/changes/26/837726/1/openstack-sriov-neutron-master-direct : ABORTED\n\n* openstack-sriov-neutron-master-macvtap http://13.74.249.42/refs/changes/26/837726/1/openstack-sriov-neutron-master-macvtap : ABORTED","accounts_in_message":[],"_revision_number":1},{"id":"faa9eb4c8bdf909465a11c7ef6d6e04d0a67e131","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2022-04-13 13:19:20.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"f2922b08d3bc3faaba0a85f282b9787222fdaefa","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2022-04-13 13:24:09.000000000","message":"Patch Set 1: Code-Review-1","accounts_in_message":[],"_revision_number":1},{"id":"24f350a1ec7ba8ce0fa66478b080affa19268f0f","tag":"autogenerated:gerrit:deleteVote","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2022-04-13 13:30:53.000000000","message":"Removed Code-Review-1 by Rodolfo Alonso \u003cralonsoh@redhat.com\u003e\n","accounts_in_message":[],"_revision_number":1},{"id":"12dbe946dda27e8677ff0d8559b857ea75918f47","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2022-04-13 13:30:59.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"1cda34679d9fee41510fb3b4ce13f72232df8390","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2022-04-13 13:44:10.000000000","message":"Patch Set 1:\n\n(2 comments)","accounts_in_message":[],"_revision_number":1},{"id":"d2abc6d1b787371aae779527e0ef8054100dafc8","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-04-13 14:28:28.000000000","message":"Patch Set 1: 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\n\n\n- neutron-tempest-plugin-scenario-linuxbridge https://zuul.opendev.org/t/openstack/build/83953c216c32478ba84b8e32c2591066 : FAILURE in 10m 11s\n- neutron-tempest-plugin-scenario-openvswitch https://zuul.opendev.org/t/openstack/build/ac93396fa834480c8591c821a51e4f17 : FAILURE in 10m 13s\n- neutron-tempest-plugin-scenario-openvswitch-iptables_hybrid https://zuul.opendev.org/t/openstack/build/2597ca000f8a4e56af083751a5a1d1cb : FAILURE in 10m 17s\n- neutron-tempest-plugin-scenario-ovn https://zuul.opendev.org/t/openstack/build/64cbe6ac7b944f829cf81692c4babc2c : FAILURE in 10m 29s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/2b174d5512fc4daea96a0e2649af81a7 : SUCCESS in 40m 13s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/847a5bdb62964772a2ba117759286584 : SUCCESS in 35m 03s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/196b37d4b5a24d99b8d3cbb1217601b6 : SUCCESS in 15m 15s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/3e27c9788d6f437abd56d5afa40c7f8d : SUCCESS in 22m 21s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/b6b132fcc85b4ed09ca4054d6878cb39 : SUCCESS in 28m 08s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/41c3abcdac3c4bc28d53652d8172d3f2 : SUCCESS in 34m 14s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ff2da7a57c5543ffa88cf65f7801ec53 : SUCCESS in 9m 30s\n- neutron-functional-with-uwsgi https://zuul.opendev.org/t/openstack/build/1df6fbc053e84c15905a3d68f293f780 : SUCCESS in 1h 02m 14s\n- neutron-fullstack-with-uwsgi https://zuul.opendev.org/t/openstack/build/6689a7daac8245d18b8c4621853efc5e : SUCCESS in 1h 34m 58s\n- neutron-ovs-rally-task https://zuul.opendev.org/t/openstack/build/9fa01610a6cc4085a83c06d094f62dd9 : FAILURE in 16m 30s\n- neutron-ovs-grenade-multinode https://zuul.opendev.org/t/openstack/build/d9b9fba9c6814efdbc4e66da3883ad01 : FAILURE in 25m 31s\n- neutron-ovs-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/80569849cdfa4d38a4dab1d48176820e : FAILURE in 25m 47s\n- neutron-ovs-tempest-multinode-full https://zuul.opendev.org/t/openstack/build/a4c738e2eaa4462ebc51903206a842e7 : FAILURE in 19m 53s\n- neutron-ovs-tempest-dvr-ha-multinode-full https://zuul.opendev.org/t/openstack/build/737fb0a86f91438f8b0725418128ccae : FAILURE in 22m 09s\n- neutron-ovn-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9cd9187d68e84c84ba3318409e39f031 : FAILURE in 16m 16s\n- neutron-ovn-rally-task https://zuul.opendev.org/t/openstack/build/66ee651a2564469490bae4a9fb882fa6 : FAILURE in 9m 20s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"990ce9499ce987e4dc73d9a11233fa6b1296e357","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2022-04-13 14:36:02.000000000","message":"Patch Set 1:\n\nBuild Successful \n\n* openstack-sriov-neutron-master-direct http://13.74.249.42/refs/changes/26/837726/1/openstack-sriov-neutron-master-direct : SUCCESS\n\n* openstack-sriov-neutron-master-macvtap http://13.74.249.42/refs/changes/26/837726/1/openstack-sriov-neutron-master-macvtap : SUCCESS","accounts_in_message":[],"_revision_number":1},{"id":"01df25779dd59bb723715cacf804bcd91799c764","tag":"autogenerated:gerrit:abandon","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2022-06-15 14:58:25.000000000","message":"Abandoned\n\nThis review is \u003e 4 weeks without comment, and failed Zuul jobs the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a \u0027recheck\u0027 comment to get fresh test results.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"2e845a225f42c034b7cee71060bb9bfc308127f8","revisions":{"2e845a225f42c034b7cee71060bb9bfc308127f8":{"kind":"REWORK","_number":1,"created":"2022-04-13 12:45:51.000000000","uploader":{"_account_id":30463,"name":"Krzysztof Tomaszewski","display_name":"Krzysztof Tomaszewski","email":"ktomaszewski@cloudferro.com","username":"labedz"},"ref":"refs/changes/26/837726/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/26/837726/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/26/837726/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/26/837726/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/26/837726/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/26/837726/1"}}},"commit":{"parents":[{"commit":"bb4c26eb7245465bf7cea7e0f07342601eb78ede","subject":"Merge \"Also add B324 to bandit skip list for python3.9+\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/bb4c26eb7245465bf7cea7e0f07342601eb78ede"}]}],"author":{"name":"labedz","email":"krzysztof.tomaszewski@ovhcloud.com","date":"2022-04-13 12:22:39.000000000","tz":0},"committer":{"name":"labedz","email":"krzysztof.tomaszewski@ovhcloud.com","date":"2022-04-13 12:32:07.000000000","tz":120},"subject":"Ensure to remove conntrack after applying deferred openflows","message":"Ensure to remove conntrack after applying deferred openflows\n\nAvoid race condition between applying deferred openflow rules\nand removing conntrack entries by ensuring that action\nto remove conntrack entries will be started after applying\nflows.\n\nDeferred openflow rules are applies on context exit in\n_apply_port_filter[1] but conntrack entries removing action\nstored in greenlet queue is triggered in the last moment by eventlet sleep in\nprocess_trusted_ports() [2] and also in meantime\nduring openflow batch applying by run_ofctl(). That leads to race condition\nbetween conntrack removal and openflow apply. It may happen that\nconntrack removal action is executed before coresponding flow is\napplied.\n\nTo remove this race condition set a lock on processing conntrack queue\nduring deferred flow apply.\n\nPartial-Bug: #1934917\n\n[1] agent/securitygroups_rpc.py L#175\n[2] openvswitch_firewall/firewall.py L#854\n\nChange-Id: Ibfd91832a26c18c8047e026138546eb7ecd43e5c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/2e845a225f42c034b7cee71060bb9bfc308127f8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/2e845a225f42c034b7cee71060bb9bfc308127f8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}