Skip to content

ci: add workflow-results and workflow-notifier to merge, release, and scheduled - #2909

Merged
DerekRoberts merged 4 commits into
mainfrom
ci/workflow-notifier
Oct 27, 2026
Merged

DerekRoberts merged 4 commits into
mainfrom
ci/workflow-notifier

Conversation

@DerekRoberts

@DerekRoberts DerekRoberts commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Failures in merge.yml, release.yml, and scheduled.yml had no rollup gate or failure alert.

  • Add Merge Results, Release Results, and Scheduled Results jobs using a single bcgov/actions/workflow-results step pinned to 091f1da89c669d36bf4899830381e5b12d2c889a # v0.11.0.
  • v0.11.0 bakes notification into workflow-results: with notify unset, it opens/updates a failure issue on non-PR events (merge, release, scheduled). No separate workflow-notifier step.
  • Job permissions: contents: read, issues: write, pull-requests: read.

@DerekRoberts
DerekRoberts marked this pull request as ready for review October 26, 2026 04:11
Copilot AI balanced review requested due to automatic review settings October 26, 2026 04:11
@DerekRoberts DerekRoberts self-assigned this Oct 26, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The additions cover all upstream jobs, use compatible action inputs and permissions, and have no identified blocking issues.

No open issues 🎉

What changed in this PR

Adds result rollups and failure notifications to the merge, release, and scheduled workflows, following the shared-action pattern already used elsewhere in the repository.

Changes:

  • Add always-running results jobs covering each workflow’s upstream jobs.
  • Notify on upstream failures using workflow-notifier.
  • Pin both shared actions to the existing v0.9.0 commit.
File Description
.github/​workflows/​scheduled.yml Adds Scheduled Results and failure notifications for maintenance and test jobs.
.github/​workflows/​release.yml Adds Release Results and failure notifications for the production release pipeline.
.github/​workflows/​merge.yml Adds Merge Results and failure notifications for the merge deployment pipeline.

Review effort: Balanced

@DerekRoberts DerekRoberts moved this from New to Active in DevOps (NR) Oct 27, 2026
@DerekRoberts
DerekRoberts merged commit cb66d37 into main Oct 27, 2026
22 of 24 checks passed
@DerekRoberts
DerekRoberts deleted the ci/workflow-notifier branch October 27, 2026 03:26
@github-project-automation github-project-automation Bot moved this from Active to Done in DevOps (NR) Oct 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants