)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"84af5402f815040dd62e081a35eb25425251e6a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"165af7d5_da8c5006","updated":"2025-12-09 07:57:50.000000000","message":"This fix needs a reno too","commit_id":"64cd82cdb62cf9218bf1941d14914d84f3828b71"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9169112710126ff9a32002a51a11a89e941ea8a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fb8736f0_c0d4564f","in_reply_to":"165af7d5_da8c5006","updated":"2025-12-09 09:29:42.000000000","message":"I don\u0027t think it needs reno really. This patch effectively don\u0027t change anything because previously we were explicitly checking \"if is_service_role\" in the \"_validate_privileges\" function. Now this check is simply moved to the Context.can_set_project_id property.","commit_id":"64cd82cdb62cf9218bf1941d14914d84f3828b71"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a37b239a76d9bf03bf1a0507ed09aa4d0f51dd3d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"74bc32a3_bc6a0688","in_reply_to":"fb8736f0_c0d4564f","updated":"2025-12-09 14:26:46.000000000","message":"Right, this is a refactor","commit_id":"64cd82cdb62cf9218bf1941d14914d84f3828b71"}],"neutron_lib/context.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"84af5402f815040dd62e081a35eb25425251e6a2","unresolved":true,"context_lines":[{"line_number":127,"context_line":"        return self.is_admin or self._has_global_access"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    @property"},{"line_number":130,"context_line":"    def can_set_project_id(self):"},{"line_number":131,"context_line":"        return ("},{"line_number":132,"context_line":"            self.is_admin or"},{"line_number":133,"context_line":"            self.is_service_role or"}],"source_content_type":"text/x-python","patch_set":1,"id":"d61a5f85_5fc64478","line":130,"updated":"2025-12-09 07:57:50.000000000","message":"Just asking: during the `is_advsvc` deprecation time, should we include it too here?","commit_id":"64cd82cdb62cf9218bf1941d14914d84f3828b71"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a37b239a76d9bf03bf1a0507ed09aa4d0f51dd3d","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        return self.is_admin or self._has_global_access"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    @property"},{"line_number":130,"context_line":"    def can_set_project_id(self):"},{"line_number":131,"context_line":"        return ("},{"line_number":132,"context_line":"            self.is_admin or"},{"line_number":133,"context_line":"            self.is_service_role or"}],"source_content_type":"text/x-python","patch_set":1,"id":"e4b71abf_f565a2b7","line":130,"in_reply_to":"d3b08fa8_a03ce5e4","updated":"2025-12-09 14:26:46.000000000","message":"Ahhhh yes:\n```\ndef is_advsvc():\n  return self.is_service_role\n```\nYou are right! Thanks","commit_id":"64cd82cdb62cf9218bf1941d14914d84f3828b71"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9c2377e385a94f311ed170aa41facc667a0cdef1","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        return self.is_admin or self._has_global_access"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    @property"},{"line_number":130,"context_line":"    def can_set_project_id(self):"},{"line_number":131,"context_line":"        return ("},{"line_number":132,"context_line":"            self.is_admin or"},{"line_number":133,"context_line":"            self.is_service_role or"}],"source_content_type":"text/x-python","patch_set":1,"id":"d3b08fa8_a03ce5e4","line":130,"in_reply_to":"d61a5f85_5fc64478","updated":"2025-12-09 09:27:20.000000000","message":"we don\u0027t need to as if you look at propert `is_advsvc` it already returns what \"is_service_role\" returns.","commit_id":"64cd82cdb62cf9218bf1941d14914d84f3828b71"}]}