)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2cf2091e1fbc11f87ce656862f66463dc4d6e947","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"91af8a76_ecf29809","updated":"2026-06-24 14:09:34.000000000","message":"1. Open a LP bug.\n2. Add the corresponding tests","commit_id":"126d8cca9d77b542a4ff0d55e729107121ee2dc6"}],"neutron/db/securitygroups_db.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2cf2091e1fbc11f87ce656862f66463dc4d6e947","unresolved":true,"context_lines":[{"line_number":1189,"context_line":"        # without this exception such ports would get no default SG."},{"line_number":1190,"context_line":"        if (project_id !\u003d context.project_id and"},{"line_number":1191,"context_line":"                not context.is_admin and"},{"line_number":1192,"context_line":"                \u0027service\u0027 not in (context.roles or [])):"},{"line_number":1193,"context_line":"            return"},{"line_number":1194,"context_line":"        if not extensions.is_extension_supported(self, \u0027security-group\u0027):"},{"line_number":1195,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":1,"id":"cd13e808_0c5ebc4c","line":1192,"range":{"start_line":1192,"start_character":16,"end_line":1192,"end_character":56},"updated":"2026-06-24 14:09:34.000000000","message":"You should use `context.is_service_role`","commit_id":"126d8cca9d77b542a4ff0d55e729107121ee2dc6"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"f04716bbfdf38a754fb8d584654125e32e65750b","unresolved":false,"context_lines":[{"line_number":1189,"context_line":"        # without this exception such ports would get no default SG."},{"line_number":1190,"context_line":"        if (project_id !\u003d context.project_id and"},{"line_number":1191,"context_line":"                not context.is_admin and"},{"line_number":1192,"context_line":"                \u0027service\u0027 not in (context.roles or [])):"},{"line_number":1193,"context_line":"            return"},{"line_number":1194,"context_line":"        if not extensions.is_extension_supported(self, \u0027security-group\u0027):"},{"line_number":1195,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":1,"id":"dea633bd_855b02a6","line":1192,"range":{"start_line":1192,"start_character":16,"end_line":1192,"end_character":56},"in_reply_to":"3e9e4a04_818de1fd","updated":"2026-06-26 13:52:58.000000000","message":"Done","commit_id":"126d8cca9d77b542a4ff0d55e729107121ee2dc6"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"425186fb62f0dfb4b3e0a5eeeb6acb659e9cad9a","unresolved":true,"context_lines":[{"line_number":1189,"context_line":"        # without this exception such ports would get no default SG."},{"line_number":1190,"context_line":"        if (project_id !\u003d context.project_id and"},{"line_number":1191,"context_line":"                not context.is_admin and"},{"line_number":1192,"context_line":"                \u0027service\u0027 not in (context.roles or [])):"},{"line_number":1193,"context_line":"            return"},{"line_number":1194,"context_line":"        if not extensions.is_extension_supported(self, \u0027security-group\u0027):"},{"line_number":1195,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":1,"id":"3e9e4a04_818de1fd","line":1192,"range":{"start_line":1192,"start_character":16,"end_line":1192,"end_character":56},"in_reply_to":"cd13e808_0c5ebc4c","updated":"2026-06-26 13:35:08.000000000","message":"agree, will do!","commit_id":"126d8cca9d77b542a4ff0d55e729107121ee2dc6"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"833c3dd10bc4a0c6f7619ca967ac6d31c2918387","unresolved":true,"context_lines":[{"line_number":1189,"context_line":"        # without this exception such ports would get no default SG."},{"line_number":1190,"context_line":"        if (project_id !\u003d context.project_id and"},{"line_number":1191,"context_line":"                not context.is_admin and"},{"line_number":1192,"context_line":"                not context.is_service_role):"},{"line_number":1193,"context_line":"            return"},{"line_number":1194,"context_line":"        if not extensions.is_extension_supported(self, \u0027security-group\u0027):"},{"line_number":1195,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":2,"id":"db149e81_ee796893","line":1192,"updated":"2026-07-09 14:48:04.000000000","message":"This is yet another of those things which we maybe should remove from here completely and rely only on the RBAC API policies. I\u0027m ok with merging this patch now but I also reported https://bugs.launchpad.net/neutron/+bug/2160239 and will try to fix it","commit_id":"2c20413a1eb1e0066ddfbb4175abd590e535d36a"}]}
