)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8980c28aa19cc5183f95fc20956c5718f273ad32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"e201c227_5218bdd8","updated":"2022-02-09 10:55:09.000000000","message":"Two potentially unrelated things in here. Could you split them out if so? +2 otherwise","commit_id":"68b75f0ac176e6e5d6fef5886ff493c717c28ddd"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"88c5ee567f917024dd6cb522550b3218c2dd0b72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"2e3bf656_e8e082f9","updated":"2022-04-14 06:46:48.000000000","message":"recheck","commit_id":"3124f6f62c7138ffd681cdb08e63a9116294bda2"}],"openstack/compute/v2/limits.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8980c28aa19cc5183f95fc20956c5718f273ad32","unresolved":true,"context_lines":[{"line_number":95,"context_line":"        # TODO(mordred) We shouldn\u0027t have to subclass just to declare"},{"line_number":96,"context_line":"        # requires_id \u003d False."},{"line_number":97,"context_line":"        return super(Limits, self).fetch("},{"line_number":98,"context_line":"            session\u003dsession, requires_id\u003dFalse, error_message\u003derror_message,"},{"line_number":99,"context_line":"            base_path\u003dbase_path, skip_cache\u003dskip_cache, **params)"}],"source_content_type":"text/x-python","patch_set":9,"id":"f4de0778_b57750fe","line":98,"range":{"start_line":98,"start_character":41,"end_line":98,"end_character":46},"updated":"2022-02-09 10:55:09.000000000","message":"intentional?","commit_id":"68b75f0ac176e6e5d6fef5886ff493c717c28ddd"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"ce0c7cb961cf9070dd194688084378751c07783c","unresolved":false,"context_lines":[{"line_number":95,"context_line":"        # TODO(mordred) We shouldn\u0027t have to subclass just to declare"},{"line_number":96,"context_line":"        # requires_id \u003d False."},{"line_number":97,"context_line":"        return super(Limits, self).fetch("},{"line_number":98,"context_line":"            session\u003dsession, requires_id\u003dFalse, error_message\u003derror_message,"},{"line_number":99,"context_line":"            base_path\u003dbase_path, skip_cache\u003dskip_cache, **params)"}],"source_content_type":"text/x-python","patch_set":9,"id":"858e2051_e784902e","line":98,"range":{"start_line":98,"start_character":41,"end_line":98,"end_character":46},"in_reply_to":"f4de0778_b57750fe","updated":"2022-04-13 11:35:43.000000000","message":"Done","commit_id":"68b75f0ac176e6e5d6fef5886ff493c717c28ddd"}],"openstack/proxy.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8980c28aa19cc5183f95fc20956c5718f273ad32","unresolved":false,"context_lines":[{"line_number":9,"context_line":"# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":10,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":"import functools"},{"line_number":13,"context_line":"import urllib"},{"line_number":14,"context_line":"from urllib.parse import urlparse"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"6514b99b_7e5d15d9","line":12,"updated":"2022-02-09 10:55:09.000000000","message":"nit: missing newline before this","commit_id":"68b75f0ac176e6e5d6fef5886ff493c717c28ddd"}],"openstack/tests/unit/cloud/test_caching.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8980c28aa19cc5183f95fc20956c5718f273ad32","unresolved":true,"context_lines":[{"line_number":541,"context_line":"            dict(method\u003d\u0027GET\u0027,"},{"line_number":542,"context_line":"                 uri\u003dself.get_mock_url("},{"line_number":543,"context_line":"                     \u0027network\u0027, \u0027public\u0027, append\u003d[\u0027v2.0\u0027, \u0027ports\u0027],"},{"line_number":544,"context_line":"                     qs_elements\u003d[\u0027status\u003dDOWN\u0027]),"},{"line_number":545,"context_line":"                 json\u003d{\u0027ports\u0027: ["},{"line_number":546,"context_line":"                     down_port,"},{"line_number":547,"context_line":"                     active_port,"}],"source_content_type":"text/x-python","patch_set":9,"id":"2e6a40e4_aed28c50","line":544,"range":{"start_line":544,"start_character":21,"end_line":544,"end_character":48},"updated":"2022-02-09 10:55:09.000000000","message":"intentional? This should probably be a separate precursor change if so (this change is so big as-is)","commit_id":"68b75f0ac176e6e5d6fef5886ff493c717c28ddd"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"ce0c7cb961cf9070dd194688084378751c07783c","unresolved":true,"context_lines":[{"line_number":541,"context_line":"            dict(method\u003d\u0027GET\u0027,"},{"line_number":542,"context_line":"                 uri\u003dself.get_mock_url("},{"line_number":543,"context_line":"                     \u0027network\u0027, \u0027public\u0027, append\u003d[\u0027v2.0\u0027, \u0027ports\u0027],"},{"line_number":544,"context_line":"                     qs_elements\u003d[\u0027status\u003dDOWN\u0027]),"},{"line_number":545,"context_line":"                 json\u003d{\u0027ports\u0027: ["},{"line_number":546,"context_line":"                     down_port,"},{"line_number":547,"context_line":"                     active_port,"}],"source_content_type":"text/x-python","patch_set":9,"id":"820e1208_c90f145f","line":544,"range":{"start_line":544,"start_character":21,"end_line":544,"end_character":48},"in_reply_to":"2e6a40e4_aed28c50","updated":"2022-04-13 11:35:43.000000000","message":"sadly impossible. This is now appearing here precisely as comment above mentions - due to the changed caching logic we now need to check params","commit_id":"68b75f0ac176e6e5d6fef5886ff493c717c28ddd"}],"openstack/tests/unit/test_proxy.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8980c28aa19cc5183f95fc20956c5718f273ad32","unresolved":false,"context_lines":[{"line_number":9,"context_line":"# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":10,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":"import copy"},{"line_number":13,"context_line":"import queue"},{"line_number":14,"context_line":"from unittest import mock"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"03b8b2c7_90390607","line":12,"updated":"2022-02-09 10:55:09.000000000","message":"nit: missing newline","commit_id":"68b75f0ac176e6e5d6fef5886ff493c717c28ddd"}]}