)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"9f87d4901f297be6b0f3ff78a2b1ef539db5e74c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d10c47ef_47884e9c","line":10,"updated":"2023-08-21 06:35:33.000000000","message":"related blueprint tag","commit_id":"7739ef46b5b44983ae63949ea57b4b0944bc2d9e"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a9dcabcd3fc19ede8b6d42e175e5f2127e7b025a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4d41d1b5_89d43a18","line":10,"in_reply_to":"d10c47ef_47884e9c","updated":"2023-08-24 11:44:22.000000000","message":"Done","commit_id":"7739ef46b5b44983ae63949ea57b4b0944bc2d9e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"4dd649e349303e37d691bf15dc55e8cb66102d2a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2adef408_1a90d4d0","updated":"2023-08-16 16:45:38.000000000","message":"I assume this is related? It wasn\u0027t failing on the previous iteration:\n\nhttps://zuul.opendev.org/t/openstack/build/08d8421f36ba416d9c62f491d7e19958/log/controller/logs/screen-n-api.txt#12041","commit_id":"5e5bc7f2c3e2e028b28b8c8b87dd7b158dd87d0d"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"318fed2c61ad19fb682f47547b71ada6ccd48304","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9da20970_a240b369","in_reply_to":"1f408954_37c82f66","updated":"2023-08-21 13:50:52.000000000","message":"I don\u0027t understand what you\u0027re saying. The two patches in nova are stacked so that we test with new glanceclient and old glance, and then new glanceclient *and* new glance. Without the dependencies, we\u0027re not testing any of the new code.","commit_id":"5e5bc7f2c3e2e028b28b8c8b87dd7b158dd87d0d"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"9e4ad3f4f91baa5f0aa9217d754b10c6f091a04b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1f408954_37c82f66","in_reply_to":"2adef408_1a90d4d0","updated":"2023-08-17 13:10:44.000000000","message":"But in the g-api logs[1] of the same, I can see the new add location api is been called, whereas it should not be called if client patch\u0027s dependency is removed,\nIs the base patch dependency causing this problem ? \n\n\n[1]: https://021e56c52303ca004b13-05f128eabb5bedc4fedd9d8e8c31f9fd.ssl.cf5.rackcdn.com/891207/2/check/nova-ceph-multistore/08d8421/controller/logs/screen-g-api.txt","commit_id":"5e5bc7f2c3e2e028b28b8c8b87dd7b158dd87d0d"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"c83af9beb4f36862f336c358b9388a2a04e592ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d26b7747_52f9cd83","in_reply_to":"46105cd1_e4a5db9f","updated":"2023-08-24 11:47:19.000000000","message":"ack, I think it would better to add recheck again on nova patch and see if it still fails since the location api patches are also recently updated, and the tempest test which is failing on that job is passing locally with or without loc api changes","commit_id":"5e5bc7f2c3e2e028b28b8c8b87dd7b158dd87d0d"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"260ab649df366cda2b18f4221b998099ed2c8e02","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"fe1bda4c_ae0e6e61","in_reply_to":"9da20970_a240b369","updated":"2023-08-23 14:52:20.000000000","message":"I\u0027m just doubtful about the dependency you\u0027ve set by creating stack for nova patches, I think the base patch should have glance patch dependency and the top patch should have client patch, bcz in the g-api logs of failed job, i can see the location import has been executed which is not expected without client patch right?\n\nAlso I have confirmed locally, the tempest test which is failing on that job is passing with and without my changes.","commit_id":"5e5bc7f2c3e2e028b28b8c8b87dd7b158dd87d0d"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"9e3a5747fb5dbc9e297951c76b3a0a92c162d243","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"46105cd1_e4a5db9f","in_reply_to":"fe1bda4c_ae0e6e61","updated":"2023-08-23 15:16:29.000000000","message":"Nova can\u0027t control what version of glance you use, but it can control what version of the client library it requires. So, we need to make sure that nova continues to work with old glance and new glance, with the version of the client we require. The first patch ensures that we can require the new client and still work with old glance. The second patch ensures we can use both new glance and new client. Your own changes here ensure that an older nova (without my changes) continues to work with the new glance changes you\u0027ve added.","commit_id":"5e5bc7f2c3e2e028b28b8c8b87dd7b158dd87d0d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d1c1a4bbe43320bb8ca818e2fc5c41b788e87299","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"35b09284_faf168d4","updated":"2023-08-28 14:48:59.000000000","message":"See comment inline.","commit_id":"d1b1902119e3b30072b423822737be76dad57414"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a04793a3e570b855b5c97dd75e477db96b599307","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"95e49832_c700d774","updated":"2023-08-28 15:26:07.000000000","message":"Thanks Brian !","commit_id":"c51d4854f66de6db7c2cb31295da3eaac7c9517f"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"f4a4c3b21f62e46cc9d176e688b3912a2268e025","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"13225bb7_26844cdc","updated":"2023-08-29 16:40:58.000000000","message":"recheck on timeout exception while reassigning ports between servers","commit_id":"a464a46382bfd2f8ce62531cb7332e243686d47e"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"151930fc870cf7814fa633d177e6373dcee61256","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"318053d0_93dfc29e","updated":"2023-08-31 10:08:08.000000000","message":"recheck timeouts","commit_id":"2a79fd654cf6b71b09c0d7c7a39b1454c76db743"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8128208399231d4378ce8cc8cf4bd7a3e95c56be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"71c43967_0dda92b1","updated":"2023-09-28 13:04:25.000000000","message":"See comment inline.","commit_id":"e1afc6c083fde527669ddc71d6c21f33084d1649"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"90c084b53157a0581a3d006be3323eeb1ab42b81","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":45,"id":"0de5237a_d9ae65f1","updated":"2024-02-28 10:30:50.000000000","message":"Looks good!","commit_id":"01fb6278c95fcabc95201f7fce5f2fd3bfedf873"}],"glance/api/versions.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"9f87d4901f297be6b0f3ff78a2b1ef539db5e74c","unresolved":true,"context_lines":[{"line_number":89,"context_line":"            ])"},{"line_number":90,"context_line":"        if CONF.enabled_backends:"},{"line_number":91,"context_line":"            version_objs.extend(["},{"line_number":92,"context_line":"                build_version_object(\u00272.17\u0027, \u0027v2\u0027, \u0027CURRENT\u0027),"},{"line_number":93,"context_line":"                build_version_object(\u00272.13\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"},{"line_number":94,"context_line":"                build_version_object(\u00272.12\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"},{"line_number":95,"context_line":"                build_version_object(\u00272.11\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"}],"source_content_type":"text/x-python","patch_set":6,"id":"4e3d4162_5e03bb8e","line":92,"range":{"start_line":92,"start_character":16,"end_line":92,"end_character":36},"updated":"2023-08-21 06:35:33.000000000","message":"Since this feature is available for both single and multi backend instead of updating the version in both if and else blocks can we just update it once outside the blocks?","commit_id":"7739ef46b5b44983ae63949ea57b4b0944bc2d9e"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a9dcabcd3fc19ede8b6d42e175e5f2127e7b025a","unresolved":false,"context_lines":[{"line_number":89,"context_line":"            ])"},{"line_number":90,"context_line":"        if CONF.enabled_backends:"},{"line_number":91,"context_line":"            version_objs.extend(["},{"line_number":92,"context_line":"                build_version_object(\u00272.17\u0027, \u0027v2\u0027, \u0027CURRENT\u0027),"},{"line_number":93,"context_line":"                build_version_object(\u00272.13\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"},{"line_number":94,"context_line":"                build_version_object(\u00272.12\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"},{"line_number":95,"context_line":"                build_version_object(\u00272.11\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"}],"source_content_type":"text/x-python","patch_set":6,"id":"a6d81628_b7fafa7e","line":92,"range":{"start_line":92,"start_character":16,"end_line":92,"end_character":36},"in_reply_to":"4e3d4162_5e03bb8e","updated":"2023-08-24 11:44:22.000000000","message":"Done","commit_id":"7739ef46b5b44983ae63949ea57b4b0944bc2d9e"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d1c1a4bbe43320bb8ca818e2fc5c41b788e87299","unresolved":true,"context_lines":[{"line_number":79,"context_line":"        version_objs \u003d []"},{"line_number":80,"context_line":"        if CONF.image_cache_dir:"},{"line_number":81,"context_line":"            version_objs.extend(["},{"line_number":82,"context_line":"                build_version_object(\u00272.16\u0027, \u0027v2\u0027, \u0027CURRENT\u0027),"},{"line_number":83,"context_line":"                build_version_object(\u00272.15\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"},{"line_number":84,"context_line":"                build_version_object(\u00272.14\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"},{"line_number":85,"context_line":"            ])"}],"source_content_type":"text/x-python","patch_set":11,"id":"84faef78_260b75ee","line":82,"range":{"start_line":82,"start_character":52,"end_line":82,"end_character":59},"updated":"2023-08-28 14:48:59.000000000","message":"Since 2.17 is being unconditionally declared the current version at line 104, this should be changed to SUPPORTED","commit_id":"d1b1902119e3b30072b423822737be76dad57414"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a04793a3e570b855b5c97dd75e477db96b599307","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        version_objs \u003d []"},{"line_number":80,"context_line":"        if CONF.image_cache_dir:"},{"line_number":81,"context_line":"            version_objs.extend(["},{"line_number":82,"context_line":"                build_version_object(\u00272.16\u0027, \u0027v2\u0027, \u0027CURRENT\u0027),"},{"line_number":83,"context_line":"                build_version_object(\u00272.15\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"},{"line_number":84,"context_line":"                build_version_object(\u00272.14\u0027, \u0027v2\u0027, \u0027SUPPORTED\u0027),"},{"line_number":85,"context_line":"            ])"}],"source_content_type":"text/x-python","patch_set":11,"id":"ce5f9d66_fa312c81","line":82,"range":{"start_line":82,"start_character":52,"end_line":82,"end_character":59},"in_reply_to":"84faef78_260b75ee","updated":"2023-08-28 15:26:07.000000000","message":"Done","commit_id":"d1b1902119e3b30072b423822737be76dad57414"}],"glance/tests/unit/test_versions.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"2f141bc7c53e6eb49561dc2931edb20ded80ef59","unresolved":true,"context_lines":[{"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":"            \u0027id\u0027: \u0027v2.17\u0027,"},{"line_number":41,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":42,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":43,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":44,"context_line":"        },"},{"line_number":45,"context_line":"        {"},{"line_number":46,"context_line":"            \u0027id\u0027: \u0027v2.9\u0027,"},{"line_number":47,"context_line":"            \u0027status\u0027: \u0027SUPPORTED\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"01254134_864d6f9f","line":44,"range":{"start_line":39,"start_character":8,"end_line":44,"end_character":10},"updated":"2023-08-10 06:11:17.000000000","message":"I think this should be moved to top","commit_id":"31af0bcebbbb342eb11df0bf0fa7f595522d2d37"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"ce60114bf0ea01e80d2985c9ae11805b8badf80c","unresolved":true,"context_lines":[{"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":"            \u0027id\u0027: \u0027v2.17\u0027,"},{"line_number":41,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":42,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":43,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":44,"context_line":"        },"},{"line_number":45,"context_line":"        {"},{"line_number":46,"context_line":"            \u0027id\u0027: \u0027v2.9\u0027,"},{"line_number":47,"context_line":"            \u0027status\u0027: \u0027SUPPORTED\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"b7729265_66eee349","line":44,"range":{"start_line":39,"start_character":8,"end_line":44,"end_character":10},"in_reply_to":"01254134_864d6f9f","updated":"2023-08-16 09:36:32.000000000","message":"This is bcz the sequence of newly bumped version is this way [1].\nwill that be ok if move 2.17 to upper else part and move 2.15 below in the index () of versions.py? \n\n[1]: https://review.opendev.org/c/openstack/glance/+/890509/5/glance/api/versions.py#88","commit_id":"31af0bcebbbb342eb11df0bf0fa7f595522d2d37"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"9c02c1558c6376015849cb552b231a36fa169f51","unresolved":false,"context_lines":[{"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":"            \u0027id\u0027: \u0027v2.17\u0027,"},{"line_number":41,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":42,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":43,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":44,"context_line":"        },"},{"line_number":45,"context_line":"        {"},{"line_number":46,"context_line":"            \u0027id\u0027: \u0027v2.9\u0027,"},{"line_number":47,"context_line":"            \u0027status\u0027: \u0027SUPPORTED\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ecf6e38e_e6d8d587","line":44,"range":{"start_line":39,"start_character":8,"end_line":44,"end_character":10},"in_reply_to":"62702422_c2d951c3","updated":"2023-10-31 09:33:22.000000000","message":"Done","commit_id":"31af0bcebbbb342eb11df0bf0fa7f595522d2d37"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"9f87d4901f297be6b0f3ff78a2b1ef539db5e74c","unresolved":true,"context_lines":[{"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":"            \u0027id\u0027: \u0027v2.17\u0027,"},{"line_number":41,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":42,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":43,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":44,"context_line":"        },"},{"line_number":45,"context_line":"        {"},{"line_number":46,"context_line":"            \u0027id\u0027: \u0027v2.9\u0027,"},{"line_number":47,"context_line":"            \u0027status\u0027: \u0027SUPPORTED\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"cb2c4037_7d0f7921","line":44,"range":{"start_line":39,"start_character":8,"end_line":44,"end_character":10},"in_reply_to":"b7729265_66eee349","updated":"2023-08-21 06:35:33.000000000","message":"I think its test data, so will it matter?","commit_id":"31af0bcebbbb342eb11df0bf0fa7f595522d2d37"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a9dcabcd3fc19ede8b6d42e175e5f2127e7b025a","unresolved":true,"context_lines":[{"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":"            \u0027id\u0027: \u0027v2.17\u0027,"},{"line_number":41,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":42,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":43,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":44,"context_line":"        },"},{"line_number":45,"context_line":"        {"},{"line_number":46,"context_line":"            \u0027id\u0027: \u0027v2.9\u0027,"},{"line_number":47,"context_line":"            \u0027status\u0027: \u0027SUPPORTED\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"62702422_c2d951c3","line":44,"range":{"start_line":39,"start_character":8,"end_line":44,"end_character":10},"in_reply_to":"cb2c4037_7d0f7921","updated":"2023-08-24 11:44:22.000000000","message":"yeah it fails otherwise.","commit_id":"31af0bcebbbb342eb11df0bf0fa7f595522d2d37"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8128208399231d4378ce8cc8cf4bd7a3e95c56be","unresolved":true,"context_lines":[{"line_number":376,"context_line":"        request \u003d webob.Request.blank(\u0027/v2.16/images\u0027)"},{"line_number":377,"context_line":"        self.middleware.process_request(request)"},{"line_number":378,"context_line":"        self.assertEqual(\u0027/v2/images\u0027, request.path_info)"},{"line_number":379,"context_line":""},{"line_number":380,"context_line":"    # version 2.17 does not exist"},{"line_number":381,"context_line":"    def test_request_url_v2_17_default_unsupported(self):"},{"line_number":382,"context_line":"        request \u003d webob.Request.blank(\u0027/v2.17/images\u0027)"},{"line_number":383,"context_line":"        resp \u003d self.middleware.process_request(request)"}],"source_content_type":"text/x-python","patch_set":21,"id":"50de6276_82e838e5","side":"PARENT","line":380,"range":{"start_line":379,"start_character":0,"end_line":380,"end_character":33},"updated":"2023-09-28 13:04:25.000000000","message":"I think you should keep tests to make sure the next version, which would be 2.18, does not return a path.  I\u0027m not sure you need to have both tests, but on the other hand, because we have logic in the versions controller that changes the available versions based on some config options, we should probably make sure we get the response we expect under all circumstances.","commit_id":"fa8e5358620560d59ce9c5642907a4d1e7dd34d4"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"5f7ac3854f8de89411f2daf28e9f8af58714d585","unresolved":false,"context_lines":[{"line_number":376,"context_line":"        request \u003d webob.Request.blank(\u0027/v2.16/images\u0027)"},{"line_number":377,"context_line":"        self.middleware.process_request(request)"},{"line_number":378,"context_line":"        self.assertEqual(\u0027/v2/images\u0027, request.path_info)"},{"line_number":379,"context_line":""},{"line_number":380,"context_line":"    # version 2.17 does not exist"},{"line_number":381,"context_line":"    def test_request_url_v2_17_default_unsupported(self):"},{"line_number":382,"context_line":"        request \u003d webob.Request.blank(\u0027/v2.17/images\u0027)"},{"line_number":383,"context_line":"        resp \u003d self.middleware.process_request(request)"}],"source_content_type":"text/x-python","patch_set":21,"id":"93ef4b8c_58d2d0b2","side":"PARENT","line":380,"range":{"start_line":379,"start_character":0,"end_line":380,"end_character":33},"in_reply_to":"50de6276_82e838e5","updated":"2023-10-19 07:07:24.000000000","message":"Done","commit_id":"fa8e5358620560d59ce9c5642907a4d1e7dd34d4"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"c51fd0e59cbda9cb06d936df4a38ec7830769451","unresolved":true,"context_lines":[{"line_number":47,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":48,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":49,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":50,"context_line":"        },"},{"line_number":51,"context_line":"        {"},{"line_number":52,"context_line":"            \u0027id\u0027: \u0027v2.7\u0027,"},{"line_number":53,"context_line":"            \u0027status\u0027: \u0027SUPPORTED\u0027,"}],"source_content_type":"text/x-python","patch_set":34,"id":"7e8aa3a7_a112c3b6","line":50,"updated":"2024-02-13 02:39:21.000000000","message":"So why is this not the first entry in the list? All other entries seem to be sorted.\n\nIf there is a reason for \"v2.17\" to be the third entry, I\u0027d like to see a comment. And where will we add \"v2.18\"?","commit_id":"0b7f8f8ad38ebad74d1deb0c04ddb696e67b7fb4"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"e32a324db45fc87d4f36129a7256a6f23a627f94","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":48,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":49,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":50,"context_line":"        },"},{"line_number":51,"context_line":"        {"},{"line_number":52,"context_line":"            \u0027id\u0027: \u0027v2.7\u0027,"},{"line_number":53,"context_line":"            \u0027status\u0027: \u0027SUPPORTED\u0027,"}],"source_content_type":"text/x-python","patch_set":34,"id":"72dd7616_e0c32234","line":50,"in_reply_to":"70513684_f6907e79","updated":"2024-04-18 12:06:48.000000000","message":"Done","commit_id":"0b7f8f8ad38ebad74d1deb0c04ddb696e67b7fb4"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"dc6f49708a0c1cd5d8d397eca16e82b07f2dcd19","unresolved":true,"context_lines":[{"line_number":47,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":48,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":49,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":50,"context_line":"        },"},{"line_number":51,"context_line":"        {"},{"line_number":52,"context_line":"            \u0027id\u0027: \u0027v2.7\u0027,"},{"line_number":53,"context_line":"            \u0027status\u0027: \u0027SUPPORTED\u0027,"}],"source_content_type":"text/x-python","patch_set":34,"id":"f270544d_2a4ccb73","line":50,"in_reply_to":"7e8aa3a7_a112c3b6","updated":"2024-02-15 09:09:39.000000000","message":"Please see: https://review.opendev.org/c/openstack/glance/+/890509/3..34/glance/tests/unit/test_versions.py","commit_id":"0b7f8f8ad38ebad74d1deb0c04ddb696e67b7fb4"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"311dc9d2aecb60fa9c87bdd14f3be4cf6e9d267c","unresolved":true,"context_lines":[{"line_number":47,"context_line":"            \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":48,"context_line":"            \u0027links\u0027: [{\u0027rel\u0027: \u0027self\u0027,"},{"line_number":49,"context_line":"                       \u0027href\u0027: \u0027%s/v2/\u0027 % url}],"},{"line_number":50,"context_line":"        },"},{"line_number":51,"context_line":"        {"},{"line_number":52,"context_line":"            \u0027id\u0027: \u0027v2.7\u0027,"},{"line_number":53,"context_line":"            \u0027status\u0027: \u0027SUPPORTED\u0027,"}],"source_content_type":"text/x-python","patch_set":34,"id":"70513684_f6907e79","line":50,"in_reply_to":"f270544d_2a4ccb73","updated":"2024-02-20 17:01:49.000000000","message":"This feels a bit like voodoo, but ok :-p","commit_id":"0b7f8f8ad38ebad74d1deb0c04ddb696e67b7fb4"}]}