Skip to content

Correct the service-level min-instances runbook flag#3744

Draft
anth-volk wants to merge 1 commit into
masterfrom
fix/service-min-instances-runbook
Draft

Correct the service-level min-instances runbook flag#3744
anth-volk wants to merge 1 commit into
masterfrom
fix/service-min-instances-runbook

Conversation

@anth-volk

Copy link
Copy Markdown
Collaborator

Summary

Executing the Stage 3 post-merge runbook surfaced that its command was wrong: gcloud run services update --min-instances 1 sets the revision-level minimum — the exact per-tag warm-16Gi cost bomb the surrounding section warns against. The service-level flag is --min (verified against current Cloud Run docs).

The runbook now documents the correct flag, the gcloud-version caveat (older SDKs lack --min; the YAML-replace fallback sets run.googleapis.com/minScale on service metadata and requires the Cloud Resource Manager API), and records that the step was executed 2026-07-08 via that fallback.

Docs + changelog only; no runtime changes. (The actual production service already carries the correct service-level annotation — verified: template minScale absent, service metadata minScale=1.)


🤖 Generated with Claude Code

The runbook documented gcloud run services update --min-instances 1 as
the service-level warm-capacity command, but that flag sets the
REVISION-level minimum — the exact per-tag warm-instance cost bomb the
same section warns against. The service-level flag is --min (recent
gcloud only; the YAML-replace fallback and its Cloud Resource Manager
API prerequisite are now documented). Caught while executing the
runbook for the Stage 3 exit gates.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.89%. Comparing base (08176b4) to head (4f42610).
⚠️ Report is 12 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3744   +/-   ##
=======================================
  Coverage   79.89%   79.89%           
=======================================
  Files          70       70           
  Lines        4332     4332           
  Branches      807      807           
=======================================
  Hits         3461     3461           
  Misses        652      652           
  Partials      219      219           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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