hotfix(docs): Harbor registry is public — drop docker login from pull instructions - #417
Merged
Merged
Conversation
… instructions The airstack project on airlab-docker.andrew.cmu.edu is now public (verified: anonymous token grant, tag list, manifest and blob fetch all succeed with no credentials), so pulling images no longer requires an AirLab account. Remove the docker login step from all pull instructions (getting started, CLI docker usage, Isaac Sim / MS-AirSim image management) and note that pushing still requires login. VERSION 0.20.4 → 0.20.5 (gate). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This was referenced Aug 29, 2026
andrewjong
added a commit
that referenced
this pull request
Aug 29, 2026
… them for all hotfixes (#418) * docs(release-notes): dated sections for 0.20.1–0.20.5; require them for all hotfixes Add a dated release-notes section for this hotfix (0.20.5, public registry pulls) and backfill 0.20.1–0.20.4, which landed without sections. Codify the rule in the bump-version-and-release skill and the AGENTS.md skill table: every hotfix on main — docs-only and CI-only included — adds its own dated '## X.Y.Z — YYYY-MM-DD' patch-notes section, since the MAJOR.MINOR docs build renders all of a line's patch sections together. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * hotfix(docs): dated release-notes sections for 0.20.1–0.20.5; require them for all hotfixes Follow-up to #417, whose release-notes/skill commit was pushed after the PR had already merged. Adds the dated patch-notes sections for hotfixes 0.20.1–0.20.5 (none had one) and codifies the rule in the bump-version-and-release skill and the AGENTS.md skill table: every hotfix on main adds its own dated '## X.Y.Z — YYYY-MM-DD' section. VERSION 0.20.5 → 0.20.6 (gate). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Contributor
Test Metrics —
|
| Test | Pass | Fail | Skip | Rate |
|---|---|---|---|---|
| test_colcon_build_gcs | 1 | 0 | 0 | 100% |
| test_colcon_build_ms_airsim | 1 | 0 | 0 | 100% |
| test_colcon_build_robot | 1 | 0 | 0 | 100% |
| test_colcon_test_robot | 1 | 0 | 0 | 100% |
Metrics
| Test | Metric | Value |
|---|---|---|
| test_colcon_build_robot | duration_s | 132.9s |
| test_colcon_test_robot | duration_s | 45.76s |
| test_colcon_build_gcs | duration_s | 75.78s |
| test_colcon_build_ms_airsim | duration_s | 15.26s |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The
airstackproject on the AirLab Harbor registry (airlab-docker.andrew.cmu.edu) is now public, so images can be pulled without an account. This PR updates every user-facing pull instruction that still told readers todocker loginwith an Andrew ID.Verified anonymously (no credentials anywhere in the chain):
"public": "true"for the projectdocker pullusesChanges
docs/getting_started/index.md— removed the login step from Option 1 (pull); reworded the alpha warning (account needed only for push / Nucleus / internal resources); addeddocker loginbeforeairstack images pushin Option 2docs/development/beginner/airstack-cli/docker_usage.md— pull section no longer requires login; push note addeddocs/simulation/isaac_sim/container_workflows.md— dropped login from image pull snippetdocs/simulation/ms-airsim/docker.md— dropped login from image pull snippet.env— VERSION 0.20.4 → 0.20.5 (version gate; docs-only, so docker-build should retag rather than rebuild)docs/release_notes/index.md— new dated## 0.20.5section for this hotfix, plus backfilled dated sections for hotfixes 0.20.1–0.20.4 (they landed without notes).agents/skills/bump-version-and-release/SKILL.md+AGENTS.md— codified the rule: every hotfix onmain(docs/CI-only included) adds its own dated## X.Y.Z — YYYY-MM-DDpatch-notes sectionLeft unchanged: OSMO credential setup docs and CI cache docs — those cover credentialed push/CI flows that still require login.
🤖 Generated with Claude Code