)]}'
{"/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":"e5c1f14b3c0de1e596fa70ac65a4e6ffe0b688d5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"245b00a3_64edbaca","updated":"2026-05-07 10:30:28.000000000","message":"The commit message lacks detail on how the tests are implemented and what specific scenarios they cover. A clearer commit message would be: \u0027Implement automated tests for Community Edition restrictions, including 5-node limit enforcement and service topology restrictions. Added utility functions for node management and restriction validation. Updated configuration for test setup.\u0027","commit_id":"ee932258579293e38bcf109a6a50208cb5a54162"}],"pytests/ns_server/ce_restrictions_util.py":[{"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":"e5c1f14b3c0de1e596fa70ac65a4e6ffe0b688d5","unresolved":true,"context_lines":[{"line_number":31,"context_line":"    def verify_ce_edition_via_diag_eval(self):"},{"line_number":32,"context_line":"        \"\"\"Verify cluster is running CE via diag/eval.\"\"\""},{"line_number":33,"context_line":"        shell \u003d RemoteMachineShellConnection(self.cluster.master)"},{"line_number":34,"context_line":"        shell.enable_diag_eval_on_non_local_hosts()"},{"line_number":35,"context_line":"        shell.disconnect()"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        status, content \u003d self.rest.diag_eval("}],"source_content_type":"text/x-python","patch_set":4,"id":"783f6d1f_210837b4","line":34,"range":{"start_line":34,"start_character":8,"end_line":34,"end_character":51},"updated":"2026-05-07 10:30:28.000000000","message":"The method `verify_ce_edition_via_diag_eval` in `ce_restrictions_util.py` uses a potentially risky approach by enabling diagnostic evaluation on non-local hosts without any security checks. This could expose the cluster to unauthorized access. Consider adding security checks or warnings to ensure this action is safe.","commit_id":"ee932258579293e38bcf109a6a50208cb5a54162"}]}