Skip to content

ci: notify on scheduled collector failure#2

Open
dmatking wants to merge 1 commit into
mainfrom
notify-on-failure
Open

ci: notify on scheduled collector failure#2
dmatking wants to merge 1 commit into
mainfrom
notify-on-failure

Conversation

@dmatking
Copy link
Copy Markdown
Owner

Summary

  • Adds an if: failure() step that opens a GitHub issue when the scheduled run fails (only on schedule, not on manual dispatch)
  • Dedupes via a collector-failure label — repeated failures append a comment to the existing issue rather than opening new ones
  • Grants the workflow issues: write permission

Context: the TRAFFIC_TOKEN PAT silently expired and the collector failed for 2 days before being noticed. This makes the next expiry visible.

Test plan

  • Merge, then revoke/rotate TRAFFIC_TOKEN locally and manually trigger via workflow_dispatch — confirm no issue (dispatch is excluded)
  • Wait for next scheduled run with bad token, or temporarily change the cron — confirm a single issue is opened
  • Re-run while broken — confirm a comment is appended, not a new issue

🤖 Generated with Claude Code

Opens a GitHub issue when the scheduled run fails (typically expired
TRAFFIC_TOKEN). Deduped via a label so repeated failures append a
comment to the existing issue rather than spamming new ones.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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.

1 participant