)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"df735d025b047bbfd1d64708ffb1137da3156cd4","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We had a bug for OSP13 with openvswitch firewall driver that the established"},{"line_number":10,"context_line":"connection can\u0027t be resumed after the security group has been removed from"},{"line_number":11,"context_line":"the port and than added back."},{"line_number":12,"context_line":"There are 2 new tests:"},{"line_number":13,"context_line":"- To make sure that the new connection can be initiated after security"},{"line_number":14,"context_line":"group has been re-attached to the server"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"2a7d7e52_85128f2c","line":11,"updated":"2021-05-31 14:52:51.000000000","message":"I think that it is this bug https://bugs.launchpad.net/neutron/+bug/1915530 so You can mark this patch as related","commit_id":"cb4ae7523d63b5c72b1c332d5e8f0ced60c4e594"},{"author":{"_account_id":31450,"name":"Alex Katz","email":"akatz@redhat.com","username":"akatz"},"change_message_id":"13e0586a9a1b08005291629748875d6f1d4fd854","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We had a bug for OSP13 with openvswitch firewall driver that the established"},{"line_number":10,"context_line":"connection can\u0027t be resumed after the security group has been removed from"},{"line_number":11,"context_line":"the port and than added back."},{"line_number":12,"context_line":"There are 2 new tests:"},{"line_number":13,"context_line":"- To make sure that the new connection can be initiated after security"},{"line_number":14,"context_line":"group has been re-attached to the server"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1bc5b0a2_45a8303c","line":11,"in_reply_to":"2a7d7e52_85128f2c","updated":"2021-06-01 07:55:48.000000000","message":"Done","commit_id":"cb4ae7523d63b5c72b1c332d5e8f0ced60c4e594"}],"neutron_tempest_plugin/common/utils.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"df735d025b047bbfd1d64708ffb1137da3156cd4","unresolved":true,"context_lines":[{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    def __init__(self, client_ssh, server_ssh, target_ip, target_port):"},{"line_number":153,"context_line":"        self.c_ssh \u003d client_ssh"},{"line_number":154,"context_line":"        self.s_ssh \u003d server_ssh"},{"line_number":155,"context_line":"        self.ip \u003d target_ip"},{"line_number":156,"context_line":"        self.port \u003d target_port"},{"line_number":157,"context_line":"        self.connection_started \u003d False"}],"source_content_type":"text/x-python","patch_set":2,"id":"d5fcd21a_2f2530b6","line":154,"updated":"2021-05-31 14:52:51.000000000","message":"please name it \"client_ssh\" and \"server_ssh\" or \"client\" and \"server\".","commit_id":"cb4ae7523d63b5c72b1c332d5e8f0ced60c4e594"},{"author":{"_account_id":31450,"name":"Alex Katz","email":"akatz@redhat.com","username":"akatz"},"change_message_id":"13e0586a9a1b08005291629748875d6f1d4fd854","unresolved":false,"context_lines":[{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    def __init__(self, client_ssh, server_ssh, target_ip, target_port):"},{"line_number":153,"context_line":"        self.c_ssh \u003d client_ssh"},{"line_number":154,"context_line":"        self.s_ssh \u003d server_ssh"},{"line_number":155,"context_line":"        self.ip \u003d target_ip"},{"line_number":156,"context_line":"        self.port \u003d target_port"},{"line_number":157,"context_line":"        self.connection_started \u003d False"}],"source_content_type":"text/x-python","patch_set":2,"id":"7d0e2b70_a114c08a","line":154,"in_reply_to":"d5fcd21a_2f2530b6","updated":"2021-06-01 07:55:48.000000000","message":"Done","commit_id":"cb4ae7523d63b5c72b1c332d5e8f0ced60c4e594"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"df735d025b047bbfd1d64708ffb1137da3156cd4","unresolved":true,"context_lines":[{"line_number":202,"context_line":"            except ConnectionError as e:"},{"line_number":203,"context_line":"                if time.time() \u003e end_time:"},{"line_number":204,"context_line":"                    raise e"},{"line_number":205,"context_line":"            time.sleep(sleep_timer)"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    def __exit__(self, type, value, traceback):"},{"line_number":208,"context_line":"        self.s_ssh.exec_command(\u0027killall nc\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"20f913d4_8655c17d","line":205,"updated":"2021-05-31 14:52:51.000000000","message":"You can return True/False from the _test_connection() method and then use wait_until_true() function which we already have.","commit_id":"cb4ae7523d63b5c72b1c332d5e8f0ced60c4e594"},{"author":{"_account_id":31450,"name":"Alex Katz","email":"akatz@redhat.com","username":"akatz"},"change_message_id":"13e0586a9a1b08005291629748875d6f1d4fd854","unresolved":false,"context_lines":[{"line_number":202,"context_line":"            except ConnectionError as e:"},{"line_number":203,"context_line":"                if time.time() \u003e end_time:"},{"line_number":204,"context_line":"                    raise e"},{"line_number":205,"context_line":"            time.sleep(sleep_timer)"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    def __exit__(self, type, value, traceback):"},{"line_number":208,"context_line":"        self.s_ssh.exec_command(\u0027killall nc\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"eec8fa2c_102ae93c","line":205,"in_reply_to":"20f913d4_8655c17d","updated":"2021-06-01 07:55:48.000000000","message":"Done","commit_id":"cb4ae7523d63b5c72b1c332d5e8f0ced60c4e594"}],"neutron_tempest_plugin/scenario/test_security_groups.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"df735d025b047bbfd1d64708ffb1137da3156cd4","unresolved":true,"context_lines":[{"line_number":342,"context_line":"            con.test_connection(should_pass\u003dFalse)"},{"line_number":343,"context_line":"            self.client.update_port(srv_port[\u0027id\u0027],"},{"line_number":344,"context_line":"                    security_groups\u003d[ssh_sg[\u0027id\u0027], sg[\u0027id\u0027]])"},{"line_number":345,"context_line":"            con.test_connection()"},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"    @decorators.idempotent_id(\u00277ed39b86-006d-40fb-887a-ae46693dabc9\u0027)"},{"line_number":348,"context_line":"    def test_remote_group(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"4289cca5_13ef7002","line":345,"updated":"2021-05-31 14:52:51.000000000","message":"can You maybe combine those 2 tests into one? I think that it would require add one additional step to the previous test only but it would avoid spawning new vms and everything else again so should be faster in overall.","commit_id":"cb4ae7523d63b5c72b1c332d5e8f0ced60c4e594"},{"author":{"_account_id":31450,"name":"Alex Katz","email":"akatz@redhat.com","username":"akatz"},"change_message_id":"13e0586a9a1b08005291629748875d6f1d4fd854","unresolved":false,"context_lines":[{"line_number":342,"context_line":"            con.test_connection(should_pass\u003dFalse)"},{"line_number":343,"context_line":"            self.client.update_port(srv_port[\u0027id\u0027],"},{"line_number":344,"context_line":"                    security_groups\u003d[ssh_sg[\u0027id\u0027], sg[\u0027id\u0027]])"},{"line_number":345,"context_line":"            con.test_connection()"},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"    @decorators.idempotent_id(\u00277ed39b86-006d-40fb-887a-ae46693dabc9\u0027)"},{"line_number":348,"context_line":"    def test_remote_group(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"14e0d094_340e78c8","line":345,"in_reply_to":"4289cca5_13ef7002","updated":"2021-06-01 07:55:48.000000000","message":"Done","commit_id":"cb4ae7523d63b5c72b1c332d5e8f0ced60c4e594"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a00ea7c9f4467674f421cde8a241f7f49ec2d4a2","unresolved":true,"context_lines":[{"line_number":282,"context_line":"        \"\"\"Test existing connection remain open after sg has been re-attached"},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"        Verifies that new packets can pass over the existing connection when"},{"line_number":285,"context_line":"        the security group has been removed from the server and than added"},{"line_number":286,"context_line":"        back"},{"line_number":287,"context_line":"        \"\"\""},{"line_number":288,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"faeeb630_0da8bc92","line":285,"range":{"start_line":285,"start_character":64,"end_line":285,"end_character":68},"updated":"2021-06-08 09:12:23.000000000","message":"then","commit_id":"248fd03198f2497443cbb3ba6657f61157a2b8cf"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"dc2ea2807919500c71c778b996aa96857c6285ee","unresolved":false,"context_lines":[{"line_number":282,"context_line":"        \"\"\"Test existing connection remain open after sg has been re-attached"},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"        Verifies that new packets can pass over the existing connection when"},{"line_number":285,"context_line":"        the security group has been removed from the server and than added"},{"line_number":286,"context_line":"        back"},{"line_number":287,"context_line":"        \"\"\""},{"line_number":288,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"fcf581a8_cd74b5d5","line":285,"range":{"start_line":285,"start_character":64,"end_line":285,"end_character":68},"in_reply_to":"faeeb630_0da8bc92","updated":"2021-06-09 07:18:37.000000000","message":"Done","commit_id":"248fd03198f2497443cbb3ba6657f61157a2b8cf"}],"zuul.d/master_jobs.yaml":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"f484e1b340a5a3a275bb1b0de3a6135912c7b0cd","unresolved":true,"context_lines":[{"line_number":210,"context_line":"      # bug https://bugzilla.redhat.com/show_bug.cgi?id\u003d1965036 will be fixed"},{"line_number":211,"context_line":"      tempest_exclude_regex: \"\\"},{"line_number":212,"context_line":"          (^neutron_tempest_plugin.scenario.test_trunk.TrunkTest.test_subport_connectivity)|\\"},{"line_number":213,"context_line":"          (^neutron_tempest_plugin.scenario.test_security_groups.NetworkSecGroupTest.test_established_tcp_session_after_re_attachinging_sg)\""},{"line_number":214,"context_line":"      devstack_localrc:"},{"line_number":215,"context_line":"        Q_AGENT: openvswitch"},{"line_number":216,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"a1057232_1034533d","line":213,"updated":"2021-06-07 07:48:13.000000000","message":"interesting that it works fine in the linuxbridge job, which is also using iptables (but not hybrid) driver.","commit_id":"248fd03198f2497443cbb3ba6657f61157a2b8cf"},{"author":{"_account_id":31450,"name":"Alex Katz","email":"akatz@redhat.com","username":"akatz"},"change_message_id":"034d82aca10858edab8bb074d87128303e1c95c3","unresolved":true,"context_lines":[{"line_number":210,"context_line":"      # bug https://bugzilla.redhat.com/show_bug.cgi?id\u003d1965036 will be fixed"},{"line_number":211,"context_line":"      tempest_exclude_regex: \"\\"},{"line_number":212,"context_line":"          (^neutron_tempest_plugin.scenario.test_trunk.TrunkTest.test_subport_connectivity)|\\"},{"line_number":213,"context_line":"          (^neutron_tempest_plugin.scenario.test_security_groups.NetworkSecGroupTest.test_established_tcp_session_after_re_attachinging_sg)\""},{"line_number":214,"context_line":"      devstack_localrc:"},{"line_number":215,"context_line":"        Q_AGENT: openvswitch"},{"line_number":216,"context_line":"        Q_ML2_TENANT_NETWORK_TYPE: vxlan"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"0f629ff6_c99abc6b","line":213,"in_reply_to":"a1057232_1034533d","updated":"2021-06-08 04:23:07.000000000","message":"I saw it passed in `neutron-tempest-plugin-scenario-openvswitch-iptables_hybrid-wallaby` on June the 3rd. So maybe there is a kind of race condition","commit_id":"248fd03198f2497443cbb3ba6657f61157a2b8cf"}]}