[WT-1042] GHA release trigger fine-tuning#17141
Merged
stevejalim merged 4 commits intomainfrom Apr 17, 2026
Merged
Conversation
Include the git log --oneline output (origin/prod..HEAD) in the #www Slack notification, not just the count.
GitHub does not fire push-triggered workflows when the push is made using GITHUB_TOKEN. Using a PAT (BEDROCK_GHA_RELEASE_WORKFLOW_PAT) means the push is attributed to a real user and build-and-push.yml fires as expected.
stevejalim
force-pushed
the
WT-1042--gha-release-trigger--amends
branch
from
c7f6493 to
2f16680
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Bedrock GitHub Actions release workflow to improve release notifications and ensure downstream workflows are triggered when pushing stage/prod refs.
Changes:
- Include a commit log summary in the initial Slack “release started” message.
- Add PAT-based authentication to
actions/checkoutin stage/prod deploy jobs so branch/tag pushes trigger downstream workflows.
- Use jq to properly JSON-encode the commit log so commit subjects containing quotes or backslashes don't break the Slack payload - Truncate raw log before encoding to avoid splitting escape sequences - Use printf instead of echo for predictable backslash handling
stevejalim
changed the title
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refines the release GitHub Actions workflow by improving Slack notifications with a commit list and ensuring release branch/tag pushes correctly trigger downstream workflows (e.g., build-and-push.yml).
Changes:
- Capture and include the pending commit log in the “starting release” Slack message, with escaping intended to make it safe for Slack’s Block Kit JSON payload.
- Add a fallback
commit_logoutput whenorigin/prodis unavailable. - Switch
actions/checkoutto use a PAT for the stage/prod deploy jobs so pushes trigger downstream workflow runs.
Comment on lines
+76
to
+78
| # otherwise break the JSON. Truncate the raw log before encoding to avoid | ||
| # splitting escape sequences, leaving headroom for Slack's 3000-char block limit | ||
| # after the prefix text the Slack action prepends. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
If this changeset needs to go into the FXC codebase, please add the
WMO and FXClabel.One-line summary
Significant changes and points to review
Issue / Bugzilla link
Testing