Skip to content

Disable unowned AI GitHub Actions workflows - #19933

Merged
kcreddy merged 1 commit into
elastic:mainfrom
kcreddy:github-disable-agentic-workflow
Jul 7, 2026
Merged

Disable unowned AI GitHub Actions workflows#19933
kcreddy merged 1 commit into
elastic:mainfrom
kcreddy:github-disable-agentic-workflow

Conversation

@kcreddy

@kcreddy kcreddy commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

ci: disable unowned AI GitHub Actions workflows

The sweep-* and trigger-* workflows wrap reusable workflows from
elastic/ai-github-actions. They were set up by a contributor who has
since left, and no team currently owns them.

Left running unowned, they generate noise that no one is triaging:
100+ AI-authored issues opened since March with no team labels, issues
filed against deprecated integrations, and issues created when the bots
themselves error out. Disable them to stop the noise until ownership is
decided — rather than deleting them, since deletion would lose the
config and history and a team may want to adopt and re-enable them.

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.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

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.
@kcreddy kcreddy self-assigned this Jul 2, 2026
@kcreddy kcreddy added the Team:Ecosystem Packages Ecosystem team [elastic/ecosystem] label Jul 2, 2026
@kcreddy kcreddy changed the title ci: disable unowned AI GitHub Actions workflows Disable unowned AI GitHub Actions workflows Jul 2, 2026
@kcreddy
kcreddy marked this pull request as ready for review July 2, 2026 12:23
@kcreddy
kcreddy requested a review from a team as a code owner July 2, 2026 12:23
@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

Pinging @elastic/ecosystem (Team:Ecosystem)

@mrodm

mrodm commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

We (@elastic/ecosystem) did not add those agent workflows. Adding as a reviewers @v1v and @theletterf in case they know about these workflows.

@mrodm
mrodm requested review from theletterf and v1v July 2, 2026 13:23
@v1v
v1v requested a review from a team July 5, 2026 08:47
@fr4nc1sc0-r4m0n

Copy link
Copy Markdown
Contributor

If the problem is the noisy issues that are being generated by the execution failures, I think this elastic/ai-github-actions#1397 could help to reduce the noise.
On the contrary, If you think that some of them are not giving any value, I'd prefer to remove them because we keep the workflows directory as clean as possible, we don't waste resources in non desired executions and if necessary, we could always recover them because they're in the git history.
WDYT?

@teresaromero teresaromero 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.

i would rather delete them if no one use this

@mergify

mergify Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@kcreddy
kcreddy merged commit 30e2f88 into elastic:main Jul 7, 2026
9 checks passed

@v1v v1v left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd suggest to disable the workflows instead of jobs, otherwise, they will run on events without doing much:

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Ecosystem Packages Ecosystem team [elastic/ecosystem]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants