Skip to content

[Fix] Publish GHCR Images fails on unauthorized roomote-web:develop pull - #118

Closed
roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/ghcr-deployment-acceptance-login-13ip6x46rlwsa
Closed

[Fix] Publish GHCR Images fails on unauthorized roomote-web:develop pull#118
roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/ghcr-deployment-acceptance-login-13ip6x46rlwsa

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

Created by Roomote. Follow up by mentioning @roomote, in the web UI, or in Slack.

What changed

Deployment acceptance on Publish GHCR Images was failing while pulling the previous develop channel for upgrade validation, with an unauthorized error on roomote-web:develop (and the other newer split app packages). The job already had packages: read, but unlike the build and publish jobs it never logged Docker into GHCR before pnpm deployment:smoke.

This change adds the same GHCR Docker login step to the deployment acceptance job so baseline package tags can be pulled with GITHUB_TOKEN before the smoke suite runs.

Why this change was made

Acceptance gates every develop publish and blocks image builds when it fails. After split service images landed, several package names defaulted to private visibility; anonymous pulls return unauthorized even when the tag exists. Authenticated pulls match the permission the workflow already declares and unblock the gate.

Impact

No product behavior change. Publish GHCR Images on develop should get past Deployment acceptance baseline pulls and continue into build/publish again. Full smoke still depends on the next germane CI run because Docker is not available in this task environment.

Deployment acceptance must log in with GITHUB_TOKEN so private package
channel tags like roomote-web:develop can be pulled for upgrade checks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants