)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"cd0bb0197830e41c0078d947d5beb9ce5e0f8a57","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3742c292_db3b9d53","updated":"2023-03-24 12:19:26.000000000","message":"recheck\nhmmmm, seems the patch is sutck in the zuul queue, neutron-tempest-plugin-ovn remained in queued state","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"b0cb0eacf72fb7756f14078b66cea1f72c018d23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5ac275bd_2a5d590a","updated":"2023-03-23 12:27:46.000000000","message":"recheck\nneutron-tempest-plugin-openvswitch-iptables_hybrid-zed timeout","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"ff8840971701c74bc3723cb2d817d9ac83dd4f7a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"de24e13b_bd0885a2","updated":"2023-03-23 08:00:59.000000000","message":"recheck - unrelated qos and ssh failures","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2d4343485a3287484c016cb66e53e1d89d0c3695","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"19b62dcf_74b88ced","updated":"2023-03-23 13:29:49.000000000","message":"recheck - unrelated timeout","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"}],"neutron_tempest_plugin/scenario/test_security_groups.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"e29fd34077c543d5d65883a2588a8318ffcba44f","unresolved":true,"context_lines":[{"line_number":866,"context_line":"@testtools.skipIf("},{"line_number":867,"context_line":"    CONF.neutron_plugin_options.firewall_driver in [\u0027openvswitch\u0027, \u0027None\u0027],"},{"line_number":868,"context_line":"    \"Firewall driver other than \u0027openvswitch\u0027 is required to use stateless \""},{"line_number":869,"context_line":"    \"security groups.\")"},{"line_number":870,"context_line":"class StatelessNetworkSecGroupTest(BaseNetworkSecGroupTest):"},{"line_number":871,"context_line":"    required_extensions \u003d [\u0027security-group\u0027, \u0027stateful-security-group\u0027]"},{"line_number":872,"context_line":"    stateless_sg \u003d True"}],"source_content_type":"text/x-python","patch_set":1,"id":"0bf9e2cc_e69366d5","line":869,"updated":"2023-03-20 17:19:36.000000000","message":"So with OVN the firewall_driver setting is ignored, right? So it can be None? I guess I figured that was the default but I do see the tempest tests here were run.","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"6a2e086c908a4f03772686d3728cd92064dc9e77","unresolved":true,"context_lines":[{"line_number":866,"context_line":"@testtools.skipIf("},{"line_number":867,"context_line":"    CONF.neutron_plugin_options.firewall_driver in [\u0027openvswitch\u0027, \u0027None\u0027],"},{"line_number":868,"context_line":"    \"Firewall driver other than \u0027openvswitch\u0027 is required to use stateless \""},{"line_number":869,"context_line":"    \"security groups.\")"},{"line_number":870,"context_line":"class StatelessNetworkSecGroupTest(BaseNetworkSecGroupTest):"},{"line_number":871,"context_line":"    required_extensions \u003d [\u0027security-group\u0027, \u0027stateful-security-group\u0027]"},{"line_number":872,"context_line":"    stateless_sg \u003d True"}],"source_content_type":"text/x-python","patch_set":1,"id":"66afd299_b4219ee9","line":869,"in_reply_to":"0bf9e2cc_e69366d5","updated":"2023-03-21 10:14:57.000000000","message":"With OVN backend firewall_driver is set to \"ovn\" (it\u0027s tempest config, not neutron) and those tests are run properly, see https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b29/877944/1/check/neutron-tempest-plugin-ovn/b297dbf/controller/logs/local_conf.txt and test results: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b29/877944/1/check/neutron-tempest-plugin-ovn/b297dbf/testr_results.html","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5e29508ef45818ca96f51de2afae29bee04143d6","unresolved":false,"context_lines":[{"line_number":866,"context_line":"@testtools.skipIf("},{"line_number":867,"context_line":"    CONF.neutron_plugin_options.firewall_driver in [\u0027openvswitch\u0027, \u0027None\u0027],"},{"line_number":868,"context_line":"    \"Firewall driver other than \u0027openvswitch\u0027 is required to use stateless \""},{"line_number":869,"context_line":"    \"security groups.\")"},{"line_number":870,"context_line":"class StatelessNetworkSecGroupTest(BaseNetworkSecGroupTest):"},{"line_number":871,"context_line":"    required_extensions \u003d [\u0027security-group\u0027, \u0027stateful-security-group\u0027]"},{"line_number":872,"context_line":"    stateless_sg \u003d True"}],"source_content_type":"text/x-python","patch_set":1,"id":"c7dc569f_bb64fce4","line":869,"in_reply_to":"66afd299_b4219ee9","updated":"2023-03-21 14:07:23.000000000","message":"Ack","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"e29fd34077c543d5d65883a2588a8318ffcba44f","unresolved":true,"context_lines":[{"line_number":907,"context_line":""},{"line_number":908,"context_line":"    @testtools.skipUnless("},{"line_number":909,"context_line":"        CONF.neutron_plugin_options.firewall_driver \u003d\u003d \u0027openvswitch\u0027,"},{"line_number":910,"context_line":"        \"Openvswitch agent is required to run this test\")"},{"line_number":911,"context_line":"    @decorators.idempotent_id(\u00279fae530d-2711-4c61-a4a5-8efe6e58ab14\u0027)"},{"line_number":912,"context_line":"    @tempest_utils.requires_ext(extension\u003d\"address-group\", service\u003d\"network\")"},{"line_number":913,"context_line":"    @tempest_utils.requires_ext("}],"source_content_type":"text/x-python","patch_set":1,"id":"2fc4e9e4_bc12b359","line":910,"updated":"2023-03-20 17:19:36.000000000","message":"So this test only runs with openvswitch firewall driver, can this be removed?","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"6a2e086c908a4f03772686d3728cd92064dc9e77","unresolved":true,"context_lines":[{"line_number":907,"context_line":""},{"line_number":908,"context_line":"    @testtools.skipUnless("},{"line_number":909,"context_line":"        CONF.neutron_plugin_options.firewall_driver \u003d\u003d \u0027openvswitch\u0027,"},{"line_number":910,"context_line":"        \"Openvswitch agent is required to run this test\")"},{"line_number":911,"context_line":"    @decorators.idempotent_id(\u00279fae530d-2711-4c61-a4a5-8efe6e58ab14\u0027)"},{"line_number":912,"context_line":"    @tempest_utils.requires_ext(extension\u003d\"address-group\", service\u003d\"network\")"},{"line_number":913,"context_line":"    @tempest_utils.requires_ext("}],"source_content_type":"text/x-python","patch_set":1,"id":"fb0d7ad9_73c5bdbf","line":910,"in_reply_to":"2fc4e9e4_bc12b359","updated":"2023-03-21 10:14:57.000000000","message":"I can remove it but if at some point we will implement support for stateless SG for openvswitch fw driver, we will probably forget to add this test again to this class. Will it be problem if we will keep it here like it\u0027s now?","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5e29508ef45818ca96f51de2afae29bee04143d6","unresolved":true,"context_lines":[{"line_number":907,"context_line":""},{"line_number":908,"context_line":"    @testtools.skipUnless("},{"line_number":909,"context_line":"        CONF.neutron_plugin_options.firewall_driver \u003d\u003d \u0027openvswitch\u0027,"},{"line_number":910,"context_line":"        \"Openvswitch agent is required to run this test\")"},{"line_number":911,"context_line":"    @decorators.idempotent_id(\u00279fae530d-2711-4c61-a4a5-8efe6e58ab14\u0027)"},{"line_number":912,"context_line":"    @tempest_utils.requires_ext(extension\u003d\"address-group\", service\u003d\"network\")"},{"line_number":913,"context_line":"    @tempest_utils.requires_ext("}],"source_content_type":"text/x-python","patch_set":1,"id":"5a79d582_5e2d913e","line":910,"in_reply_to":"fb0d7ad9_73c5bdbf","updated":"2023-03-21 14:07:23.000000000","message":"No, I guess it\u0027s fine to leave it. I guess if @testtools.skipUnless() were used it would be more obvious this only applied to OVN (for now).","commit_id":"9b04e7bdc18152e39a5f74dad8fb5da555c867e3"}]}