Skip to content

fix(vertex): switch CLOUD_ML_REGION default from us-east5 to global#1453

Merged
mergify[bot] merged 2 commits intomainfrom
ambient/switch-cloud-ml-region-to-global
Apr 23, 2026
Merged

fix(vertex): switch CLOUD_ML_REGION default from us-east5 to global#1453
mergify[bot] merged 2 commits intomainfrom
ambient/switch-cloud-ml-region-to-global

Conversation

@ambient-code
Copy link
Copy Markdown
Contributor

@ambient-code ambient-code Bot commented Apr 23, 2026

Summary

  • Remove the backend override in getAnthropicClient() that forced CLOUD_ML_REGION=global back to us-east5 — the original constraint (claude-haiku-4-5 not available in global) no longer applies
  • Update the control-plane config default, MPP-OpenShift overlay, CI workflow, scripts, and docs to consistently use global instead of us-east5
  • Aligns all components with the production ConfigMap that already sets CLOUD_ML_REGION: "global"

Changes

File Change
components/backend/handlers/display_name.go Remove global → us-east5 override; default to global when env var is empty
components/ambient-control-plane/internal/config/config.go Change fallback default from us-east5 to global
components/manifests/overlays/mpp-openshift/ambient-control-plane.yaml Update manifest value to global
.github/workflows/pull-reviews.yml Update gcp_region to global
.github/scripts/model-discovery.py Update example in docstring
scripts/setup-vertex-kind.sh Update examples and error messages
docs/ Update documentation references
components/backend/handlers/display_name_test.go Update test to use global

Test plan

  • Verify backend display name generation works with CLOUD_ML_REGION=global in staging
  • Confirm claude-haiku-4-5 model calls succeed via the global endpoint
  • Verify control-plane reconciler passes CLOUD_ML_REGION=global to runner pods
  • Monitor Vertex AI costs after rollout to confirm regional premium is eliminated

Closes #1451


🤖 Ambient Session

Remove the backend override that forced `global` → `us-east5` in
getAnthropicClient(). The original constraint (claude-haiku-4-5 not
available in global region) no longer applies. Update the control-plane
config default, MPP overlay, CI workflow, scripts, and docs to use
`global` consistently — aligning with the production ConfigMap that
already sets CLOUD_ML_REGION=global.

Closes #1451

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ambient-code ambient-code Bot added ambient-code:managed PR managed by AI automation ambient-code:self-reviewed Self-reviewed by Ambient agent labels Apr 23, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 23, 2026

Deploy Preview for cheerful-kitten-f556a0 canceled.

Name Link
🔨 Latest commit 64c2d13
🔍 Latest deploy log https://app.netlify.com/projects/cheerful-kitten-f556a0/deploys/69eaabdee39d0e00081c1270

@mergify mergify Bot added the queued label Apr 23, 2026
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 23, 2026

Merge Queue Status

  • Entered queue2026-04-23 23:44 UTC · Rule: default
  • Checks skipped · PR is already up-to-date
  • Merged2026-04-23 23:44 UTC · at 64c2d1391244f5d4b475fae0f577c05bb6397210 · squash

This pull request spent 9 seconds in the queue, including 1 second running CI.

Required conditions to merge

@mergify mergify Bot merged commit 85af3da into main Apr 23, 2026
71 checks passed
@mergify mergify Bot deleted the ambient/switch-cloud-ml-region-to-global branch April 23, 2026 23:44
@mergify mergify Bot removed the queued label Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ambient-code:managed PR managed by AI automation ambient-code:self-reviewed Self-reviewed by Ambient agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Evaluate switching CLOUD_ML_REGION from us-east5 to global for Vertex AI cost optimization

0 participants