Skip to content

ci(release): notify Slack when a prerelease succeeds - #4605

Merged
matthewgrossman merged 2 commits into
mainfrom
ci/4604-prerelease-success-slack/matthewgrossman
Oct 15, 2026
Merged

matthewgrossman merged 2 commits into
mainfrom
ci/4604-prerelease-success-slack/matthewgrossman

Conversation

@matthewgrossman

Copy link
Copy Markdown
Member

Summary

Post a top-level Slack message when a prerelease succeeds, so it's visible in the channel when a release recovers after a run of failures.

Related Issue

Closes #4604

Changes

  • Add a notify-prerelease-success job to release-tag.yml, mirroring notify-prerelease-failure: it gates on success() of release-helm and publish-qualification, runs only for -pre. tags, posts to the same SLACK_OPENSHELL_TRIAGE_WEBHOOK_URL, never blocks the release, and is skipped when the webhook is unset.
  • The success message omits the triage mention to keep notifications low.
  • Document the success notification in CI.md.

Testing

  • Workflow YAML parses; zizmor reports no new medium/high findings
  • Unit tests added/updated (if applicable)
  • E2E tests added/updated (if applicable)

Not yet exercised against a live prerelease run.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

Post a top-level message to the triage webhook when a prerelease completes, so recovery after failures is visible. The message omits the triage mention.

Closes #4604

Signed-off-by: Matthew Grossman <mgrossman@nvidia.com>
@github-actions

Copy link
Copy Markdown

Signed-off-by: Matthew Grossman <mgrossman@nvidia.com>

@purp purp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🚢

@matthewgrossman
matthewgrossman added this pull request to the merge queue Oct 15, 2026
Merged via the queue into main with commit c3ca45f Oct 15, 2026
81 checks passed
@matthewgrossman
matthewgrossman deleted the ci/4604-prerelease-success-slack/matthewgrossman branch October 15, 2026 17:28
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.

ci(release): notify Slack when a prerelease succeeds

2 participants