Disable unowned AI GitHub Actions workflows#19933
Open
kcreddy wants to merge 1 commit into
Open
Conversation
The sweep-* and trigger-* workflows wrap reusable workflows from
elastic/ai-github-actions. These currently have no owner, so disable
them rather than deleting them — deletion would lose the config and
history, and someone may want to adopt and re-enable them later.
Disable via a source-level `if: false` on the `run` job (not the
Actions UI toggle) so that re-enabling requires a reviewed PR and is
visible in git history — a silent flip-back-on isn't possible.
Disabled 16 workflows:
sweep-dashboard-data-scope, sweep-field-mapping-conflicts,
sweep-httpjson-pagination, sweep-ingest-pipeline-safety,
sweep-pipeline-error-handling, trigger-breaking-change-detector,
trigger-bug-hunter, trigger-docs-patrol,
trigger-duplicate-issue-detector, trigger-issue-triage,
trigger-mention-in-issue, trigger-mention-in-pr,
trigger-newbie-contributor-patrol, trigger-pr-review,
trigger-text-auditor, trigger-stale-issues
For the 12 single-job workflows, `if: false` plus a comment was added
directly to the `run` job.
For the 4 workflows that already had an `if:` activation condition
(trigger-issue-triage, trigger-mention-in-issue, trigger-mention-in-pr,
trigger-pr-review), YAML permits only one `if:` key per job, so the
original condition was moved into a comment above the job and replaced
with `if: false`. The comment preserves the exact original expression
so it can be restored verbatim on re-enable.
Excluded (intentionally not disabled):
- trigger-package-tests-security-ml: actively maintained by the
security-ml team (and uses elastic/ci-gh-actions, not
elastic/ai-github-actions).
- trigger-pr-actions-detective: has an extra step-level job
(actions/github-script) before the reusable-workflow call, so it
needs separate handling rather than a blanket if: false.
Note: workflows still trigger on their existing events/schedules but
the job is skipped, producing empty "skipped" runs. This is the
accepted trade-off for keeping the disabled state in source.
|
Pinging @elastic/ecosystem (Team:Ecosystem) |
Collaborator
|
We (@elastic/ecosystem) did not add those agent workflows. Adding as a reviewers @v1v and @theletterf in case they know about these workflows. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed commit message
Checklist
changelog.ymlfile.