fix(ci): derive REGISTRY_USER from GITHUB_REPOSITORY_OWNER #4

Merged
David merged 2 commits from fix/ci-registry-user-from-github-owner into main 2026-05-13 11:52:27 +02:00
Owner

Summary

After PR #3 swapped REGISTRY_PASSWORD to secrets.FORGEJO_PAT, the workflow failed at docker login with username is empty. Cause: REGISTRY_USER was being set from vars.A8N_TOOLS_PRIVATE_PACKAGE_OWNER, which is unset for this repo. Switch to deriving the owner from $env.GITHUB_REPOSITORY_OWNER (lowercased), matching the sibling da-info workflow that also uses FORGEJO_PAT.

Test plan

  • Trigger the workflow (tag push or branch push touching watched paths); confirm docker login, image push, and Generic Package upload all succeed.
## Summary After PR #3 swapped `REGISTRY_PASSWORD` to `secrets.FORGEJO_PAT`, the workflow failed at `docker login` with `username is empty`. Cause: `REGISTRY_USER` was being set from `vars.A8N_TOOLS_PRIVATE_PACKAGE_OWNER`, which is unset for this repo. Switch to deriving the owner from `$env.GITHUB_REPOSITORY_OWNER` (lowercased), matching the sibling `da-info` workflow that also uses `FORGEJO_PAT`. ## Test plan - [ ] Trigger the workflow (tag push or branch push touching watched paths); confirm `docker login`, image push, and Generic Package upload all succeed.
fix(ci): derive REGISTRY_USER from GITHUB_REPOSITORY_OWNER
All checks were successful
Check / fmt + clippy + build + tests (pull_request) Successful in 6s
7471b4feaa
A8N_TOOLS_PRIVATE_PACKAGE_OWNER is unset for this repo, so REGISTRY_USER resolved to an empty string and docker login failed with "username is empty". Drop the var lookup and use $env.GITHUB_REPOSITORY_OWNER (lowercased), matching the sibling da-info workflow now that login uses FORGEJO_PAT.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Merge branch 'main' into fix/ci-registry-user-from-github-owner
All checks were successful
Check / fmt + clippy + build + tests (pull_request) Successful in 6s
Create release / Create release from merged PR (pull_request) Has been skipped
ac894fc194
David merged commit 902cb4a5c7 into main 2026-05-13 11:52:27 +02:00
David deleted branch fix/ci-registry-user-from-github-owner 2026-05-13 11:52:27 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
pandoras-box/youtrack-cli!4
No description provided.