)]}'
{"/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":"8b36eae86a072676daf3c8f5df1ee5f38f995138","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a5420edc_c31384ca","updated":"2026-05-21 14:13:14.000000000","message":"The commit message lacks detail about the specific changes made in the code. It should provide more context on why including cluster spec names in Prometheus snapshots metadata is necessary and how it impacts the system. A clearer commit message would be \u0027Add cluster spec names to Prometheus snapshots metadata to enhance traceability and monitoring capabilities.\u0027","commit_id":"3cf7c212d310db8a18bb9e2fa51a7dceb4549bb1"},{"author":{"_account_id":1005024,"name":"Salim Salim","email":"salim.salim@couchbase.com","username":"sudosalim","avatars":[{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"1e764a3b006ad225299362dd85e2a01407e732fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"92870d2f_040a77ac","in_reply_to":"a5420edc_c31384ca","updated":"2026-05-26 11:14:42.000000000","message":"Acknowledged","commit_id":"3cf7c212d310db8a18bb9e2fa51a7dceb4549bb1"}],"perfrunner/metrics/prometheus_agent.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":"0b37bbfcc1ba3d455ce26d99af26e699a6a809e7","unresolved":true,"context_lines":[{"line_number":72,"context_line":"        try:"},{"line_number":73,"context_line":"            # Collect all server targets as scrape configs"},{"line_number":74,"context_line":"            scheme \u003d \"http\" if not self.use_tls_ports else \"https\""},{"line_number":75,"context_line":"            build_label \u003d os.getenv(\"BUILD_URL\", \"\")"},{"line_number":76,"context_line":"            cluster_label \u003d self.cluster_spec.metadata_cluster_name"},{"line_number":77,"context_line":"            label \u003d f\"{build_label} [{cluster_label}]\" if build_label else cluster_label"},{"line_number":78,"context_line":"            payload \u003d {"},{"line_number":79,"context_line":"                \"configs\": ["},{"line_number":80,"context_line":"                    {"}],"source_content_type":"text/x-python","patch_set":1,"id":"4db5cb31_77d58ccd","line":77,"range":{"start_line":75,"start_character":12,"end_line":77,"end_character":88},"updated":"2026-05-20 15:15:46.000000000","message":"In \u0027perfrunner/metrics/prometheus_agent.py\u0027, the change from using \u0027os.getenv(\"BUILD_URL\", \"\")\u0027 directly to constructing a \u0027label\u0027 with \u0027build_label\u0027 and \u0027cluster_label\u0027 might introduce complexity without clear benefit. Consider if this change is necessary or if it can be simplified.","commit_id":"ef9a2775433a6b5ee2ff9ee96ac8b2650cd1bb4d"},{"author":{"_account_id":1005024,"name":"Salim Salim","email":"salim.salim@couchbase.com","username":"sudosalim","avatars":[{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"1e764a3b006ad225299362dd85e2a01407e732fa","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        try:"},{"line_number":73,"context_line":"            # Collect all server targets as scrape configs"},{"line_number":74,"context_line":"            scheme \u003d \"http\" if not self.use_tls_ports else \"https\""},{"line_number":75,"context_line":"            build_label \u003d os.getenv(\"BUILD_URL\", \"\")"},{"line_number":76,"context_line":"            cluster_label \u003d self.cluster_spec.metadata_cluster_name"},{"line_number":77,"context_line":"            label \u003d f\"{build_label} [{cluster_label}]\" if build_label else cluster_label"},{"line_number":78,"context_line":"            payload \u003d {"},{"line_number":79,"context_line":"                \"configs\": ["},{"line_number":80,"context_line":"                    {"}],"source_content_type":"text/x-python","patch_set":1,"id":"f618f4fc_d3bd70cf","line":77,"range":{"start_line":75,"start_character":12,"end_line":77,"end_character":88},"in_reply_to":"4db5cb31_77d58ccd","updated":"2026-05-26 11:14:42.000000000","message":"Acknowledged","commit_id":"ef9a2775433a6b5ee2ff9ee96ac8b2650cd1bb4d"}],"perfrunner/tests/__init__.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":"0b37bbfcc1ba3d455ce26d99af26e699a6a809e7","unresolved":true,"context_lines":[{"line_number":115,"context_line":"        self.collector_agent \u003d PrometheusAgent(self.cluster_spec, self.stats_settings, self.rest)"},{"line_number":116,"context_line":"        self.cbmonitor_snapshots.append(self.collector_agent.snapshot_id)"},{"line_number":117,"context_line":"        self.cbmonitor_clusters.append("},{"line_number":118,"context_line":"            self.cluster_spec.metadata_cluster_name"},{"line_number":119,"context_line":"        )  # We dont use this when using Prometheus, just here for compatibility with the old store"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        self.metrics \u003d PrometheusMetricsHelper(self)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3dbde9ed_d7f2e0f0","line":118,"range":{"start_line":118,"start_character":12,"end_line":118,"end_character":51},"updated":"2026-05-20 15:15:46.000000000","message":"In \u0027perfrunner/tests/\\_\\_init\\_\\_.py\u0027, the change from \u0027self.cluster_spec.name\u0027 to \u0027self.cluster_spec.metadata_cluster_name\u0027 should be verified to ensure that \u0027metadata_cluster_name\u0027 is correctly populated and used throughout the system. This change could affect other parts of the code that rely on this attribute.","commit_id":"ef9a2775433a6b5ee2ff9ee96ac8b2650cd1bb4d"},{"author":{"_account_id":1005024,"name":"Salim Salim","email":"salim.salim@couchbase.com","username":"sudosalim","avatars":[{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/130e9e93bf02bbfade74e9470215dec1.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"1e764a3b006ad225299362dd85e2a01407e732fa","unresolved":false,"context_lines":[{"line_number":115,"context_line":"        self.collector_agent \u003d PrometheusAgent(self.cluster_spec, self.stats_settings, self.rest)"},{"line_number":116,"context_line":"        self.cbmonitor_snapshots.append(self.collector_agent.snapshot_id)"},{"line_number":117,"context_line":"        self.cbmonitor_clusters.append("},{"line_number":118,"context_line":"            self.cluster_spec.metadata_cluster_name"},{"line_number":119,"context_line":"        )  # We dont use this when using Prometheus, just here for compatibility with the old store"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        self.metrics \u003d PrometheusMetricsHelper(self)"}],"source_content_type":"text/x-python","patch_set":1,"id":"e9aa427b_51f6c035","line":118,"range":{"start_line":118,"start_character":12,"end_line":118,"end_character":51},"in_reply_to":"3dbde9ed_d7f2e0f0","updated":"2026-05-26 11:14:42.000000000","message":"Acknowledged","commit_id":"ef9a2775433a6b5ee2ff9ee96ac8b2650cd1bb4d"}]}