)]}'
{"neutron_tempest_plugin/api/admin/test_security_groups.py":[{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"inactive contributor"},"change_message_id":"f6edf5e5f17b4f9c87bad87169053c95e44bb551","unresolved":true,"context_lines":[{"line_number":44,"context_line":"            if sg[\u0027name\u0027] \u003d\u003d \u0027default\u0027:"},{"line_number":45,"context_line":"                self.admin_client.delete_security_group(sg[\u0027id\u0027])"},{"line_number":46,"context_line":"        self.update_port(port, name\u003d\u0027update\u0027)"},{"line_number":47,"context_line":"        utils.wait_until_true(self._check_default_exists, timeout\u003d30)"}],"source_content_type":"text/x-python","patch_set":1,"id":"6b15bbe3_ff7e470d","line":47,"updated":"2021-04-12 14:25:37.000000000","message":"While this makes the error go away, I\u0027m not sure if the fact we have to wait here is not a bug in itself.\n\nThe way I remember this code is that the secgroup should be recreated already when the port is returned. Now I found this (please note the hook is BEFORE_UPDATE):\n\nhttps://opendev.org/openstack/neutron/src/branch/master/neutron/db/securitygroups_db.py#L902\n\nSo how is this possible?\nCould it be that we re-create the default secgroup, but somehow do not include it in the port update response?\nWhat do you think about fixing this from the neutron side?","commit_id":"5e7630b9ced58c80cf97a7c7a060e2cc56f2aba7"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"inactive contributor"},"change_message_id":"6577f24c0855e9e7c84314181e96ff79d102a445","unresolved":true,"context_lines":[{"line_number":44,"context_line":"            if sg[\u0027name\u0027] \u003d\u003d \u0027default\u0027:"},{"line_number":45,"context_line":"                self.admin_client.delete_security_group(sg[\u0027id\u0027])"},{"line_number":46,"context_line":"        self.update_port(port, name\u003d\u0027update\u0027)"},{"line_number":47,"context_line":"        utils.wait_until_true(self._check_default_exists, timeout\u003d30)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9e5f99ac_7bf755aa","line":47,"in_reply_to":"6b15bbe3_ff7e470d","updated":"2021-04-13 08:19:28.000000000","message":"Please see also:\nhttps://review.opendev.org/c/openstack/neutron/+/785614","commit_id":"5e7630b9ced58c80cf97a7c7a060e2cc56f2aba7"}]}