)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"363385768b16c9041a154e16afe16dd5e30dbe0d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"907f2dcf_71ea69f8","updated":"2026-08-05 10:16:04.000000000","message":"General change is fine. Just got some issues with the naming","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"8584b93d2979df824b40c201e0d8cf6ea82149e3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"998d146d_925c2453","updated":"2026-07-17 07:38:37.000000000","message":"Looks good ! Tested \u0026 works as expected. Thank you !","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"71ef794404abcdaaaa54d0687a90d4f902d58367","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cdf7b307_4539a7e9","updated":"2026-08-05 11:56:06.000000000","message":"Thank you for inputs Stephenfin!!","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"ae63064cecdc03ad73dd0f58fc6cb4d226305eda","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7b036245_accab9bd","updated":"2026-07-17 07:38:50.000000000","message":"recheck","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"c72f159cd3609df86987839fa1345c191909250c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"daf87b81_44ad7f11","updated":"2026-07-15 08:24:47.000000000","message":"recheck unrelated failure","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"248663c06dd1f5f9c603fc0caf8ece2a900ae4ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8997c1fa_15fa2ce8","updated":"2026-08-10 15:18:29.000000000","message":"Sorry, one further question inline","commit_id":"2f8e9508db4ee372196b8e9b4631571b19d98005"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"85cf4f05c772bc7d402ddf408abfe521929ac677","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"68cb80e0_0dd93de0","updated":"2026-08-11 11:38:49.000000000","message":"Thank you for review!!","commit_id":"2f8e9508db4ee372196b8e9b4631571b19d98005"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8c7ff13f7ac9a3a2de3dc222a020b8b0ea6e011b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"146df16e_21244854","updated":"2026-08-25 16:35:23.000000000","message":"recheck","commit_id":"19466104fb8079e6d4a854b8a48c87f4ebee21e6"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"fce7743ce23ea80430f2fab43af2063432de56dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"cb6b97d6_16711d4f","updated":"2026-08-25 18:55:24.000000000","message":"recheck","commit_id":"19466104fb8079e6d4a854b8a48c87f4ebee21e6"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5e4ae8717e70cd276c727c8fcc1469a381dd4ec6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"95ce3621_60bede38","updated":"2026-08-25 11:23:43.000000000","message":"recheck CI result looks like a flake","commit_id":"19466104fb8079e6d4a854b8a48c87f4ebee21e6"}],"openstack/image/v2/_proxy.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"363385768b16c9041a154e16afe16dd5e30dbe0d","unresolved":true,"context_lines":[{"line_number":127,"context_line":"        cache \u003d self._get_resource(_cache.Cache, None)"},{"line_number":128,"context_line":"        cache.clear(self, target)"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    def list_cached_nodes("},{"line_number":131,"context_line":"        self,"},{"line_number":132,"context_line":"        image: str | _image.Image,"},{"line_number":133,"context_line":"    ) -\u003e list[str]:"}],"source_content_type":"text/x-python","patch_set":2,"id":"783ea82c_6e8c7bfe","line":130,"updated":"2026-08-05 10:16:04.000000000","message":"We don\u0027t use `list_` in proxy methods. How about e.g. `cached_image_nodes`?","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"71ef794404abcdaaaa54d0687a90d4f902d58367","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        cache \u003d self._get_resource(_cache.Cache, None)"},{"line_number":128,"context_line":"        cache.clear(self, target)"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    def list_cached_nodes("},{"line_number":131,"context_line":"        self,"},{"line_number":132,"context_line":"        image: str | _image.Image,"},{"line_number":133,"context_line":"    ) -\u003e list[str]:"}],"source_content_type":"text/x-python","patch_set":2,"id":"b142b28c_4c5e470b","line":130,"in_reply_to":"783ea82c_6e8c7bfe","updated":"2026-08-05 11:56:06.000000000","message":"Acknowledged","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"}],"openstack/image/v2/cache.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"363385768b16c9041a154e16afe16dd5e30dbe0d","unresolved":true,"context_lines":[{"line_number":88,"context_line":"        exceptions.raise_from_response(response)"},{"line_number":89,"context_line":"        return response"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def list_cached_nodes("},{"line_number":92,"context_line":"        self,"},{"line_number":93,"context_line":"        session: adapter.Adapter,"},{"line_number":94,"context_line":"        image: str | resource.Resource,"}],"source_content_type":"text/x-python","patch_set":2,"id":"2915c6ca_964b4c71","line":91,"updated":"2026-08-05 10:16:04.000000000","message":"Can we call this `image_nodes`? The `cache` bit is redundant given the class name","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"71ef794404abcdaaaa54d0687a90d4f902d58367","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        exceptions.raise_from_response(response)"},{"line_number":89,"context_line":"        return response"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def list_cached_nodes("},{"line_number":92,"context_line":"        self,"},{"line_number":93,"context_line":"        session: adapter.Adapter,"},{"line_number":94,"context_line":"        image: str | resource.Resource,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a1a9032_e8d51f78","line":91,"in_reply_to":"2915c6ca_964b4c71","updated":"2026-08-05 11:56:06.000000000","message":"Acknowledged","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"363385768b16c9041a154e16afe16dd5e30dbe0d","unresolved":true,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        response \u003d session.get(url, microversion\u003dmicroversion)"},{"line_number":110,"context_line":"        exceptions.raise_from_response(response)"},{"line_number":111,"context_line":"        return response.json()  # type: ignore[no-any-return]"}],"source_content_type":"text/x-python","patch_set":2,"id":"c7908bd5_2913e622","line":111,"updated":"2026-08-05 10:16:04.000000000","message":"You can/should fix this with a `cast` if you\u0027re confident it\u0027s a list of strings","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"71ef794404abcdaaaa54d0687a90d4f902d58367","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        response \u003d session.get(url, microversion\u003dmicroversion)"},{"line_number":110,"context_line":"        exceptions.raise_from_response(response)"},{"line_number":111,"context_line":"        return response.json()  # type: ignore[no-any-return]"}],"source_content_type":"text/x-python","patch_set":2,"id":"b8ece94b_51adbf05","line":111,"in_reply_to":"c7908bd5_2913e622","updated":"2026-08-05 11:56:06.000000000","message":"Acknowledged","commit_id":"c0f97ad09aabf427f63a184da10bbfbb0b3a9448"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"248663c06dd1f5f9c603fc0caf8ece2a900ae4ba","unresolved":true,"context_lines":[{"line_number":90,"context_line":"        exceptions.raise_from_response(response)"},{"line_number":91,"context_line":"        return response"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    def image_nodes("},{"line_number":94,"context_line":"        self,"},{"line_number":95,"context_line":"        session: adapter.Adapter,"},{"line_number":96,"context_line":"        image: str | resource.Resource,"}],"source_content_type":"text/x-python","patch_set":3,"id":"8035126e_0782288e","line":93,"updated":"2026-08-10 15:18:29.000000000","message":"Should this be a `classmethod`? It doesn\u0027t use any attribute of the parent resource and `_get_microversion` is a classmethod so it can be called. Also, should it be a  method of the `CachedImage` class above?","commit_id":"2f8e9508db4ee372196b8e9b4631571b19d98005"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"85cf4f05c772bc7d402ddf408abfe521929ac677","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        exceptions.raise_from_response(response)"},{"line_number":91,"context_line":"        return response"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    def image_nodes("},{"line_number":94,"context_line":"        self,"},{"line_number":95,"context_line":"        session: adapter.Adapter,"},{"line_number":96,"context_line":"        image: str | resource.Resource,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff958155_f1fe70b2","line":93,"in_reply_to":"8035126e_0782288e","updated":"2026-08-11 11:38:49.000000000","message":"Done","commit_id":"2f8e9508db4ee372196b8e9b4631571b19d98005"}]}