Skip to content

deployment: add gcp pod monitoring to hybrid environment overlays#14192

Open
nimrod-starkware wants to merge 1 commit into
05-25-deployment_add_liveness_probe_to_hybrid_environment_overlaysfrom
05-25-deployment_add_gcp_pod_monitoring_to_hybrid_environment_overlays
Open

deployment: add gcp pod monitoring to hybrid environment overlays#14192
nimrod-starkware wants to merge 1 commit into
05-25-deployment_add_liveness_probe_to_hybrid_environment_overlaysfrom
05-25-deployment_add_gcp_pod_monitoring_to_hybrid_environment_overlays

Conversation

@nimrod-starkware
Copy link
Copy Markdown
Contributor

@nimrod-starkware nimrod-starkware commented May 25, 2026

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@nimrod-starkware nimrod-starkware self-assigned this May 26, 2026
@nimrod-starkware nimrod-starkware marked this pull request as ready for review May 26, 2026 06:29
@cursor
Copy link
Copy Markdown

cursor Bot commented May 26, 2026

PR Summary

Low Risk
Observability-only Helm overlay changes; no application logic, auth, or runtime behavior changes beyond metrics collection config.

Overview
Enables GCP PodMonitoring for hybrid sequencer deployments on mainnet, sepolia-alpha, and sepolia-integration by setting gcpPodMonitoring.enabled: true in each environment’s common.yaml overlay.

Each overlay adds the same scrape spec: port 8082, path /monitoring/metrics, interval 10s, aligned with the existing monitoring-endpoint service port already defined in those files.

Reviewed by Cursor Bugbot for commit 77b2c59. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 77b2c59. Configure here.

endpoints:
- port: 8082
path: /monitoring/metrics
interval: 10s
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Common overlay enabled: true silently overridden by service defaults

High Severity

Setting gcpPodMonitoring.enabled: true in the common overlay has no effect. Every service layout file (e.g. core.yaml, gateway.yaml, etc.) already defines gcpPodMonitoring.enabled: false. During _merge_common_into_service, the deep_merge function uses the service dict as base and common dict as overlay, and only adds keys from the overlay that don't already exist in the base. Since enabled: false already exists in each service, the common's enabled: true is silently discarded. Pod monitoring will remain disabled for all services.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 77b2c59. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants