Commit a696866
committed
fix: chain docker publish from release workflow
GitHub's anti-recursion rule prevents workflows from triggering other
workflows when acting as GITHUB_TOKEN. publish-release.yml pushes the
release tag with the default token, so the push:tags:v* trigger on
publish-docker.yml never fires. Result: v1.11.0 through v1.14.3 all
published to npm but no Docker image reached GHCR, and the v1.14.3 OOM
fix never made it to the Railway service pulling ghcr.io/copilotkit/aimock:latest.
Fix by explicitly dispatching publish-docker.yml via `gh workflow run
--ref <TAG>` after the npm publish step succeeds. This does not depend
on tag-trigger behavior and is guarded by the same
steps.check.outputs.published == 'false' condition as the publish, so
Docker only builds when npm publish actually happened.
Also restore the workflow_dispatch trigger on publish-docker.yml (added
in 486ccd9, inadvertently removed in 63aab1e); `gh workflow run`
requires workflow_dispatch on the ref being invoked.
The existing push:tags:v* trigger stays as belt-and-suspenders for
anyone pushing tags manually from a local clone with a PAT.
Option chosen: explicit chain over PAT or workflow_call. Least invasive
(no new secrets, no cross-workflow refactor), most observable (the
dispatch shows up as a discrete step in the release run), and the
failure mode is clearly visible in the release workflow log.1 parent db21dcd commit a696866
2 files changed
Lines changed: 13 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
10 | 11 | | |
11 | 12 | | |
12 | 13 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| 13 | + | |
13 | 14 | | |
14 | 15 | | |
15 | 16 | | |
| |||
81 | 82 | | |
82 | 83 | | |
83 | 84 | | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
84 | 96 | | |
85 | 97 | | |
86 | 98 | | |
| |||
0 commit comments