)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1003869,"name":"Couchbase Robot","email":"build-team@couchbase.com","username":"cb-robot","avatars":[{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"8f5e8b1dcfc295acb5070e3213ce8685b84740bd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1d958f68_2bc14c5a","updated":"2025-04-29 11:26:27.000000000","message":"The commit message lacks detail about the specific tests added for encryption. It should provide more context on what aspects of encryption are being tested and why these tests are necessary.","commit_id":"b32e81465b58db24fbcd70bf5dfcd384cbf29ac0"}],"conf/ns_server/encryption_at_rest.conf":[{"author":{"_account_id":1003869,"name":"Couchbase Robot","email":"build-team@couchbase.com","username":"cb-robot","avatars":[{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"8f5e8b1dcfc295acb5070e3213ce8685b84740bd","unresolved":true,"context_lines":[{"line_number":1,"context_line":"ns_server.encryption_at_rest.EncryptionAtRest:"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"    test_bucket_encryption_auto_rotation,nodes_init\u003d3,services_init\u003dkv-kv:index-kv:backup,bucket_spec\u003dmulti_bucket.range_scan_data_load,enable_encryption_at_rest\u003dTrue,enable_config_encryption_at_rest\u003dTrue,enable_log_encryption_at_rest\u003dFalse,config_dekRotationInterval\u003d100,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,config_dekLifetime\u003d100,rotationIntervalInSeconds\u003d1000,encryptionAtRestDekRotationInterval\u003d100,kv_quota_percent\u003d50,rotationIntervalInSeconds\u003d1000,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,encryptionAtRestDekRotationInterval\u003d100,enable_audit_encryption_at_rest\u003dTrue,enable_config_encryption_at_rest\u003dTrue,audit_dekRotationInterval\u003d100,config_dekRotationInterval\u003d100,config_dekLifetime\u003d100,enable_encryption_at_rest\u003dTrue,enable_log_encryption_at_rest\u003dFalse,GROUP\u003dP0"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"    test_bucket_encryption_auto_rotation,nodes_init\u003d6,services_init\u003dkv-fts-n1ql-kv-kv:index-kv:backup,bucket_spec\u003dmulti_bucket.buckets_for_rebalance_tests,enable_encryption_at_rest\u003dTrue,enable_config_encryption_at_rest\u003dTrue,enable_log_encryption_at_rest\u003dFalse,config_dekRotationInterval\u003d100,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,config_dekLifetime\u003d100,rotationIntervalInSeconds\u003d1000,encryptionAtRestDekRotationInterval\u003d100,kv_quota_percent\u003d50,rotationIntervalInSeconds\u003d1000,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,encryptionAtRestDekRotationInterval\u003d100,enable_audit_encryption_at_rest\u003dTrue,enable_config_encryption_at_rest\u003dTrue,audit_dekRotationInterval\u003d100,config_dekRotationInterval\u003d200,config_dekLifetime\u003d100,enable_encryption_at_rest\u003dTrue,enable_log_encryption_at_rest\u003dFalse,GROUP\u003dP1"},{"line_number":6,"context_line":""}],"source_content_type":"text/plain","patch_set":1,"id":"3cb72bd8_7d861dfe","line":3,"range":{"start_line":3,"start_character":4,"end_line":3,"end_character":240},"updated":"2025-04-29 11:26:27.000000000","message":"The test names in \u0027conf/ns_server/encryption_at_rest.conf\u0027 are very long and include many parameters directly in the name. Consider using a more concise naming convention and documenting the parameters separately to improve readability.","commit_id":"b32e81465b58db24fbcd70bf5dfcd384cbf29ac0"},{"author":{"_account_id":1003869,"name":"Couchbase Robot","email":"build-team@couchbase.com","username":"cb-robot","avatars":[{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"8f5e8b1dcfc295acb5070e3213ce8685b84740bd","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    test_bucket_encryption_at_rest,nodes_init\u003d6,services_init\u003dkv-fts-n1ql-kv-kv:index-kv:backup,bucket_spec\u003dmulti_bucket.range_scan_data_load,enable_encryption_at_rest\u003dFalse,enable_config_encryption_at_rest\u003dTrue,enable_log_encryption_at_rest\u003dTrue,config_dekRotationInterval\u003d100,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,config_dekLifetime\u003d10,rotationIntervalInSeconds\u003d100,encryptionAtRestDekRotationInterval\u003d100,kv_quota_percent\u003d50,rotationIntervalInSeconds\u003d100,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,encryptionAtRestDekRotationInterval\u003d100,enable_audit_encryption_at_rest\u003dTrue,enable_config_encryption_at_rest\u003dFalse,audit_dekRotationInterval\u003d100,config_dekRotationInterval\u003d100,config_dekLifetime\u003d0,enable_encryption_at_rest\u003dFalse,enable_log_encryption_at_rest\u003dTrue,GROUP\u003dP1"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    test_rapid_dek_rotation_with_crud_and_recovery,nodes_init\u003d3,services_init\u003dkv-kv-kv,bucket_spec\u003danalytics.multi_bucket,graceful\u003dfalse,GROUP\u003dP0"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    test_rapid_dek_rotation_with_crud_and_recovery,nodes_init\u003d3,services_init\u003dkv-kv-kv,bucket_spec\u003danalytics.multi_bucket,GROUP\u003dP0"},{"line_number":26,"context_line":""}],"source_content_type":"text/plain","patch_set":1,"id":"77dc49ca_05431480","line":23,"range":{"start_line":23,"start_character":4,"end_line":23,"end_character":145},"updated":"2025-04-29 11:26:27.000000000","message":"In \u0027conf/ns_server/encryption_at_rest.conf\u0027, the test case \u0027test_rapid_dek_rotation_with_crud_and_recovery\u0027 is duplicated with only a difference in the \u0027graceful\u0027 parameter. Consider combining these into a single test with a parameterized approach to improve maintainability.","commit_id":"b32e81465b58db24fbcd70bf5dfcd384cbf29ac0"}],"conf/ns_server/min_replica_set.conf":[{"author":{"_account_id":1003869,"name":"Couchbase Robot","email":"build-team@couchbase.com","username":"cb-robot","avatars":[{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/d465bd97bd9f40f1453c5b112ef4a021.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"8f5e8b1dcfc295acb5070e3213ce8685b84740bd","unresolved":true,"context_lines":[{"line_number":10,"context_line":"    test_minimum_replica_setting,num_items\u003d10000,nodes_init\u003d4,bucket_size\u003d256,new_minimum_replica\u003d-1,expect_min_replica_update_fail\u003dTrue,GROUP\u003dP0"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    #############################################Encryption At Rest#######################################################################################"},{"line_number":13,"context_line":"    test_bucket_encryption_properties,nodes_init\u003d3,services_init\u003dkv-kv:index-kv:backup,bucket_spec\u003dmulti_bucket.range_scan_data_load,enable_encryption_at_rest\u003dFalse,enable_config_encryption_at_rest\u003dTrue,enable_log_encryption_at_rest\u003dFalse,config_dekRotationInterval\u003d1000,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,config_dekLifetime\u003d10,rotationIntervalInSeconds\u003d200,encryptionAtRestDekRotationInterval\u003d100,kv_quota_percent\u003d50,rotationIntervalInSeconds\u003d1000,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,encryptionAtRestDekRotationInterval\u003d100,enable_audit_encryption_at_rest\u003dTrue,enable_config_encryption_at_rest\u003dFalse,audit_dekRotationInterval\u003d100,config_dekRotationInterval\u003d100,config_dekLifetime\u003d0,enable_encryption_at_rest\u003dFalse,enable_log_encryption_at_rest\u003dFalse,Group\u003dP0_enc"},{"line_number":14,"context_line":"    test_bucket_encryption_properties,nodes_init\u003d6,services_init\u003dkv-fts-n1ql-kv-kv:index-kv:backup,bucket_spec\u003dmulti_bucket.range_scan_data_load,enable_encryption_at_rest\u003dFalse,enable_config_encryption_at_rest\u003dTrue,enable_log_encryption_at_rest\u003dFalse,config_dekRotationInterval\u003d100,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,config_dekLifetime\u003d1000,rotationIntervalInSeconds\u003d100,encryptionAtRestDekRotationInterval\u003d1000,kv_quota_percent\u003d50,rotationIntervalInSeconds\u003d500,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,encryptionAtRestDekRotationInterval\u003d100,enable_audit_encryption_at_rest\u003dTrue,enable_config_encryption_at_rest\u003dFalse,audit_dekRotationInterval\u003d100,config_dekRotationInterval\u003d100,config_dekLifetime\u003d0,enable_encryption_at_rest\u003dFalse,enable_log_encryption_at_rest\u003dFalse,Group\u003dP0_enc"},{"line_number":15,"context_line":"    test_enable_log_encryption_and_test,nodes_init\u003d6,services_init\u003dkv-fts-n1ql-kv-kv:index-kv:backup,bucket_spec\u003dmulti_bucket.range_scan_data_load,enable_encryption_at_rest\u003dFalse,enable_config_encryption_at_rest\u003dTrue,enable_log_encryption_at_rest\u003dFalse,config_dekRotationInterval\u003d100,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,config_dekLifetime\u003d100,rotationIntervalInSeconds\u003d100,encryptionAtRestDekRotationInterval\u003d100,kv_quota_percent\u003d50,rotationIntervalInSeconds\u003d100,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,encryptionAtRestDekRotationInterval\u003d100,enable_audit_encryption_at_rest\u003dTrue,enable_config_encryption_at_rest\u003dFalse,audit_dekRotationInterval\u003d100,config_dekRotationInterval\u003d100,config_dekLifetime\u003d0,enable_encryption_at_rest\u003dFalse,enable_log_encryption_at_rest\u003dFalse,Group\u003dP0_enc"},{"line_number":16,"context_line":"    test_enable_config_encryption_and_test,nodes_init\u003d3,services_init\u003dkv-kv:index-kv:backup,bucket_spec\u003dmulti_bucket.range_scan_data_load,enable_encryption_at_rest\u003dFalse,enable_config_encryption_at_rest\u003dTrue,enable_log_encryption_at_rest\u003dFalse,config_dekRotationInterval\u003d1000,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,config_dekLifetime\u003d100,rotationIntervalInSeconds\u003d200,encryptionAtRestDekRotationInterval\u003d1000,kv_quota_percent\u003d50,rotationIntervalInSeconds\u003d300,log_dekRotationInterval\u003d100,log_dekLifetime\u003d0,encryptionAtRestDekRotationInterval\u003d100,enable_audit_encryption_at_rest\u003dTrue,enable_config_encryption_at_rest\u003dFalse,audit_dekRotationInterval\u003d100,config_dekRotationInterval\u003d100,config_dekLifetime\u003d0,enable_encryption_at_rest\u003dFalse,enable_log_encryption_at_rest\u003dFalse,Group\u003dP0_enc"}],"source_content_type":"text/plain","patch_set":1,"id":"39441381_f21591d4","line":13,"range":{"start_line":13,"start_character":429,"end_line":13,"end_character":545},"updated":"2025-04-29 11:26:27.000000000","message":"The new test cases for encryption at rest in \u0027conf/ns_server/min_replica_set.conf\u0027 have redundant parameters such as \u0027rotationIntervalInSeconds\u0027 and \u0027log_dekRotationInterval\u0027 being repeated multiple times with the same values. Consider consolidating these parameters to avoid redundancy.","commit_id":"b32e81465b58db24fbcd70bf5dfcd384cbf29ac0"}]}