)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"c4624a20d22f3c707cafb4c390433ab645e18dcd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"db1b895f_f8dc1fd1","updated":"2021-11-04 03:18:58.000000000","message":"recheck","commit_id":"f6e39803b1f5ce1b537529ff48c46958b3c5f8a0"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"1dd523dcbbeb0ebf431f10c52c6762b92306c095","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b68b2ec4_c3c1f068","updated":"2021-11-04 16:47:06.000000000","message":"recheck","commit_id":"12fbac7b98d6843c4a38db5a287576137671135e"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"33a1b33ed7fc8e16b635f342d9722217d9fdc13a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"cd9b450b_473e7e92","updated":"2021-11-29 14:30:07.000000000","message":"recheck neutron-tempest-plugin-scenario-ovn timeout","commit_id":"77402d803a07898504fcb25bde285b86ab180ddf"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"2cad974502b1e00be6652183e9776451b5e4f047","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"09540d62_ea1fd03b","updated":"2021-11-30 13:20:52.000000000","message":"Thanks Ihar!","commit_id":"b0b597f803328b021474f1072f2cdeff9297391a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4c0cc0cb39236f0fca82f6e80f8e78f5259980f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"713c347a_2610d470","updated":"2021-11-30 16:46:55.000000000","message":"recheck","commit_id":"b0b597f803328b021474f1072f2cdeff9297391a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"321eb7d75524aa509ea8abb1722b0405c228ee1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f628e818_41631afb","updated":"2021-11-30 09:08:42.000000000","message":"recheck","commit_id":"b0b597f803328b021474f1072f2cdeff9297391a"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f2439bbdfee8a7c5a10e6bb0fbb2f04f3018da09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"fb00e00b_85564415","updated":"2021-12-01 00:05:38.000000000","message":"recheck neutron-tempest-plugin-bgpvpn-bagpipe-wallaby","commit_id":"b0b597f803328b021474f1072f2cdeff9297391a"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f6110fc759fbca8e33e40b907071e94bd60ede52","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ec27b4a7_1f38958a","updated":"2021-11-29 18:36:37.000000000","message":"recheck rebased dep","commit_id":"b0b597f803328b021474f1072f2cdeff9297391a"}],"neutron_tempest_plugin/api/test_security_groups.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"26985e97c826ad8688556c9d0ae31ac2d10f0a9f","unresolved":true,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"class StatelessSecGroupTest(base.BaseAdminNetworkTest):"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"    required_extensions \u003d [\u0027security-group\u0027, \u0027stateful-security-group\u0027]"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    @decorators.idempotent_id(\u00270a6c1476-3d1a-11ec-b0ec-0800277ac3d9\u0027)"},{"line_number":151,"context_line":"    def test_stateless_security_group(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"415c7438_c5f95ce2","line":148,"updated":"2021-11-12 15:44:47.000000000","message":"maybe You can add that \"stateful-security-group\" to the list of enabled extensions in the definiton of the neutron-tempest-plugin-api job so this test will not be skipped.","commit_id":"12fbac7b98d6843c4a38db5a287576137671135e"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"a06019cf64eb9f57b31c9093827db3699ac011c9","unresolved":true,"context_lines":[{"line_number":148,"context_line":"    required_extensions \u003d [\u0027security-group\u0027, \u0027stateful-security-group\u0027]"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    @decorators.idempotent_id(\u00270a6c1476-3d1a-11ec-b0ec-0800277ac3d9\u0027)"},{"line_number":151,"context_line":"    def test_stateless_security_group(self):"},{"line_number":152,"context_line":"        security_group \u003d self.create_security_group(stateful\u003dTrue)"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"        # List security groups and verify if created group is there in response"}],"source_content_type":"text/x-python","patch_set":6,"id":"f9e00ae4_aef522db","line":151,"range":{"start_line":151,"start_character":8,"end_line":151,"end_character":37},"updated":"2021-11-29 14:45:38.000000000","message":"This method gives the impression that is actually testing the stateless SGs whereas what it just tests is the create and update operations. Do you perhaps want to rename it (eg. test_stateless_security_group_update?)\n\nWould it make sense to create a \u0027test_stateless_security_group\u0027 method that actually blocks/allows some traffic patterns and the test verifies that traffic is allowed/dropped as expected?","commit_id":"77402d803a07898504fcb25bde285b86ab180ddf"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"9465ab3eae46e19a393c709c4a379cfca561be66","unresolved":true,"context_lines":[{"line_number":148,"context_line":"    required_extensions \u003d [\u0027security-group\u0027, \u0027stateful-security-group\u0027]"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    @decorators.idempotent_id(\u00270a6c1476-3d1a-11ec-b0ec-0800277ac3d9\u0027)"},{"line_number":151,"context_line":"    def test_stateless_security_group(self):"},{"line_number":152,"context_line":"        security_group \u003d self.create_security_group(stateful\u003dTrue)"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"        # List security groups and verify if created group is there in response"}],"source_content_type":"text/x-python","patch_set":6,"id":"eab5723e_da7a4a60","line":151,"range":{"start_line":151,"start_character":8,"end_line":151,"end_character":37},"in_reply_to":"f9e00ae4_aef522db","updated":"2021-11-29 17:56:14.000000000","message":"Renamed; the other request to add a fullstack test to validate actual traffic doesn\u0027t belong here but is welcome, yes.","commit_id":"77402d803a07898504fcb25bde285b86ab180ddf"}]}