)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"25ceb8b2dfae267fc48fa08a6327b2f017a6ed15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3511df02_d4a5f8c1","updated":"2024-10-13 02:00:15.000000000","message":"recheck the failure is not reproduced in my local. Can it be somehow a timing problem ?","commit_id":"dca8cd22df519d24954c40b109e3c22ba30f04df"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4c32a65b6bb1b7d4e1ff1ea39c9754e90f8abf36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ba5cbd4f_9ce465b7","updated":"2024-10-13 05:23:46.000000000","message":"recheck timeout","commit_id":"dca8cd22df519d24954c40b109e3c22ba30f04df"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"6d85a3191fb0ecaeceb252e9b2831829f9fa1cff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0b8f5b71_be0c3efa","updated":"2024-10-18 16:05:03.000000000","message":"I think we might also fix https://bugs.launchpad.net/glance/+bug/1480330 with this change and its followup doc patch.","commit_id":"1c42b043552da33e8a8355a2de29426d1c56c19f"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"81d50e71a242ee65094d3e464b7c76bb3c49c708","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"58262911_daf143b8","updated":"2024-10-18 16:01:06.000000000","message":"OK, I think we\u0027ll want a follow-up patch fixing doc/source/configuration/configuring.rst, right? This would also fix https://bugs.launchpad.net/glance/+bug/1323646 .","commit_id":"1c42b043552da33e8a8355a2de29426d1c56c19f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"62b949620b0f7756e581513aa9ae6acc25f55122","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b8b01c23_552c0e54","in_reply_to":"0b8f5b71_be0c3efa","updated":"2024-10-19 07:54:49.000000000","message":"Thanks for catching these. The suggestion makes sense. I also proposed the follow-up doc patch https://review.opendev.org/c/openstack/glance/+/932768","commit_id":"1c42b043552da33e8a8355a2de29426d1c56c19f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"bdc7cc772b6e61abbe49f1516a79bd224d761622","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"eb1854a5_9f6e5ea2","updated":"2024-10-21 11:47:51.000000000","message":"recheck is the failure consistently seen ?","commit_id":"a59d704d80d814f64d55eb1b9ac8d4e1525c8bb5"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"00fa38ac93407c843f46bee02effb29c894a0c51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"9a0271e9_a552015f","updated":"2025-06-30 16:33:03.000000000","message":"OK, let\u0027s maybe fix the typos in the release notes since they are user-facing 😊\nThen I\u0027m OK to merge this and also https://review.opendev.org/c/openstack/glance/+/932768","commit_id":"a1520ad7ce1169e7e09589c1758ee7738d989ece"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ffb9ed82709a57ee120a7ade95173addb69e9d5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c4573c52_235598a3","updated":"2025-06-30 16:34:37.000000000","message":"Thanks for spotting these two typos ! I\u0027ve fixed these.","commit_id":"d71ba189b9b2f5954701135e9d73216ee5d439ef"}],"glance_store/_drivers/swift/utils.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"0f909c5ee81b11c3874789d1a1654bd9de47849f","unresolved":true,"context_lines":[{"line_number":43,"context_line":"    cfg.StrOpt(\u0027swift_store_auth_version\u0027, default\u003d\u00273\u0027,"},{"line_number":44,"context_line":"               choices\u003d[\u00273\u0027],"},{"line_number":45,"context_line":"               help\u003d\u0027Version of the authentication service to use. \u0027"},{"line_number":46,"context_line":"                    \u0027Valid versions is 3.\u0027,"},{"line_number":47,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":48,"context_line":"               deprecated_reason\u003d\"\"\""},{"line_number":49,"context_line":"The option \u0027auth_version\u0027 in the Swift back-end configuration file is"}],"source_content_type":"text/x-python","patch_set":3,"id":"0e4c03aa_c64aee59","line":46,"range":{"start_line":46,"start_character":42,"end_line":46,"end_character":43},"updated":"2024-10-18 16:03:49.000000000","message":"The grammar is weird here :) And also it\u0027s confusing for users to have an option that only has a single possible valid value.\n\nMaybe we should deprecate this, and the help text should be \"The keystone version to use for authentication: the only valid version is 3. This option is kept for backward-compatibility reasons, and might be removed at any time in the future\".","commit_id":"1c42b043552da33e8a8355a2de29426d1c56c19f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"5153d43a25f71d62a21ab5db43cb26ebbc23716b","unresolved":true,"context_lines":[{"line_number":43,"context_line":"    cfg.StrOpt(\u0027swift_store_auth_version\u0027, default\u003d\u00273\u0027,"},{"line_number":44,"context_line":"               choices\u003d[\u00273\u0027],"},{"line_number":45,"context_line":"               help\u003d\u0027Version of the authentication service to use. \u0027"},{"line_number":46,"context_line":"                    \u0027Valid versions is 3.\u0027,"},{"line_number":47,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":48,"context_line":"               deprecated_reason\u003d\"\"\""},{"line_number":49,"context_line":"The option \u0027auth_version\u0027 in the Swift back-end configuration file is"}],"source_content_type":"text/x-python","patch_set":3,"id":"207fb0ef_128473af","line":46,"range":{"start_line":46,"start_character":42,"end_line":46,"end_character":43},"in_reply_to":"0e4c03aa_c64aee59","updated":"2024-10-19 07:40:40.000000000","message":"I agree it\u0027s confusing to keep this parameter with only a single value supported. So I\u0027ve updated the change to deprecate the whole mechanism to select auth_version. wdyt ?","commit_id":"1c42b043552da33e8a8355a2de29426d1c56c19f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"62b949620b0f7756e581513aa9ae6acc25f55122","unresolved":false,"context_lines":[{"line_number":43,"context_line":"    cfg.StrOpt(\u0027swift_store_auth_version\u0027, default\u003d\u00273\u0027,"},{"line_number":44,"context_line":"               choices\u003d[\u00273\u0027],"},{"line_number":45,"context_line":"               help\u003d\u0027Version of the authentication service to use. \u0027"},{"line_number":46,"context_line":"                    \u0027Valid versions is 3.\u0027,"},{"line_number":47,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":48,"context_line":"               deprecated_reason\u003d\"\"\""},{"line_number":49,"context_line":"The option \u0027auth_version\u0027 in the Swift back-end configuration file is"}],"source_content_type":"text/x-python","patch_set":3,"id":"f906ba62_83f112ca","line":46,"range":{"start_line":46,"start_character":42,"end_line":46,"end_character":43},"in_reply_to":"207fb0ef_128473af","updated":"2024-10-19 07:54:49.000000000","message":"Done","commit_id":"1c42b043552da33e8a8355a2de29426d1c56c19f"}],"glance_store/tests/unit/test_swift_store.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9cc31c4e43771772549f93cbfc4da017107fc141","unresolved":false,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":"class SwiftTests(object):"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def mock_keystone_client(self):"},{"line_number":73,"context_line":"        # mock keystone client functions to avoid dependency errors"},{"line_number":74,"context_line":"        swift.ks_v3 \u003d mock.MagicMock()"},{"line_number":75,"context_line":"        swift.ks_session \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":7,"id":"0800d81a_369a68ac","side":"PARENT","line":72,"range":{"start_line":72,"start_character":8,"end_line":72,"end_character":28},"updated":"2024-10-22 18:30:00.000000000","message":"It turned out this incomplete patching caused occasional failure. I updated this to be used as consistent patch method to fix the issue.","commit_id":"086bc2058fe95165b93a92a983a03466e4bdbbb2"}],"releasenotes/notes/swift-remove-auth-version-v1-and-v2-d2372f9feea42f2c.yaml":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"00fa38ac93407c843f46bee02effb29c894a0c51","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Swift store driver no longer supports auth_version less thant 3. Now"},{"line_number":5,"context_line":"    the driver uses identity v3 API by default."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"deprecations:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"8c88e20f_f2416dfa","line":4,"range":{"start_line":4,"start_character":60,"end_line":4,"end_character":65},"updated":"2025-06-30 16:33:03.000000000","message":"s/thant/than/","commit_id":"a1520ad7ce1169e7e09589c1758ee7738d989ece"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ffb9ed82709a57ee120a7ade95173addb69e9d5f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Swift store driver no longer supports auth_version less thant 3. Now"},{"line_number":5,"context_line":"    the driver uses identity v3 API by default."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"deprecations:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"54e8a50a_4b006877","line":4,"range":{"start_line":4,"start_character":60,"end_line":4,"end_character":65},"in_reply_to":"8c88e20f_f2416dfa","updated":"2025-06-30 16:34:37.000000000","message":"Done","commit_id":"a1520ad7ce1169e7e09589c1758ee7738d989ece"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"00fa38ac93407c843f46bee02effb29c894a0c51","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"deprecations:"},{"line_number":8,"context_line":"  - |"},{"line_number":9,"context_line":"    The ``auth_version`` option of the swift back-end confugrations file"},{"line_number":10,"context_line":"    has been deprecated and will be removed in a future release."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"46b819ce_5812187a","line":9,"range":{"start_line":9,"start_character":54,"end_line":9,"end_character":67},"updated":"2025-06-30 16:33:03.000000000","message":"s/confugrations/configuration/","commit_id":"a1520ad7ce1169e7e09589c1758ee7738d989ece"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ffb9ed82709a57ee120a7ade95173addb69e9d5f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"deprecations:"},{"line_number":8,"context_line":"  - |"},{"line_number":9,"context_line":"    The ``auth_version`` option of the swift back-end confugrations file"},{"line_number":10,"context_line":"    has been deprecated and will be removed in a future release."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"c61e91c3_8280b5a4","line":9,"range":{"start_line":9,"start_character":54,"end_line":9,"end_character":67},"in_reply_to":"46b819ce_5812187a","updated":"2025-06-30 16:34:37.000000000","message":"Done","commit_id":"a1520ad7ce1169e7e09589c1758ee7738d989ece"}]}
