)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"880c2a8cceb6107d3b05e4e8e02fd0e4635d7da5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6c2bfe80_48cdcdd4","updated":"2026-05-25 18:46:12.000000000","message":"Perfect, thank you 🙂\nNot sure if we need to backport this to earlier versions.","commit_id":"d0b996a143ceb1ec96823e07f27906f6190f2bb8"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"6b5cf4f91f505539e3d7fef2a08b0449cf8622c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"33b938d9_06fa1830","updated":"2026-04-09 08:58:21.000000000","message":"recheck","commit_id":"d0b996a143ceb1ec96823e07f27906f6190f2bb8"}],"trove/common/clients_admin.py":[{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"mangust404@gmail.com","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"4c232c1dcc3601818dd7f4ffe805fbe2a5daa25d","unresolved":true,"context_lines":[{"line_number":98,"context_line":"        LOG.debug(\u0027Re-use admin cinder client\u0027)"},{"line_number":99,"context_line":"        return ADMIN_CINDER_CLIENT"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    version \u003d \u00273\u0027"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    ks_session \u003d get_keystone_session()"},{"line_number":104,"context_line":"    ADMIN_CINDER_CLIENT \u003d CinderClient.Client("}],"source_content_type":"text/x-python","patch_set":1,"id":"7282fa04_52d2adb6","line":101,"updated":"2026-04-06 09:59:53.000000000","message":"Hello, Dmitriy! Thank you for the provided patch.\n\nLGTM, and the current version detection logic seems broken, but:\n\n1. Probably, instead of hardcoding, the better approach will be to add additional parameter like the existing `CONF.nova_client_version` and `CONF.glance_client_version`, how do you think?\n\n2. We need to add a reno note, since this changes the behavior.","commit_id":"88028bbe2a03cb04c13ae73c13f2076dc6cb056a"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"ba4ab384f0f23e40fd42ff92c76f1910a2adf2a6","unresolved":false,"context_lines":[{"line_number":98,"context_line":"        LOG.debug(\u0027Re-use admin cinder client\u0027)"},{"line_number":99,"context_line":"        return ADMIN_CINDER_CLIENT"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    version \u003d \u00273\u0027"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    ks_session \u003d get_keystone_session()"},{"line_number":104,"context_line":"    ADMIN_CINDER_CLIENT \u003d CinderClient.Client("}],"source_content_type":"text/x-python","patch_set":1,"id":"5842f1f1_7699df11","line":101,"in_reply_to":"7282fa04_52d2adb6","updated":"2026-04-08 08:29:23.000000000","message":"So the reason why I did not introduced the new config, is because all these clients must be replaced with openstacksdk, as neutronclient/novaclient/cinderclient are deprecated at this point.\nBut yeah, we can pass volume_api_version to sdk as well","commit_id":"88028bbe2a03cb04c13ae73c13f2076dc6cb056a"}]}