Skip to content

Add failure issue notifications#8583

Open
trask wants to merge 2 commits into
open-telemetry:mainfrom
trask:remove-unused-failure-workflow
Open

Add failure issue notifications#8583
trask wants to merge 2 commits into
open-telemetry:mainfrom
trask:remove-unused-failure-workflow

Conversation

@trask

@trask trask commented Jul 10, 2026

Copy link
Copy Markdown
Member

Add failure issue notifications to scheduled workflows and workflows that run only on pushes to main, making failures visible to the full team instead of relying on GitHub’s limited scheduled-workflow notifications.

@trask trask requested a review from a team as a code owner July 10, 2026 18:16
@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.61%. Comparing base (6ee54e5) to head (b925fac).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #8583   +/-   ##
=========================================
  Coverage     91.61%   91.61%           
- Complexity    10310    10312    +2     
=========================================
  Files          1013     1013           
  Lines         27260    27260           
  Branches       3201     3201           
=========================================
+ Hits          24973    24975    +2     
+ Misses         1560     1559    -1     
+ Partials        727      726    -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@trask trask changed the title Remove unused failure issue workflow Add failure issue notifications Jul 10, 2026
@trask trask marked this pull request as draft July 10, 2026 19:02
@trask trask requested a review from Copilot July 10, 2026 19:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR standardizes “workflow failure -> tracking issue” notifications by wiring several scheduled and main-only workflows to the open-telemetry/shared-workflows reusable workflow that opens/comments/closes a GitHub issue based on the upstream job result, improving visibility beyond GitHub’s limited scheduled-workflow notifications.

Changes:

  • Add a workflow-notification job to multiple workflows that calls open-telemetry/shared-workflows/.github/workflows/workflow-failure-issue.yml@v0.3.1 and passes a computed success boolean from needs.*.result.
  • Update existing failure-notification usage to shared-workflows v0.3.1 (pinned by SHA).
  • Remove the now-unused local reusable workflow .github/workflows/reusable-open-issue-on-failure.yml.

Reviewed changes

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

Show a summary per file
File Description
.github/workflows/sonatype-guide-dependency-audit-daily.yml Update failure-issue notification to shared-workflows v0.3.1 and simplify the notification job condition.
.github/workflows/reusable-open-issue-on-failure.yml Delete local reusable workflow previously used to open an issue via gh.
.github/workflows/ossf-scorecard.yml Add failure-issue notification job tied to the analysis job result.
.github/workflows/javadoc-crawler.yml Add failure-issue notification job tied to the crawl job result.
.github/workflows/issue-management-stale-action.yml Add failure-issue notification job tied to the stale job result.
.github/workflows/fossa.yml Add failure-issue notification job tied to the fossa job result.
.github/workflows/codeql.yml Fix indentation for category and add schedule-only failure-issue notification tied to analyze.
.github/workflows/build-tracecontext-testsuite.yml Add failure-issue notification job tied to the publish job result.
.github/workflows/build-daily.yml Update failure-issue notification to shared-workflows v0.3.1 and simplify the notification job condition.
.github/workflows/benchmark.yml Add failure-issue notification job tied to the sdk-benchmark job result.

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

@trask trask marked this pull request as ready for review July 10, 2026 19:10
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.

2 participants