)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"2ff382aec2fbded979de1c7414ec2c91577402ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"19246d9b_99d1b845","updated":"2024-09-03 14:00:09.000000000","message":"I love it.","commit_id":"ea5ae117f11f5960b694b411825cef5238455126"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"24bb1d78606c0634429a3ab45ed9ce26ad607a6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"702609a9_de9733f6","updated":"2024-09-09 12:33:23.000000000","message":"Is this patch supposed to be on top of `api:` patch [1] where the version related changes are made ? Because I can see the same set of version related functional tests are failing for this `tests:` patch as well as for `conf:` patch [2], \n\n[1]: https://review.opendev.org/c/openstack/glance/+/922129/3 \n[2]: https://review.opendev.org/c/openstack/glance/+/922128/3","commit_id":"ea5ae117f11f5960b694b411825cef5238455126"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e1535700936b45f9c3b26ce04637ae61dc0f7288","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bc9f57ab_9f87532b","in_reply_to":"702609a9_de9733f6","updated":"2024-09-17 11:19:21.000000000","message":"Nah, I didn\u0027t realise that the `get_versions_list` fake was used in the functional tests also. Updated now.","commit_id":"ea5ae117f11f5960b694b411825cef5238455126"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"8d3eabac87942949709d3030894bd6bc40f09c35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fee05974_72ead501","in_reply_to":"bc9f57ab_9f87532b","updated":"2024-09-25 06:26:31.000000000","message":"ack, Thanks!","commit_id":"ea5ae117f11f5960b694b411825cef5238455126"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"8d3eabac87942949709d3030894bd6bc40f09c35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dc88ceb1_83a3eec4","updated":"2024-09-25 06:26:31.000000000","message":"Thank you !","commit_id":"c3af590baa9ded93dacff7f8934e6c8f1b85bb68"}],"glance/tests/unit/test_versions.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"dedb486521edb1e8da0d9aa2f809294552ea21ed","unresolved":true,"context_lines":[{"line_number":31,"context_line":"                      enabled_cache\u003dFalse):"},{"line_number":32,"context_line":"    image_versions \u003d ["},{"line_number":33,"context_line":"        {"},{"line_number":34,"context_line":"            \u0027id\u0027: \u0027v2.17\u0027,"},{"line_number":35,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":36,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":37,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":38,"context_line":"        },"},{"line_number":39,"context_line":"    ]"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ffc38526_39795465","line":37,"range":{"start_line":34,"start_character":0,"end_line":37,"end_character":48},"updated":"2024-09-09 09:16:55.000000000","message":"AFAIK 2.17 is not yet implemented at this point and this should be added later.","commit_id":"ea5ae117f11f5960b694b411825cef5238455126"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"8d3eabac87942949709d3030894bd6bc40f09c35","unresolved":false,"context_lines":[{"line_number":31,"context_line":"                      enabled_cache\u003dFalse):"},{"line_number":32,"context_line":"    image_versions \u003d ["},{"line_number":33,"context_line":"        {"},{"line_number":34,"context_line":"            \u0027id\u0027: \u0027v2.17\u0027,"},{"line_number":35,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":36,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":37,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":38,"context_line":"        },"},{"line_number":39,"context_line":"    ]"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"11de94e3_5d6420b9","line":37,"range":{"start_line":34,"start_character":0,"end_line":37,"end_character":48},"in_reply_to":"c326466e_0ca725dd","updated":"2024-09-25 06:26:31.000000000","message":"yes, 2.17 is bumped during new location api implementation.","commit_id":"ea5ae117f11f5960b694b411825cef5238455126"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e1535700936b45f9c3b26ce04637ae61dc0f7288","unresolved":false,"context_lines":[{"line_number":31,"context_line":"                      enabled_cache\u003dFalse):"},{"line_number":32,"context_line":"    image_versions \u003d ["},{"line_number":33,"context_line":"        {"},{"line_number":34,"context_line":"            \u0027id\u0027: \u0027v2.17\u0027,"},{"line_number":35,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":36,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":37,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":38,"context_line":"        },"},{"line_number":39,"context_line":"    ]"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"c326466e_0ca725dd","line":37,"range":{"start_line":34,"start_character":0,"end_line":37,"end_character":48},"in_reply_to":"ffc38526_39795465","updated":"2024-09-17 11:19:21.000000000","message":"But it was already present in the test, no? See [line 50 in the base patch](https://review.opendev.org/c/openstack/glance/+/926646/2/glance/tests/unit/test_versions.py#b50).","commit_id":"ea5ae117f11f5960b694b411825cef5238455126"}]}