)]}'
{"tempest/scenario/manager.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"fb44ef5f3cdb21a1cae2ab0205c2da96fa46fab7","unresolved":false,"context_lines":[{"line_number":1054,"context_line":"        if secgroup is None:"},{"line_number":1055,"context_line":"            # Get default secgroup for tenant_id"},{"line_number":1056,"context_line":"            default_secgroups \u003d security_groups_client.list_security_groups("},{"line_number":1057,"context_line":"                name\u003d\u0027default\u0027, tenant_id\u003dtenant_id)[\u0027security_groups\u0027]"},{"line_number":1058,"context_line":"            msg \u003d \"No default security group for tenant %s.\" % (tenant_id)"},{"line_number":1059,"context_line":"            self.assertNotEmpty(default_secgroups, msg)"},{"line_number":1060,"context_line":"            secgroup \u003d default_secgroups[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_b705d2c4","line":1057,"updated":"2017-09-25 14:04:06.000000000","message":"hmm, this is slightly different from the original code. Original code seems to filter by its logic. But this code filtered by server side. I think it\u0027s good to mention about it in the commit message or keep the logic.","commit_id":"e5cf32a41b280f33d7f7f6cbefcdb197b534a2e3"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"470a8b32d821ecb2ddb5a39de82cd1767a1e6b80","unresolved":false,"context_lines":[{"line_number":1054,"context_line":"        if secgroup is None:"},{"line_number":1055,"context_line":"            # Get default secgroup for tenant_id"},{"line_number":1056,"context_line":"            default_secgroups \u003d security_groups_client.list_security_groups("},{"line_number":1057,"context_line":"                name\u003d\u0027default\u0027, tenant_id\u003dtenant_id)[\u0027security_groups\u0027]"},{"line_number":1058,"context_line":"            msg \u003d \"No default security group for tenant %s.\" % (tenant_id)"},{"line_number":1059,"context_line":"            self.assertNotEmpty(default_secgroups, msg)"},{"line_number":1060,"context_line":"            secgroup \u003d default_secgroups[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_09c1222f","line":1057,"in_reply_to":"7f515b1d_b705d2c4","updated":"2017-09-25 18:00:11.000000000","message":"Nice point. I guess that would be not a matter, but it is better to mention it.","commit_id":"e5cf32a41b280f33d7f7f6cbefcdb197b534a2e3"}]}