)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5ec345966b20cd4c9f9a279cdbec6ab4705185de","unresolved":true,"context_lines":[{"line_number":14,"context_line":"from the module neutron_tempest_plugin.scenario.test_security_groups"},{"line_number":15,"context_line":"should be skipped in such case."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-bz: #2227138"},{"line_number":18,"context_line":"Change-Id: Ice0f3573dd51851752c745c0b1d35c82bd7808fd"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c04841c5_8775572a","line":17,"updated":"2023-08-01 19:19:11.000000000","message":"nit: this doesn\u0027t show as a hyperlink since zuul doesn\u0027t understand BZ, don\u0027t know if you should just put the full link","commit_id":"0ac45f3c5ecf2ebbcaeafe027fa487bcf77afef9"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9af06af6127d9d4d526d4a261d30dc4ca966be55","unresolved":false,"context_lines":[{"line_number":14,"context_line":"from the module neutron_tempest_plugin.scenario.test_security_groups"},{"line_number":15,"context_line":"should be skipped in such case."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-bz: #2227138"},{"line_number":18,"context_line":"Change-Id: Ice0f3573dd51851752c745c0b1d35c82bd7808fd"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"914eced6_570cfe42","line":17,"in_reply_to":"79902c4f_b42ed9f5","updated":"2023-08-02 14:58:30.000000000","message":"Done","commit_id":"0ac45f3c5ecf2ebbcaeafe027fa487bcf77afef9"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"075900a3e039c8b120266ad6135911fd842f3131","unresolved":true,"context_lines":[{"line_number":14,"context_line":"from the module neutron_tempest_plugin.scenario.test_security_groups"},{"line_number":15,"context_line":"should be skipped in such case."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-bz: #2227138"},{"line_number":18,"context_line":"Change-Id: Ice0f3573dd51851752c745c0b1d35c82bd7808fd"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"79902c4f_b42ed9f5","line":17,"in_reply_to":"c04841c5_8775572a","updated":"2023-08-02 07:25:51.000000000","message":"+1","commit_id":"0ac45f3c5ecf2ebbcaeafe027fa487bcf77afef9"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5ec345966b20cd4c9f9a279cdbec6ab4705185de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2d0d8932_a384f55e","updated":"2023-08-01 19:19:11.000000000","message":"Curious about Yatin\u0027s question as well","commit_id":"0ac45f3c5ecf2ebbcaeafe027fa487bcf77afef9"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"075900a3e039c8b120266ad6135911fd842f3131","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20082cf5_f26ee39c","updated":"2023-08-02 07:25:51.000000000","message":"Other than the nit from haleyb, I am fine with it","commit_id":"0ac45f3c5ecf2ebbcaeafe027fa487bcf77afef9"}],"neutron_tempest_plugin/scenario/test_security_groups.py":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"58447f083822261149eb5c383e43c6107556760a","unresolved":true,"context_lines":[{"line_number":1184,"context_line":"        self._test_default_sec_grp_scenarios()"},{"line_number":1185,"context_line":""},{"line_number":1186,"context_line":""},{"line_number":1187,"context_line":"@testtools.skipIf("},{"line_number":1188,"context_line":"    CONF.neutron_plugin_options.firewall_driver in [\u0027openvswitch\u0027, \u0027None\u0027],"},{"line_number":1189,"context_line":"    \"Firewall driver other than \u0027openvswitch\u0027 is required to use \""},{"line_number":1190,"context_line":"    \"stateless security groups.\")"},{"line_number":1191,"context_line":"class StatelessSecGroupDualStackDHCPv6StatelessTest("},{"line_number":1192,"context_line":"        StatelessSecGroupDualStackSlaacTest):"},{"line_number":1193,"context_line":"    required_extensions \u003d [\u0027security-group\u0027, \u0027stateful-security-group\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"a9c79599_527094d6","line":1190,"range":{"start_line":1187,"start_character":0,"end_line":1190,"end_character":33},"updated":"2023-08-01 14:04:08.000000000","message":"Do we need to add skip explicitly here considering the parent class have Skip added?","commit_id":"0ac45f3c5ecf2ebbcaeafe027fa487bcf77afef9"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"0f42680ef3c507dbad4218fff2fff7489db54523","unresolved":true,"context_lines":[{"line_number":1184,"context_line":"        self._test_default_sec_grp_scenarios()"},{"line_number":1185,"context_line":""},{"line_number":1186,"context_line":""},{"line_number":1187,"context_line":"@testtools.skipIf("},{"line_number":1188,"context_line":"    CONF.neutron_plugin_options.firewall_driver in [\u0027openvswitch\u0027, \u0027None\u0027],"},{"line_number":1189,"context_line":"    \"Firewall driver other than \u0027openvswitch\u0027 is required to use \""},{"line_number":1190,"context_line":"    \"stateless security groups.\")"},{"line_number":1191,"context_line":"class StatelessSecGroupDualStackDHCPv6StatelessTest("},{"line_number":1192,"context_line":"        StatelessSecGroupDualStackSlaacTest):"},{"line_number":1193,"context_line":"    required_extensions \u003d [\u0027security-group\u0027, \u0027stateful-security-group\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7100a3fc_f72c38c4","line":1190,"range":{"start_line":1187,"start_character":0,"end_line":1190,"end_character":33},"in_reply_to":"4b31e2fd_1e718207","updated":"2023-08-02 15:00:15.000000000","message":"fine by me to keep it explicit :)","commit_id":"0ac45f3c5ecf2ebbcaeafe027fa487bcf77afef9"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9af06af6127d9d4d526d4a261d30dc4ca966be55","unresolved":true,"context_lines":[{"line_number":1184,"context_line":"        self._test_default_sec_grp_scenarios()"},{"line_number":1185,"context_line":""},{"line_number":1186,"context_line":""},{"line_number":1187,"context_line":"@testtools.skipIf("},{"line_number":1188,"context_line":"    CONF.neutron_plugin_options.firewall_driver in [\u0027openvswitch\u0027, \u0027None\u0027],"},{"line_number":1189,"context_line":"    \"Firewall driver other than \u0027openvswitch\u0027 is required to use \""},{"line_number":1190,"context_line":"    \"stateless security groups.\")"},{"line_number":1191,"context_line":"class StatelessSecGroupDualStackDHCPv6StatelessTest("},{"line_number":1192,"context_line":"        StatelessSecGroupDualStackSlaacTest):"},{"line_number":1193,"context_line":"    required_extensions \u003d [\u0027security-group\u0027, \u0027stateful-security-group\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"4b31e2fd_1e718207","line":1190,"range":{"start_line":1187,"start_character":0,"end_line":1190,"end_character":33},"in_reply_to":"a9c79599_527094d6","updated":"2023-08-02 14:58:30.000000000","message":"yes, it would be enough but IMHO it\u0027s more clear if we have decorated each class like that. If You really want to remove it from here, I can do that, just please let me know 😊","commit_id":"0ac45f3c5ecf2ebbcaeafe027fa487bcf77afef9"}]}