Skip to content

Guard workflow failure issues in forks#71

Merged
trask merged 1 commit into
open-telemetry:mainfrom
trask:guard-workflow-failure-issues-in-forks
Jul 10, 2026
Merged

Guard workflow failure issues in forks#71
trask merged 1 commit into
open-telemetry:mainfrom
trask:guard-workflow-failure-issues-in-forks

Conversation

@trask

@trask trask commented Jul 10, 2026

Copy link
Copy Markdown
Member

Restrict the workflow failure issue job to callers owned by the open-telemetry organization. Reusable workflows execute in the caller's context, so this prevents fork runs from opening failure issues while allowing official OpenTelemetry repositories to share the guard instead of repeating repository-specific conditions.

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

Restricts the reusable “workflow failure issue” workflow so it only opens/comments/closes issues when the calling repository owner is the open-telemetry organization, preventing fork-owned repositories from creating failure-tracking issues.

Changes:

  • Adds a job-level if: github.repository_owner == 'open-telemetry' guard to skip the issue-management job outside the OpenTelemetry org.
  • Updates documentation to describe the new org-only execution behavior.

Reviewed changes

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

File Description
workflow-failure-issue/README.md Documents that the workflow only runs for callers owned by open-telemetry, preventing fork runs from creating issues.
.github/workflows/workflow-failure-issue.yml Adds an org-owner guard (if) to the job that manages workflow-failure issues.

💡 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 18:32
@trask trask requested a review from a team as a code owner July 10, 2026 18:32
@trask trask merged commit 7c49790 into open-telemetry:main Jul 10, 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