)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"d4a0e2de46127d214b6a62a3d0e2be8242378473","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"886311c8_68352ce5","updated":"2026-10-20 14:35:03.000000000","message":"Thank you for inputs!","commit_id":"b03b8c44b2917a9c38560883b834b84b6674d92e"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"1f6caca4c47ffee48888ac5b4fb48797c1587f6a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a52fa950_9e22639f","updated":"2026-10-21 05:38:03.000000000","message":"recheck unrelated error related timeout while test_instance_boot_after_snapshot_deletion","commit_id":"09dbe16ed3f94fc3c3e816f4b3bb0b7d15ba2a49"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"e30f38f76a85bef7a50938c06d9d63ef8bc847e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"da5c0721_438e3246","updated":"2026-10-21 07:42:35.000000000","message":"recheck unrelated error test_post_server_rebuild_host_status_unknown_only","commit_id":"09dbe16ed3f94fc3c3e816f4b3bb0b7d15ba2a49"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"87aed58684616def06c9727a2481173dc0ed11b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"26509a6f_6c402bcb","updated":"2026-10-21 13:47:53.000000000","message":"recheck unrelated failure","commit_id":"09dbe16ed3f94fc3c3e816f4b3bb0b7d15ba2a49"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"91332ac93642a479d27caf2987c359acea566294","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"de2915be_9f3e492e","updated":"2026-10-21 16:04:39.000000000","message":"recheck unrelated failure in nova-next test_cross_tenant_traffic","commit_id":"09dbe16ed3f94fc3c3e816f4b3bb0b7d15ba2a49"}],"nova/image/glance.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"294d54132be34345c45eb81c36d52a2da5487b57","unresolved":true,"context_lines":[{"line_number":371,"context_line":""},{"line_number":372,"context_line":"        # By default (or if direct download has failed), use glance client call"},{"line_number":373,"context_line":"        # to fetch the image and fill image_chunks"},{"line_number":374,"context_line":"        try:"},{"line_number":375,"context_line":"            if CONF.glance.preferred_stores:"},{"line_number":376,"context_line":"                image_chunks \u003d self._client.call("},{"line_number":377,"context_line":"                    context, 2, \u0027data\u0027, args\u003d(image_id,),"},{"line_number":378,"context_line":"                    kwargs\u003d{\u0027prefer\u0027: CONF.glance.preferred_stores})"},{"line_number":379,"context_line":"            else:"},{"line_number":380,"context_line":"                image_chunks \u003d self._client.call("},{"line_number":381,"context_line":"                    context, 2, \u0027data\u0027, args\u003d(image_id,))"},{"line_number":382,"context_line":"        except Exception:"},{"line_number":383,"context_line":"            _reraise_translated_image_exception(image_id)"},{"line_number":384,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"f1299359_a0d7fab5","line":381,"range":{"start_line":374,"start_character":12,"end_line":381,"end_character":57},"updated":"2026-10-20 14:17:15.000000000","message":"```suggestion\n        try:\n            kwargs \u003d {}\n            if CONF.glance.preferred_stores:\n                kwargs[\u0027prefer\u0027] \u003d CONF.glance.preferred_stores\n            image_chunks \u003d self._client.call(\n                context, 2, \u0027data\u0027, args\u003d(image_id,), kwargs\u003dkwargs)\n```","commit_id":"b03b8c44b2917a9c38560883b834b84b6674d92e"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"d4a0e2de46127d214b6a62a3d0e2be8242378473","unresolved":false,"context_lines":[{"line_number":371,"context_line":""},{"line_number":372,"context_line":"        # By default (or if direct download has failed), use glance client call"},{"line_number":373,"context_line":"        # to fetch the image and fill image_chunks"},{"line_number":374,"context_line":"        try:"},{"line_number":375,"context_line":"            if CONF.glance.preferred_stores:"},{"line_number":376,"context_line":"                image_chunks \u003d self._client.call("},{"line_number":377,"context_line":"                    context, 2, \u0027data\u0027, args\u003d(image_id,),"},{"line_number":378,"context_line":"                    kwargs\u003d{\u0027prefer\u0027: CONF.glance.preferred_stores})"},{"line_number":379,"context_line":"            else:"},{"line_number":380,"context_line":"                image_chunks \u003d self._client.call("},{"line_number":381,"context_line":"                    context, 2, \u0027data\u0027, args\u003d(image_id,))"},{"line_number":382,"context_line":"        except Exception:"},{"line_number":383,"context_line":"            _reraise_translated_image_exception(image_id)"},{"line_number":384,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"c8d7d11b_a1764d8d","line":381,"range":{"start_line":374,"start_character":12,"end_line":381,"end_character":57},"in_reply_to":"f1299359_a0d7fab5","updated":"2026-10-20 14:35:03.000000000","message":"Acknowledged","commit_id":"b03b8c44b2917a9c38560883b834b84b6674d92e"}],"nova/tests/functional/test_glance.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"294d54132be34345c45eb81c36d52a2da5487b57","unresolved":true,"context_lines":[{"line_number":22,"context_line":"from nova import test"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class GlancePreferredStoresTest(test.NoDBTestCase):"},{"line_number":26,"context_line":"    def setUp(self):"},{"line_number":27,"context_line":"        super(GlancePreferredStoresTest, self).setUp()"},{"line_number":28,"context_line":"        self.context \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":1,"id":"d93942a8_04cf69b6","line":25,"range":{"start_line":25,"start_character":6,"end_line":25,"end_character":31},"updated":"2026-10-20 14:17:15.000000000","message":"This class only test behavior of GlanceImageServiceV2.download with underlying client fully mocked so it may not be quite benefitial to keep it in functional tests. Maybe the existing and new unit tests are just enough ?","commit_id":"b03b8c44b2917a9c38560883b834b84b6674d92e"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"21fdb1ada981bcc0b1a17be6616fd85ef7c80545","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from nova import test"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class GlancePreferredStoresTest(test.NoDBTestCase):"},{"line_number":26,"context_line":"    def setUp(self):"},{"line_number":27,"context_line":"        super(GlancePreferredStoresTest, self).setUp()"},{"line_number":28,"context_line":"        self.context \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":1,"id":"c5f57a40_b3d375bb","line":25,"range":{"start_line":25,"start_character":6,"end_line":25,"end_character":31},"in_reply_to":"b1b33e24_7e424474","updated":"2026-10-20 18:53:53.000000000","message":"Done","commit_id":"b03b8c44b2917a9c38560883b834b84b6674d92e"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"d4a0e2de46127d214b6a62a3d0e2be8242378473","unresolved":true,"context_lines":[{"line_number":22,"context_line":"from nova import test"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class GlancePreferredStoresTest(test.NoDBTestCase):"},{"line_number":26,"context_line":"    def setUp(self):"},{"line_number":27,"context_line":"        super(GlancePreferredStoresTest, self).setUp()"},{"line_number":28,"context_line":"        self.context \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":1,"id":"b1b33e24_7e424474","line":25,"range":{"start_line":25,"start_character":6,"end_line":25,"end_character":31},"in_reply_to":"d93942a8_04cf69b6","updated":"2026-10-20 14:35:03.000000000","message":"Ack, I will remove it in new PS","commit_id":"b03b8c44b2917a9c38560883b834b84b6674d92e"}]}
