deployment: add gcp pod monitoring to hybrid environment overlays#14192
Conversation
PR SummaryLow Risk Overview Each overlay adds the same scrape spec: port 8082, path Reviewed by Cursor Bugbot for commit 77b2c59. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ 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 |
There was a problem hiding this comment.
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)
Reviewed by Cursor Bugbot for commit 77b2c59. Configure here.



Cross-repo PR: https://github.com/starkware-industries/sequencer-devops/pull/719
Cross-repo PR: https://github.com/starkware-industries/sequencer-devops/pull/719