)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e6ed9b3a827cf353ec67f3deb406fcd8bcfc8cd0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"fa430834_48f0d267","updated":"2025-10-06 08:14:01.000000000","message":"-1 only for the missing `tenant_id`","commit_id":"2dfb5ba2148ea6088422f1dfa1568f4b1fca0b55"}],"neutron/policy.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e6ed9b3a827cf353ec67f3deb406fcd8bcfc8cd0","unresolved":true,"context_lines":[{"line_number":268,"context_line":"    \"\"\""},{"line_number":269,"context_line":"    def __init__(self, kind, match):"},{"line_number":270,"context_line":"        if kind \u003d\u003d \u0027tenant_id\u0027:"},{"line_number":271,"context_line":"            LOG.warning("},{"line_number":272,"context_line":"                \"Using \u0027tenant_id\u0027 in the API policy rules is deprecated \""},{"line_number":273,"context_line":"                \"since 2026.1 release and will be removed in \""},{"line_number":274,"context_line":"                \"the 2027.1. Please use \u0027project_id\u0027 instead.\")"}],"source_content_type":"text/x-python","patch_set":7,"id":"2dbcaf28_b0fa8213","line":271,"range":{"start_line":271,"start_character":12,"end_line":271,"end_character":24},"updated":"2025-10-06 08:14:01.000000000","message":"just a comment: I think this will print a warning message any time the policy is executed. Of course the correct way to fix that is to fix the policy file. But maybe we can change this to a ``warnings.warn`` call to be printed once only.","commit_id":"2dfb5ba2148ea6088422f1dfa1568f4b1fca0b55"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e01707f96871a378b640c871a4b3a8c0b32aed14","unresolved":false,"context_lines":[{"line_number":268,"context_line":"    \"\"\""},{"line_number":269,"context_line":"    def __init__(self, kind, match):"},{"line_number":270,"context_line":"        if kind \u003d\u003d \u0027tenant_id\u0027:"},{"line_number":271,"context_line":"            LOG.warning("},{"line_number":272,"context_line":"                \"Using \u0027tenant_id\u0027 in the API policy rules is deprecated \""},{"line_number":273,"context_line":"                \"since 2026.1 release and will be removed in \""},{"line_number":274,"context_line":"                \"the 2027.1. Please use \u0027project_id\u0027 instead.\")"}],"source_content_type":"text/x-python","patch_set":7,"id":"24dbab66_6860c655","line":271,"range":{"start_line":271,"start_character":12,"end_line":271,"end_character":24},"in_reply_to":"07da2b32_20c94768","updated":"2025-10-14 12:53:44.000000000","message":"+1","commit_id":"2dfb5ba2148ea6088422f1dfa1568f4b1fca0b55"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a1da1049c518fd54682553d6292c2310c156c692","unresolved":true,"context_lines":[{"line_number":268,"context_line":"    \"\"\""},{"line_number":269,"context_line":"    def __init__(self, kind, match):"},{"line_number":270,"context_line":"        if kind \u003d\u003d \u0027tenant_id\u0027:"},{"line_number":271,"context_line":"            LOG.warning("},{"line_number":272,"context_line":"                \"Using \u0027tenant_id\u0027 in the API policy rules is deprecated \""},{"line_number":273,"context_line":"                \"since 2026.1 release and will be removed in \""},{"line_number":274,"context_line":"                \"the 2027.1. Please use \u0027project_id\u0027 instead.\")"}],"source_content_type":"text/x-python","patch_set":7,"id":"07da2b32_20c94768","line":271,"range":{"start_line":271,"start_character":12,"end_line":271,"end_character":24},"in_reply_to":"2dbcaf28_b0fa8213","updated":"2025-10-14 12:52:07.000000000","message":"I think it is good to keep it in logs always. It will be there only if someone has custom policies defined and having only one warning in log may easily be missed by the operator. \"Spamming\" logs with those warnings will make it more visible IMO","commit_id":"2dfb5ba2148ea6088422f1dfa1568f4b1fca0b55"}],"neutron/tests/unit/conf/policies/test_subnet.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e6ed9b3a827cf353ec67f3deb406fcd8bcfc8cd0","unresolved":true,"context_lines":[{"line_number":75,"context_line":"        # This is the case where the network belongs to the project but not"},{"line_number":76,"context_line":"        # the subnet."},{"line_number":77,"context_line":"        self.alt_target_own_net \u003d {"},{"line_number":78,"context_line":"            \u0027tenant_id\u0027: self.alt_project_id,"},{"line_number":79,"context_line":"            \u0027project_id\u0027: self.alt_project_id,"},{"line_number":80,"context_line":"            \u0027network_id\u0027: self.network[\u0027id\u0027],"},{"line_number":81,"context_line":"            \u0027ext_parent_network_id\u0027: self.network[\u0027id\u0027]}"}],"source_content_type":"text/x-python","patch_set":7,"id":"647c66c4_1983e372","line":78,"range":{"start_line":78,"start_character":13,"end_line":78,"end_character":45},"updated":"2025-10-06 08:14:01.000000000","message":"Shouldn\u0027t we remove this one?","commit_id":"2dfb5ba2148ea6088422f1dfa1568f4b1fca0b55"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"7fed2a19962e6815a444ed9bd5ba7b05f0f212b9","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        # This is the case where the network belongs to the project but not"},{"line_number":76,"context_line":"        # the subnet."},{"line_number":77,"context_line":"        self.alt_target_own_net \u003d {"},{"line_number":78,"context_line":"            \u0027tenant_id\u0027: self.alt_project_id,"},{"line_number":79,"context_line":"            \u0027project_id\u0027: self.alt_project_id,"},{"line_number":80,"context_line":"            \u0027network_id\u0027: self.network[\u0027id\u0027],"},{"line_number":81,"context_line":"            \u0027ext_parent_network_id\u0027: self.network[\u0027id\u0027]}"}],"source_content_type":"text/x-python","patch_set":7,"id":"cc4f2438_8fc3515f","line":78,"range":{"start_line":78,"start_character":13,"end_line":78,"end_character":45},"in_reply_to":"63cf213d_921fcc24","updated":"2025-10-14 12:52:37.000000000","message":"it is done now","commit_id":"2dfb5ba2148ea6088422f1dfa1568f4b1fca0b55"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a1da1049c518fd54682553d6292c2310c156c692","unresolved":true,"context_lines":[{"line_number":75,"context_line":"        # This is the case where the network belongs to the project but not"},{"line_number":76,"context_line":"        # the subnet."},{"line_number":77,"context_line":"        self.alt_target_own_net \u003d {"},{"line_number":78,"context_line":"            \u0027tenant_id\u0027: self.alt_project_id,"},{"line_number":79,"context_line":"            \u0027project_id\u0027: self.alt_project_id,"},{"line_number":80,"context_line":"            \u0027network_id\u0027: self.network[\u0027id\u0027],"},{"line_number":81,"context_line":"            \u0027ext_parent_network_id\u0027: self.network[\u0027id\u0027]}"}],"source_content_type":"text/x-python","patch_set":7,"id":"63cf213d_921fcc24","line":78,"range":{"start_line":78,"start_character":13,"end_line":78,"end_character":45},"in_reply_to":"647c66c4_1983e372","updated":"2025-10-14 12:52:07.000000000","message":"right, I somehow forgot about it. Let me update that","commit_id":"2dfb5ba2148ea6088422f1dfa1568f4b1fca0b55"}]}