Skip to content

Fix/ci release docs deploy#5878

Merged
jyotshnayaparla-00 merged 1 commit intomainfrom
fix/ci-release-docs-deploy
Mar 20, 2026
Merged

Fix/ci release docs deploy#5878
jyotshnayaparla-00 merged 1 commit intomainfrom
fix/ci-release-docs-deploy

Conversation

@jyotshnayaparla-00
Copy link
Contributor

actions/create-github-app-token@v2 resolved to v2.2.2 which bumped @actions/core from 1.11.1 to 3.0.0. The 3.0.0 release is now ESM-only, which causes the app token to not be set correctly as a step output.

The docs deploy step then falls back to GITHUB_TOKEN (github-actions[bot]), which doesn't have write access to feldera/docs.feldera.com, resulting in a 403 and skipping Docker image tagging.

@jyotshnayaparla-00 jyotshnayaparla-00 requested a review from gz March 20, 2026 00:26
v2.2.2 bumped @actions/core from 1.11.1 to 3.0.0 which is now ESM-only.
This causes the app token output to not be set correctly, so the docs
deploy step falls back to GITHUB_TOKEN (github-actions[bot]) which doesn't
have write access to feldera/docs.feldera.com, causing a 403 and skipping
Docker image tagging.
@jyotshnayaparla-00 jyotshnayaparla-00 force-pushed the fix/ci-release-docs-deploy branch from a096ee6 to 511ebc7 Compare March 20, 2026 00:27
Copy link

@mythical-fred mythical-fred left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Pinning to a specific SHA is correct practice here.

@jyotshnayaparla-00 jyotshnayaparla-00 added this pull request to the merge queue Mar 20, 2026
Merged via the queue into main with commit bbb7451 Mar 20, 2026
1 check passed
@jyotshnayaparla-00 jyotshnayaparla-00 deleted the fix/ci-release-docs-deploy branch March 20, 2026 08:25
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.

3 participants