Skip to content

feat(ci): add weekly on-call Slack notification workflow - #13168

Merged
mekarpeles merged 1 commit into
internetarchive:masterfrom
jimchamp:on-call-notifications
Jul 16, 2026
Merged

feat(ci): add weekly on-call Slack notification workflow#13168
mekarpeles merged 1 commit into
internetarchive:masterfrom
jimchamp:on-call-notifications

Conversation

@jimchamp

Copy link
Copy Markdown
Collaborator

Posts an on-call announcement every Monday and a deployment-lead reminder every Tuesday, @mentioning people from the roster in pm_config.json. Lead/secondary rotate weekly, computed from days-since-Unix-epoch rather than ISO week number so the rotation doesn't glitch at year boundaries -- no stored index needed.

Adds notify_slack.yml, a reusable workflow wrapping slackapi/slack-github-action for posting (and optionally threading replies to) a Slack message, so future workflows can publish to Slack without duplicating that logic.

Copilot AI review requested due to automatic review settings July 16, 2026 00:44

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.

Pull request overview

Adds GitHub Actions automation for posting weekly PM on-call and deployment-lead reminders to Slack, backed by a roster in-repo and a reusable Slack publishing workflow to avoid duplicating Slack posting logic across future workflows.

Changes:

  • Adds a scheduled workflow to compute a weekly lead/secondary rotation and post Monday/Tuesday reminders to Slack.
  • Adds a reusable workflow_call Slack publisher that can post a parent message and optional threaded replies.
  • Introduces pm_config.json to define the on-call personnel roster (Slack @mention IDs).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/pm_on_call_notification.yml Scheduled Monday/Tuesday workflow that builds a rotation-based message and calls the reusable Slack notifier.
.github/workflows/notify_slack.yml Reusable workflow wrapping slackapi/slack-github-action to post a parent message and optional threaded replies.
.github/workflows/config/pm_config.json On-call roster configuration used by the scheduled workflow.
Comments suppressed due to low confidence (3)

.github/workflows/pm_on_call_notification.yml:52

  • This file ends with a blank line that contains trailing spaces (line 52). Trailing whitespace tends to trip pre-commit/linters; please remove the spaces so the file ends cleanly.
      message: ${{ needs.build_message.outputs.message }}
    secrets:
      slack_token: ${{ secrets.SLACK_TOKEN }}

.github/workflows/notify_slack.yml:95

  • This workflow ends with trailing whitespace on the final blank line (line 95). Removing it avoids trailing-whitespace lint failures.
            text: ${{ toJson(matrix.reply) }}


.github/workflows/config/pm_config.json:19

  • This JSON file appears to end with a blank line containing trailing spaces (line 19). Please remove the trailing whitespace so it ends cleanly.
}

Comment thread .github/workflows/pm_on_call_notification.yml
Comment thread .github/workflows/notify_slack.yml Outdated
Posts an on-call announcement every Monday and a deployment-lead
reminder every Tuesday, @mentioning people from the roster in
pm_config.json. Lead/secondary rotate weekly, computed from
days-since-Unix-epoch rather than ISO week number so the rotation
doesn't glitch at year boundaries -- no stored index needed.

Adds notify_slack.yml, a reusable workflow wrapping
slackapi/slack-github-action for posting (and optionally
threading replies to) a Slack message, so future workflows can
publish to Slack without duplicating that logic.
@jimchamp
jimchamp force-pushed the on-call-notifications branch from d94f315 to 60b94c8 Compare July 16, 2026 01:10

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I intend to move all configurations for project management workflows to this file.

@mekarpeles mekarpeles self-assigned this Jul 16, 2026
@mekarpeles
mekarpeles merged commit fad93c8 into internetarchive:master Jul 16, 2026
4 checks passed
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