branderson/dotcom-arcade
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

CI: log in to the registry as the repository owner #4

Merged
branderson merged 1 commits from ci-registry-login into main 2026-10-13 23:19:13 -07:00
Conversation 1 Commits 1 Files Changed 2
+6 -7
dejima-agent commented 2026-10-13 23:17:05 -07:00
Collaborator
Copy Link
Copy Source

Summary generated by LLM agent

The image builds failed at docker login. The workflow read the username from a REGISTRY_USERNAME secret, which was never created, so it logged in with an empty username.

The login now uses the repository owner as the username, with the existing REGISTRY_TOKEN secret: a user-level token with package write access, belonging to the repository's owner. The workflow header and the deployment docs now describe that single secret.

If the repository ever moves to an organization, the token's owner would no longer match, and the username would need its own setting again.

The workflow still parses. The real test is the run this triggers once merged to main.

**Summary generated by LLM agent** The image builds failed at `docker login`. The workflow read the username from a `REGISTRY_USERNAME` secret, which was never created, so it logged in with an empty username. The login now uses the repository owner as the username, with the existing `REGISTRY_TOKEN` secret: a user-level token with package write access, belonging to the repository's owner. The workflow header and the deployment docs now describe that single secret. If the repository ever moves to an organization, the token's owner would no longer match, and the username would need its own setting again. The workflow still parses. The real test is the run this triggers once merged to `main`.
dejima-agent added 1 commit 2026-10-13 23:17:06 -07:00
ci: log in to the registry as the repository owner e1cf8aff29 
Only REGISTRY_TOKEN is configured; the REGISTRY_USERNAME secret the
login step read doesn't exist, so it logged in with an empty username.
branderson approved these changes 2026-10-13 23:19:09 -07:00
branderson merged commit 0d08cde3c1 into main 2026-10-13 23:19:13 -07:00
Sign in to join this conversation.
Reviewers
No Reviewers
branderson
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
dejima-agent branderson
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: branderson/dotcom-arcade#4