Skip to content

Notify on manual PR dashboard full-run failures#42

Merged
trask merged 2 commits into
open-telemetry:mainfrom
trask:notify-pr-dashboard-manual-full-run
Jul 7, 2026
Merged

Notify on manual PR dashboard full-run failures#42
trask merged 2 commits into
open-telemetry:mainfrom
trask:notify-pr-dashboard-manual-full-run

Conversation

@trask

@trask trask commented Jul 7, 2026

Copy link
Copy Markdown
Member

Follow-up to #39

Extends the PR dashboard failure notification to manual full rebuilds by treating empty repository and PR inputs as the full-run signal. This keeps filtered manual refreshes excluded while matching the scheduled rebuild behavior.

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

Extends the existing PR dashboard failure-notification workflow so that manual full rebuild runs (via workflow_dispatch) also trigger the failure tracking issue behavior, not just scheduled hourly rebuilds.

Changes:

  • Updates the notify-hourly-failure job condition to treat empty repository and pr_number inputs as the “full rebuild” signal.
  • Adds inline documentation clarifying the “empty inputs == full rebuild” convention.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/pull-request-dashboard.yml Outdated

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@trask trask marked this pull request as ready for review July 7, 2026 19:48
@trask trask requested a review from a team as a code owner July 7, 2026 19:48
@trask trask merged commit 8e356ff into open-telemetry:main Jul 7, 2026
5 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