)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1004879,"name":"Matt Hall","email":"matthew@quickbeam.me.uk","username":"mattyhall","avatars":[{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"209d8b291f24e9f797dff56515217edc181d0c00","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"083345be_78b047f5","updated":"2025-03-26 09:10:46.000000000","message":"These are good changes, thank you. I think there\u0027s another issue the ticket is talking about - when `--audit-usage` is passed we are sending ns_server `audit-usage`, but it should be `audit-encryption`.","commit_id":"5f7b29a9ffc4f9101ab350c37c2907b46c912c4e"},{"author":{"_account_id":1005389,"name":"Lubo Marinski","email":"lubo.marinski@couchbase.com","username":"lubomarinski","avatars":[{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"08b4443613c834ad70dbe52af89bc32976092c1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d527d7f6_c2382896","in_reply_to":"083345be_78b047f5","updated":"2025-03-26 11:14:27.000000000","message":"Done","commit_id":"5f7b29a9ffc4f9101ab350c37c2907b46c912c4e"}],"cbmgr.py":[{"author":{"_account_id":1004879,"name":"Matt Hall","email":"matthew@quickbeam.me.uk","username":"mattyhall","avatars":[{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"209d8b291f24e9f797dff56515217edc181d0c00","unresolved":true,"context_lines":[{"line_number":3359,"context_line":"        if not opts.name:"},{"line_number":3360,"context_line":"            _exit_if_errors([\"--name must be specified\"])"},{"line_number":3361,"context_line":""},{"line_number":3362,"context_line":"        encrypt_with_key \u003d False"},{"line_number":3363,"context_line":"        if opts.encrypt_with_key or opts.encrypt_with_key \u003d\u003d 0:"},{"line_number":3364,"context_line":"            encrypt_with_key \u003d True"},{"line_number":3365,"context_line":""},{"line_number":3366,"context_line":"        usages \u003d []"},{"line_number":3367,"context_line":"        if opts.config_usage:"}],"source_content_type":"text/x-python","patch_set":1,"id":"b9b72d05_bace33a9","line":3364,"range":{"start_line":3362,"start_character":0,"end_line":3364,"end_character":35},"updated":"2025-03-26 09:10:46.000000000","message":"The old code by me was very bad, sorry 🙈. Thank you for fixing it up.\n\nThe `if` here feels wrong - I think we want to do something like `if opts.encrypt_with_key is not None:` (presuming the default is `None`, which we should make the case if not)","commit_id":"5f7b29a9ffc4f9101ab350c37c2907b46c912c4e"},{"author":{"_account_id":1005389,"name":"Lubo Marinski","email":"lubo.marinski@couchbase.com","username":"lubomarinski","avatars":[{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"08b4443613c834ad70dbe52af89bc32976092c1f","unresolved":false,"context_lines":[{"line_number":3359,"context_line":"        if not opts.name:"},{"line_number":3360,"context_line":"            _exit_if_errors([\"--name must be specified\"])"},{"line_number":3361,"context_line":""},{"line_number":3362,"context_line":"        encrypt_with_key \u003d False"},{"line_number":3363,"context_line":"        if opts.encrypt_with_key or opts.encrypt_with_key \u003d\u003d 0:"},{"line_number":3364,"context_line":"            encrypt_with_key \u003d True"},{"line_number":3365,"context_line":""},{"line_number":3366,"context_line":"        usages \u003d []"},{"line_number":3367,"context_line":"        if opts.config_usage:"}],"source_content_type":"text/x-python","patch_set":1,"id":"e1089353_1a9b07e2","line":3364,"range":{"start_line":3362,"start_character":0,"end_line":3364,"end_character":35},"in_reply_to":"b9b72d05_bace33a9","updated":"2025-03-26 11:14:27.000000000","message":"Done","commit_id":"5f7b29a9ffc4f9101ab350c37c2907b46c912c4e"},{"author":{"_account_id":1004879,"name":"Matt Hall","email":"matthew@quickbeam.me.uk","username":"mattyhall","avatars":[{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/53d1d84f1bfdafc6399d56cba3efbe08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"209d8b291f24e9f797dff56515217edc181d0c00","unresolved":true,"context_lines":[{"line_number":5195,"context_line":""},{"line_number":5196,"context_line":"        if statuses[\u0027apps\u0027] is not None:"},{"line_number":5197,"context_line":"            statuses \u003d self._get_eventing_functions_paths_to_statuses_map(statuses[\u0027apps\u0027])"},{"line_number":5198,"context_line":"            for fn in functions:"},{"line_number":5199,"context_line":"                function_path \u003d self._get_function_path_from_list_functions_endpoint(fn)"},{"line_number":5200,"context_line":"                if function_path in statuses:"},{"line_number":5201,"context_line":"                    print(function_path)"},{"line_number":5202,"context_line":"                    print(f\u0027 Status: {statuses[function_path]}\u0027)"},{"line_number":5203,"context_line":"                    if \u0027source_scope\u0027 in fn[\"depcfg\"]:"},{"line_number":5204,"context_line":"                        print(f\u0027 Source: {fn[\"depcfg\"][\"source_bucket\"]}.{fn[\"depcfg\"][\"source_scope\"]}.\u0027"},{"line_number":5205,"context_line":"                              f\u0027{fn[\"depcfg\"][\"source_collection\"]}\u0027)"},{"line_number":5206,"context_line":"                        print("},{"line_number":5207,"context_line":"                            f\u0027 Metadata: {fn[\"depcfg\"][\"metadata_bucket\"]}.{fn[\"depcfg\"][\"metadata_scope\"]}.\u0027"},{"line_number":5208,"context_line":"                            f\u0027{fn[\"depcfg\"][\"metadata_collection\"]}\u0027)"},{"line_number":5209,"context_line":"                    else:"},{"line_number":5210,"context_line":"                        print(f\u0027 Source Bucket: {fn[\"depcfg\"][\"source_bucket\"]}\u0027)"},{"line_number":5211,"context_line":"                        print(f\u0027 Metadata Bucket: {fn[\"depcfg\"][\"metadata_bucket\"]}\u0027)"},{"line_number":5212,"context_line":"        else:"},{"line_number":5213,"context_line":"            print(\u0027The cluster has no functions\u0027)"},{"line_number":5214,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ec1acde1_cd8c89c1","line":5211,"range":{"start_line":5198,"start_character":0,"end_line":5211,"end_character":85},"updated":"2025-03-26 09:10:46.000000000","message":"I think this should be covered by the commit you split out from your other patches?","commit_id":"5f7b29a9ffc4f9101ab350c37c2907b46c912c4e"},{"author":{"_account_id":1005389,"name":"Lubo Marinski","email":"lubo.marinski@couchbase.com","username":"lubomarinski","avatars":[{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/60f9d0c92536ec050b83034161160480.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"08b4443613c834ad70dbe52af89bc32976092c1f","unresolved":false,"context_lines":[{"line_number":5195,"context_line":""},{"line_number":5196,"context_line":"        if statuses[\u0027apps\u0027] is not None:"},{"line_number":5197,"context_line":"            statuses \u003d self._get_eventing_functions_paths_to_statuses_map(statuses[\u0027apps\u0027])"},{"line_number":5198,"context_line":"            for fn in functions:"},{"line_number":5199,"context_line":"                function_path \u003d self._get_function_path_from_list_functions_endpoint(fn)"},{"line_number":5200,"context_line":"                if function_path in statuses:"},{"line_number":5201,"context_line":"                    print(function_path)"},{"line_number":5202,"context_line":"                    print(f\u0027 Status: {statuses[function_path]}\u0027)"},{"line_number":5203,"context_line":"                    if \u0027source_scope\u0027 in fn[\"depcfg\"]:"},{"line_number":5204,"context_line":"                        print(f\u0027 Source: {fn[\"depcfg\"][\"source_bucket\"]}.{fn[\"depcfg\"][\"source_scope\"]}.\u0027"},{"line_number":5205,"context_line":"                              f\u0027{fn[\"depcfg\"][\"source_collection\"]}\u0027)"},{"line_number":5206,"context_line":"                        print("},{"line_number":5207,"context_line":"                            f\u0027 Metadata: {fn[\"depcfg\"][\"metadata_bucket\"]}.{fn[\"depcfg\"][\"metadata_scope\"]}.\u0027"},{"line_number":5208,"context_line":"                            f\u0027{fn[\"depcfg\"][\"metadata_collection\"]}\u0027)"},{"line_number":5209,"context_line":"                    else:"},{"line_number":5210,"context_line":"                        print(f\u0027 Source Bucket: {fn[\"depcfg\"][\"source_bucket\"]}\u0027)"},{"line_number":5211,"context_line":"                        print(f\u0027 Metadata Bucket: {fn[\"depcfg\"][\"metadata_bucket\"]}\u0027)"},{"line_number":5212,"context_line":"        else:"},{"line_number":5213,"context_line":"            print(\u0027The cluster has no functions\u0027)"},{"line_number":5214,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"d8f0dc43_2333af5a","line":5211,"range":{"start_line":5198,"start_character":0,"end_line":5211,"end_character":85},"in_reply_to":"ec1acde1_cd8c89c1","updated":"2025-03-26 11:14:27.000000000","message":"Yes, it is not needed anymore","commit_id":"5f7b29a9ffc4f9101ab350c37c2907b46c912c4e"}]}