Skip to content

Commit 8f4e2b6

Browse files
yonromaiyoblinclaude
authored
fix(iris): strip trailing slash from MirrorFileSystem prefix (#4206)
## Summary - Strip trailing slash from `MirrorFileSystem._local_prefix` to prevent double-slash paths (`s3://bucket/prefix//path`) - Fixes GPU canary validation failing with `FileNotFoundError` / GCS 401 when `MARIN_PREFIX` ends with `/` ## Test plan - [x] Local repro: validation fails before fix, passes after with `MARIN_PREFIX=s3://marin-na/marin/` - [ ] GPU canary `workflow_dispatch` with validation step passing end-to-end 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: yoblin <268258002+yoblin@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 17bb68a commit 8f4e2b6

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/iris/src/iris/marin_fs.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -718,7 +718,7 @@ def __init__(
718718
**kwargs: Any,
719719
):
720720
super().__init__(*args, **kwargs)
721-
self._local_prefix = marin_prefix()
721+
self._local_prefix = marin_prefix().rstrip("/")
722722
self._remote_prefixes = [p for p in _all_data_bucket_prefixes() if not self._local_prefix.startswith(p)]
723723
self._budget = budget if budget is not None else _global_transfer_budget
724724
self._worker_id = default_worker_id()

0 commit comments

Comments
 (0)