HYPERFLEET-1121 - feat: add safe-to-evict annotation to maestro agent/db/mqtt pods#50
HYPERFLEET-1121 - feat: add safe-to-evict annotation to maestro agent/db/mqtt pods#50kuudori wants to merge 1 commit into
Conversation
…/db/mqtt pods Wire up podAnnotations with cluster-autoscaler.kubernetes.io/safe-to-evict: "false" on maestro-agent, maestro-db (postgresql), and maestro-mqtt (mosquitto) pod templates to prevent GKE cluster-autoscaler from evicting these critical infrastructure pods during scale-down events. Bump chart dependency versions to 0.1.1 to consume upstream podAnnotations hooks.
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Important Review skippedAuto reviews are limited based on label configuration. 🚫 Review skipped — only excluded labels are configured. (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Summary
podAnnotationswithcluster-autoscaler.kubernetes.io/safe-to-evict: "false"to maestro-agent, maestro-db (postgresql), and maestro-mqtt (mosquitto) pod templatespodAnnotationshooks from feat(charts): expose podAnnotations and priorityClassName on agent/postgresql/mosquitto templates openshift-online/maestro#542Verification
Tested locally on kind cluster with upstream PR branch — all three deployments render the annotation:
Test plan
helm dependency update helm/maestrosucceedsmake install-maestro DRY_RUN=truepasseskubectl get deploy maestro-agent maestro-db maestro-mqtt -n maestro -o jsonpath='{range .items[*]}{.metadata.name}: {.spec.template.metadata.annotations}{"\n"}{end}'shows annotation on all threeScaleDownevents targeting these pods over 7-day observation window