)]}'
{"neutron_tempest_plugin/api/test_security_groups.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9e64d551383eb26960a02ca4a50df6d498a05590","unresolved":true,"context_lines":[{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    @decorators.idempotent_id(\u00271fff0d57-bb6c-4528-9c1d-2326dce1c087\u0027)"},{"line_number":168,"context_line":"    def test_show_security_group_contains_all_rules(self):"},{"line_number":169,"context_line":"        self._test_show_security_group_contains_all_rules()"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    @decorators.idempotent_id(\u0027b5923b1a-4d33-44e1-af25-088dcb55b02b\u0027)"},{"line_number":172,"context_line":"    def test_list_security_group_rules_contains_all_rules(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"822682aa_20358364","side":"PARENT","line":169,"updated":"2023-11-21 21:18:04.000000000","message":"I don\u0027t see both of these tests in tempest, or any stateless SG tests.","commit_id":"370f71e9877fad13883413225bf25051c0c9c391"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d789b3f363da106286924151bb09a3eefe363fa8","unresolved":true,"context_lines":[{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    @decorators.idempotent_id(\u00271fff0d57-bb6c-4528-9c1d-2326dce1c087\u0027)"},{"line_number":168,"context_line":"    def test_show_security_group_contains_all_rules(self):"},{"line_number":169,"context_line":"        self._test_show_security_group_contains_all_rules()"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    @decorators.idempotent_id(\u0027b5923b1a-4d33-44e1-af25-088dcb55b02b\u0027)"},{"line_number":172,"context_line":"    def test_list_security_group_rules_contains_all_rules(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"e9e7cef6_fa1d121a","side":"PARENT","line":169,"in_reply_to":"0333ac70_539fc9e0","updated":"2023-12-04 15:12:14.000000000","message":"I am more worried about the stateless part, I don\u0027t see \u0027stateless\u0027 or \u0027stateful\u0027 in the tempest tests at tempest/api/network/test_security_groups.py","commit_id":"370f71e9877fad13883413225bf25051c0c9c391"},{"author":{"_account_id":35229,"name":"Jakub Skunda","display_name":"jskunda","email":"jskunda@redhat.com","username":"jskunda"},"change_message_id":"9c7f79f9d10440367f35762ef1366347a58c8ba5","unresolved":true,"context_lines":[{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    @decorators.idempotent_id(\u00271fff0d57-bb6c-4528-9c1d-2326dce1c087\u0027)"},{"line_number":168,"context_line":"    def test_show_security_group_contains_all_rules(self):"},{"line_number":169,"context_line":"        self._test_show_security_group_contains_all_rules()"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    @decorators.idempotent_id(\u0027b5923b1a-4d33-44e1-af25-088dcb55b02b\u0027)"},{"line_number":172,"context_line":"    def test_list_security_group_rules_contains_all_rules(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"0333ac70_539fc9e0","side":"PARENT","line":169,"in_reply_to":"822682aa_20358364","updated":"2023-12-04 12:47:37.000000000","message":"Test test_show_security_group_contains_all_rules is not implemented, i probably accidentally delete it too, but if I understand that correctly test_create_list_update_show_delete_security_group should have same functionality as test with same name in tempest, or am I wrong?","commit_id":"370f71e9877fad13883413225bf25051c0c9c391"},{"author":{"_account_id":35229,"name":"Jakub Skunda","display_name":"jskunda","email":"jskunda@redhat.com","username":"jskunda"},"change_message_id":"f921144274986a7fdd5c31e8b0b38a594933d4d4","unresolved":true,"context_lines":[{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    @decorators.idempotent_id(\u00271fff0d57-bb6c-4528-9c1d-2326dce1c087\u0027)"},{"line_number":168,"context_line":"    def test_show_security_group_contains_all_rules(self):"},{"line_number":169,"context_line":"        self._test_show_security_group_contains_all_rules()"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    @decorators.idempotent_id(\u0027b5923b1a-4d33-44e1-af25-088dcb55b02b\u0027)"},{"line_number":172,"context_line":"    def test_list_security_group_rules_contains_all_rules(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"c0e69787_737d9430","side":"PARENT","line":169,"in_reply_to":"e9e7cef6_fa1d121a","updated":"2023-12-06 14:29:01.000000000","message":"Yeah I understand. Although if you follow the code in class StatefulSecGroupTest parameter stateless_sg is set to false. So when you create security group its created with no parameters.\nIn tempest test it is done the same way. Also rest of the code is same in both tests.","commit_id":"370f71e9877fad13883413225bf25051c0c9c391"}]}