Skip to content

fix(release): split docker push into internal + ghcr jobs#87

Merged
JustSamuel merged 1 commit into
mainfrom
ci/release-quoting
May 7, 2026
Merged

fix(release): split docker push into internal + ghcr jobs#87
JustSamuel merged 1 commit into
mainfrom
ci/release-quoting

Conversation

@JustSamuel

@JustSamuel JustSamuel commented May 7, 2026

Copy link
Copy Markdown
Collaborator

Why

The release on `feat: force release` failed at the GHCR push:

```
ERROR: failed to push ghcr.io/eou/plankapi:latest:
denied: permission_denied: The requested installation does not exist.
```

`docker-release.yml` reuses the single `docker-paths` for both the custom registry and GHCR (when `github-registry: "true"`). `eou/plankapi` is correct on the internal registry but invalid on GHCR — GHCR routes by GitHub org, and there's no `eou` org (the repo is under `github.com/GEWIS`).

What

Modern GEWIS/actions (since the 2026-03-31 `feat: add reusable release-pr and flexible docker workflows`) exposes a dedicated `docker-release-ghcr.yml` with its own `docker-paths`. So split the existing single `release` job into two parallel jobs:

Job Workflow Path
`release-internal` `docker-release.yml@v1` `abc.docker-registry.gewis.nl/eou/plankapi`
`release-ghcr` `docker-release-ghcr.yml@v1` `ghcr.io/gewis/plankapi`

Also includes the JSON-quoting fix (`'["."]'` instead of `"['.']"`) so the GEWIS workflow's strict `JSON.parse` accepts the inputs.

`fix:` type is intentional — guarantees semantic-release computes a patch bump and triggers a release on merge to main.

Test plan

  • After merge: release workflow on main fires
  • `release-internal` pushes to `abc.docker-registry.gewis.nl/eou/plankapi:1.3.x`
  • `release-ghcr` pushes to `ghcr.io/gewis/plankapi:1.3.x`

The GEWIS docker-release workflow now strictly JSON.parses the
`projects` and `docker-paths` inputs (`SyntaxError: Unexpected
token ', "['.']" is not valid JSON`). YAML-double-quoted strings
containing single-quoted JSON aren't valid JSON. Switch to
single-quoted YAML wrapping double-quoted JSON arrays, matching the
example in GEWIS/actions README.
@JustSamuel JustSamuel force-pushed the ci/release-quoting branch from 62552e0 to 8877766 Compare May 7, 2026 06:28
@JustSamuel JustSamuel merged commit 51f4e79 into main May 7, 2026
3 checks passed
@JustSamuel JustSamuel deleted the ci/release-quoting branch May 7, 2026 06:28
@JustSamuel JustSamuel changed the title ci(release): pass JSON arrays with double quotes fix(release): split docker push into internal + ghcr jobs May 7, 2026
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.

1 participant