Skip to content

Commit e58b955

Browse files
authored
[ci] Set MARIN_PREFIX on TPU canary validate step (#4854)
The TPU canary's validate step runs off-GCP with mirror:// but no MARIN_PREFIX, so rigging.filesystem falls back to /tmp/marin and the remote-prefix scan is skipped, raising FileNotFoundError on tracker_metrics.jsonl. Runner is already authenticated to GCP; setting MARIN_PREFIX=gs://marin-us-central1 lets mirror:// resolve the output in either v5p region.
1 parent 82d2359 commit e58b955

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

.github/workflows/marin-canary-ferry.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,16 @@ jobs:
132132
- name: Validate canary metrics
133133
continue-on-error: ${{ inputs.target_tokens != '' }}
134134
shell: bash -l {0}
135+
# MARIN_PREFIX is required so mirror:// knows which gs://marin-* bucket
136+
# counts as "local" — without it the runner falls back to /tmp/marin,
137+
# _mirror_remote_prefixes() returns [] (to avoid anonymous-caller 401s
138+
# from gcsfs off-GCP), and the read raises FileNotFoundError. The
139+
# google-github-actions/auth step above provides the credentials
140+
# gcsfs needs to actually scan the remote buckets. Canary lands on v5p
141+
# in us-central1 or us-east5; picking us-central1 keeps the hit rate
142+
# high and lets mirror:// resolve either region.
143+
env:
144+
MARIN_PREFIX: gs://marin-us-central1
135145
run: .venv/bin/python scripts/canary/validate_canary_metrics.py
136146

137147
- name: Summarize TPU canary profile

0 commit comments

Comments
 (0)