)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"73b9613b8068cd8a0270bb43de1da2b9a6a05768","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"39060ecd_385a9300","updated":"2024-02-02 02:08:27.000000000","message":"Thanks for fixing this!","commit_id":"e99afcafe8d33624b9a695b1f7576ad0e8e5354f"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"0ccb2b5cc4ab40c780f4527d548c62f0a0038d2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fcce624b_cd130a87","updated":"2024-02-16 14:56:34.000000000","message":"thanks, looks good to me","commit_id":"e99afcafe8d33624b9a695b1f7576ad0e8e5354f"}],"cinder/backup/drivers/swift.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"73b9613b8068cd8a0270bb43de1da2b9a6a05768","unresolved":true,"context_lines":[{"line_number":450,"context_line":"            return"},{"line_number":451,"context_line":"        conn \u003d swift.Connection(retries\u003dCONF.backup_swift_retry_attempts,"},{"line_number":452,"context_line":"                                preauthurl\u003dCONF.backup_swift_url,"},{"line_number":453,"context_line":"                                cacert\u003dCONF.backup_swift_ca_cert_file)"},{"line_number":454,"context_line":"        try:"},{"line_number":455,"context_line":"            conn.get_capabilities()"},{"line_number":456,"context_line":"            # TODO(e0ne) catch less general exception"}],"source_content_type":"text/x-python","patch_set":1,"id":"3894eba3_563c572a","line":453,"updated":"2024-02-02 02:08:27.000000000","message":"Good catch.  Looks like this got missed when the service check was added by change I180ee85d3ef8d19851641.  We do seem to be passing the cacert in the other places we create a swift connection in this file, though, so I think you have a complete fix here.","commit_id":"e99afcafe8d33624b9a695b1f7576ad0e8e5354f"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"18b5e19a10e0918ac02117a30fbba5d51925bcb6","unresolved":false,"context_lines":[{"line_number":450,"context_line":"            return"},{"line_number":451,"context_line":"        conn \u003d swift.Connection(retries\u003dCONF.backup_swift_retry_attempts,"},{"line_number":452,"context_line":"                                preauthurl\u003dCONF.backup_swift_url,"},{"line_number":453,"context_line":"                                cacert\u003dCONF.backup_swift_ca_cert_file)"},{"line_number":454,"context_line":"        try:"},{"line_number":455,"context_line":"            conn.get_capabilities()"},{"line_number":456,"context_line":"            # TODO(e0ne) catch less general exception"}],"source_content_type":"text/x-python","patch_set":1,"id":"c6e3680a_3b131c3b","line":453,"in_reply_to":"3894eba3_563c572a","updated":"2024-02-02 11:07:55.000000000","message":"yep. lets merge this trivial.","commit_id":"e99afcafe8d33624b9a695b1f7576ad0e8e5354f"}]}