)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"455806ae62c6ec4eb44cd9154cff180e09495c4a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d116e502_eccfc26a","updated":"2026-01-06 14:35:32.000000000","message":"please also fix the pep8 issue","commit_id":"aa7ab43631b02f4825e93fe64c18fe9e7fe5f8fe"}],"trove/common/clients_admin.py":[{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"455806ae62c6ec4eb44cd9154cff180e09495c4a","unresolved":true,"context_lines":[{"line_number":48,"context_line":"            auth_url\u003dCONF.service_credentials.auth_url)"},{"line_number":49,"context_line":"        verify \u003d ("},{"line_number":50,"context_line":"                    False if getattr(CONF.keystone_authtoken, \u0027insecure\u0027, False)"},{"line_number":51,"context_line":"                    else getattr(CONF.keystone_authtoken, \u0027cafile\u0027, True)"},{"line_number":52,"context_line":"                 )"},{"line_number":53,"context_line":"        _SESSION \u003d session.Session(auth\u003dauth, verify\u003dverify)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    return _SESSION"}],"source_content_type":"text/x-python","patch_set":1,"id":"7512cf5d_21b899b0","line":52,"range":{"start_line":51,"start_character":0,"end_line":52,"end_character":18},"updated":"2026-01-06 14:35:32.000000000","message":"we should consider the case that the cafile was explicitly set to None or empty string, this most happened in Kolla deployment e.g:\n```\nauth_type \u003d password\ncafile \u003d\nregion_name \u003d RegionOne\n```\nsuggest code snippet:\n\n```\n        # Handle SSL verification based on keystone_authtoken settings\n        insecure \u003d getattr(CONF.keystone_authtoken, \u0027insecure\u0027, False)\n        if insecure:\n            verify \u003d False\n        else:\n            cafile \u003d getattr(CONF.keystone_authtoken, \u0027cafile\u0027, None)\n            # If cafile is set and non-empty, use it; otherwise use system certs\n            verify \u003d cafile if cafile else True\n```","commit_id":"aa7ab43631b02f4825e93fe64c18fe9e7fe5f8fe"},{"author":{"_account_id":36698,"name":"hamid lotfi","display_name":"Hamid Lotfi","email":"hamid.lotfi@gmail.com","username":"hamidlotfi"},"change_message_id":"143b5e707e00682b300f1ae79c4a6c347bd8d686","unresolved":false,"context_lines":[{"line_number":48,"context_line":"            auth_url\u003dCONF.service_credentials.auth_url)"},{"line_number":49,"context_line":"        verify \u003d ("},{"line_number":50,"context_line":"                    False if getattr(CONF.keystone_authtoken, \u0027insecure\u0027, False)"},{"line_number":51,"context_line":"                    else getattr(CONF.keystone_authtoken, \u0027cafile\u0027, True)"},{"line_number":52,"context_line":"                 )"},{"line_number":53,"context_line":"        _SESSION \u003d session.Session(auth\u003dauth, verify\u003dverify)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    return _SESSION"}],"source_content_type":"text/x-python","patch_set":1,"id":"1c69597d_73f7d75a","line":52,"range":{"start_line":51,"start_character":0,"end_line":52,"end_character":18},"in_reply_to":"7512cf5d_21b899b0","updated":"2026-01-25 08:16:16.000000000","message":"Done","commit_id":"aa7ab43631b02f4825e93fe64c18fe9e7fe5f8fe"}]}