)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2f9f65abc75049823b2841bdfd14e0209584fd54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"eeb6fa2a_cd4205f5","updated":"2022-08-26 12:37:21.000000000","message":"Good catch.  The \"General information\" section of the \"Interoperable image import\" section of the api-ref already makes it clear that the available methods can vary by cloud, so nothing to update there.\n\nThe -1 is for some suggested changes to the test you modify.","commit_id":"cc02758c85c5958b09486dded30c260cc512c1bb"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"5beb68959d5e5ac0d97d76e6eda54c2995ea24e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"50c46c93_980a5e08","updated":"2022-08-26 12:48:15.000000000","message":"Adding the RP+1 flag that doesn\u0027t seem to be sticky anymore","commit_id":"400fec0aaa671db19f3a139a0bb2ba038550e379"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"695ae321dd1944182a316687a5625dc154f7e3f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"013a5273_9aee2d08","updated":"2022-08-26 12:49:33.000000000","message":"Revision LGTM.","commit_id":"400fec0aaa671db19f3a139a0bb2ba038550e379"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"cff8e97460e501f212811995babfe232e2080160","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"300d7084_c91d6e13","updated":"2022-08-26 12:47:18.000000000","message":"Thanks Brian!","commit_id":"400fec0aaa671db19f3a139a0bb2ba038550e379"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"9e49df6a1d9ae681c2a5a3a865ebd6912408ff64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e2469913_8abb557d","updated":"2022-08-26 15:50:54.000000000","message":"recheck","commit_id":"400fec0aaa671db19f3a139a0bb2ba038550e379"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"01922f465ede6bc01bb536a7f80b14e844333551","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"27805f51_ff3c7721","updated":"2022-08-26 17:53:11.000000000","message":"OK, now it really looks OK!","commit_id":"c342c0e944126c6d257fab8910477a58bdec5da7"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"4c98065a9a80dc11214d5e8016cde845281c9c42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7f6987ad_edc01cb1","updated":"2022-08-26 17:40:33.000000000","message":"Thank you Erno!","commit_id":"c342c0e944126c6d257fab8910477a58bdec5da7"}],"glance/tests/unit/v2/test_discovery_image_import.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2f9f65abc75049823b2841bdfd14e0209584fd54","unresolved":true,"context_lines":[{"line_number":33,"context_line":"        self.assertEqual([], output[\u0027import-methods\u0027][\u0027value\u0027])"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def test_get_import_info(self):"},{"line_number":36,"context_line":"        # TODO(rosmaita): change this when import methods are"},{"line_number":37,"context_line":"        # listed in the config file"},{"line_number":38,"context_line":"        import_methods \u003d [\u0027glance-direct\u0027, \u0027web-download\u0027,"},{"line_number":39,"context_line":"                          \u0027copy-image\u0027]"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3d33dc19_39bee863","line":37,"range":{"start_line":36,"start_character":0,"end_line":37,"end_character":35},"updated":"2022-08-26 12:37:21.000000000","message":"This comment doesn\u0027t apply any more; please remove it.","commit_id":"cc02758c85c5958b09486dded30c260cc512c1bb"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"cff8e97460e501f212811995babfe232e2080160","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        self.assertEqual([], output[\u0027import-methods\u0027][\u0027value\u0027])"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def test_get_import_info(self):"},{"line_number":36,"context_line":"        # TODO(rosmaita): change this when import methods are"},{"line_number":37,"context_line":"        # listed in the config file"},{"line_number":38,"context_line":"        import_methods \u003d [\u0027glance-direct\u0027, \u0027web-download\u0027,"},{"line_number":39,"context_line":"                          \u0027copy-image\u0027]"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"b06e0728_eaa2a2bc","line":37,"range":{"start_line":36,"start_character":0,"end_line":37,"end_character":35},"in_reply_to":"3d33dc19_39bee863","updated":"2022-08-26 12:47:18.000000000","message":"Done","commit_id":"cc02758c85c5958b09486dded30c260cc512c1bb"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2f9f65abc75049823b2841bdfd14e0209584fd54","unresolved":true,"context_lines":[{"line_number":35,"context_line":"    def test_get_import_info(self):"},{"line_number":36,"context_line":"        # TODO(rosmaita): change this when import methods are"},{"line_number":37,"context_line":"        # listed in the config file"},{"line_number":38,"context_line":"        import_methods \u003d [\u0027glance-direct\u0027, \u0027web-download\u0027,"},{"line_number":39,"context_line":"                          \u0027copy-image\u0027]"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        req \u003d unit_test_utils.get_fake_request()"}],"source_content_type":"text/x-python","patch_set":1,"id":"84a45ffb_e2d782c9","line":38,"range":{"start_line":38,"start_character":8,"end_line":38,"end_character":22},"updated":"2022-08-26 12:37:21.000000000","message":"What we\u0027re testing here is to make sure that the default set of import-methods has not been changed accidentally.  I suggest renaming that var to \u0027default_import_methods\u0027, and that should remind reviewers so we don\u0027t let a change like the one you\u0027re fixing slip past us next time.","commit_id":"cc02758c85c5958b09486dded30c260cc512c1bb"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"cff8e97460e501f212811995babfe232e2080160","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    def test_get_import_info(self):"},{"line_number":36,"context_line":"        # TODO(rosmaita): change this when import methods are"},{"line_number":37,"context_line":"        # listed in the config file"},{"line_number":38,"context_line":"        import_methods \u003d [\u0027glance-direct\u0027, \u0027web-download\u0027,"},{"line_number":39,"context_line":"                          \u0027copy-image\u0027]"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        req \u003d unit_test_utils.get_fake_request()"}],"source_content_type":"text/x-python","patch_set":1,"id":"f3c37f8e_09969eeb","line":38,"range":{"start_line":38,"start_character":8,"end_line":38,"end_character":22},"in_reply_to":"84a45ffb_e2d782c9","updated":"2022-08-26 12:47:18.000000000","message":"Done","commit_id":"cc02758c85c5958b09486dded30c260cc512c1bb"}]}