Skip to content

ci: bump kubectl rollout-status timeout 180s → 300s#199

Merged
mastermanas805 merged 1 commit into
masterfrom
ci/bump-rollout-timeout-300s
May 30, 2026
Merged

ci: bump kubectl rollout-status timeout 180s → 300s#199
mastermanas805 merged 1 commit into
masterfrom
ci/bump-rollout-timeout-300s

Conversation

@mastermanas805

Copy link
Copy Markdown
Member

Why

Today's PR #198 (CORS fix) deploy timed out at 180s even though the new ReplicaSet came up healthy. The rollout actually took ~240s on a fresh node with a cold image pull. The job exited red, prod was correct, and the false-failure forced a manual rerun + on-call investigation (memory: feedback_verify_live_not_just_local).

What

  • kubectl rollout status … --timeout=180s--timeout=300s
  • Healthz verify loop 6×5s → 12×5s (60s) so the public LB has time to route to the new pods if rollout took the full window

Surface (rule 22)

Same change shipping simultaneously in:

Coverage (rule 17)

YAML-only, no binary change. CI itself is the gate — the next master deploy will exercise the new timeout. No new unit tests required (no Go diff).

Symptom: `error: timed out waiting for the condition` after 180s, despite Ready pods
Enumeration: `grep -rn 'timeout=' .github/workflows/` → 1 hit per repo
Sites found: 3 (api, worker, provisioner)
Sites touched: 3
Coverage test: n/a — workflow change
Live verified: pending next master deploy

Today's #198 deploy timed out at 180s even though the new ReplicaSet
came up healthy (rollout actually took ~240s on a fresh node with a
cold image pull). The job exited red, prod was correct, and the
false-failure forced a manual rerun + on-call investigation.

Bumping to 300s removes the rerun cycle. Same change ships
simultaneously in worker + provisioner deploy.yml.

Healthz verify loop extended 6×5s → 12×5s for the same reason: if
rollout takes the full timeout, the new pods are only just live when
verify starts and the public LB needs a few seconds to route.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mastermanas805
mastermanas805 force-pushed the ci/bump-rollout-timeout-300s branch from 10ad4dc to 3c9da05 Compare May 30, 2026 11:22
@mastermanas805
mastermanas805 merged commit fe6d391 into master May 30, 2026
14 checks passed
@mastermanas805
mastermanas805 deleted the ci/bump-rollout-timeout-300s branch May 30, 2026 11:38
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.

1 participant