Skip to content

ci: notify Slack when nightly scenario reports are posted - #229

Merged
Kubuxu merged 1 commit into
mainfrom
ci/nightly-slack-reports
Nov 10, 2026
Merged

Kubuxu merged 1 commit into
mainfrom
ci/nightly-slack-reports

Conversation

@Kubuxu

@Kubuxu Kubuxu commented Nov 10, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • Enable Slack notifications for nightly CI profiles using the SLACK_WEBHOOK_URL repository Actions secret.
  • Follow issue reporting rules: notify only after a report is posted, respecting reporting-disabled and skip-on-pass settings.
  • Capture the exact report comment URL, or the issue URL for its first report, using actions/github-script.
  • Keep Slack disabled by default for other reusable workflow callers.

Verification

  • Parsed both workflow YAML files.
  • Exercised report creation and comment URL capture with a mocked GitHub API.
  • Exercised pass/fail and skip-on-pass reporting decisions.
  • Executed the Slack notification shell against a local HTTP endpoint, including HTTP rejection and missing-secret cases.
  • Live GitHub Actions and Slack delivery were not exercised.

Setup

Set repository Actions secret SLACK_WEBHOOK_URL to the full Slack incoming webhook URL.

@Kubuxu
Kubuxu requested review from beck-8 and rvagg as code owners November 10, 2026 15:47
Copilot AI balanced review requested due to automatic review settings November 10, 2026 15:47
@FilOzzy FilOzzy added this to FOC Nov 10, 2026
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FOC Nov 10, 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

No active review issues are identified for the nightly Slack notification changes.

No open issues 🎉

What changed in this PR

Adds Slack notifications for nightly scenario reports, linking each notification to the posted GitHub issue or comment.

Changes:

  • Enables Slack notifications for nightly profiles while leaving them disabled by default for other reusable workflow callers.
  • Uses actions/github-script to capture the report URL.
  • Sends notifications only after successful report posting, respecting reporting-disabled and skip-on-pass settings.
File Description
.github/​workflows/​ci_run.yml Adds optional Slack notifications and captures the posted report URL.
.github/​workflows/​ci_nightly.yml Enables Slack notifications for nightly profiles and documents the required secret.

Review effort: Balanced

@Kubuxu
Kubuxu merged commit 88aed64 into main Nov 10, 2026
5 checks passed
@Kubuxu
Kubuxu deleted the ci/nightly-slack-reports branch November 10, 2026 15:49
@github-project-automation github-project-automation Bot moved this from 📌 Triage to 🎉 Done in FOC Nov 10, 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