diff options
| author | Yuriy Andamasov <yuriy@vyos.io> | 2026-05-14 01:16:59 +0300 |
|---|---|---|
| committer | Yuriy Andamasov <yuriy@vyos.io> | 2026-05-14 01:16:59 +0300 |
| commit | d75de0c24b971a2d87e817ac3cb8e09cb816aac0 (patch) | |
| tree | 7df810a4c0fd211fb84e528204379bb5a175549e /docs/_templates/layout.html | |
| parent | 155fc647d54477245e9fff0cfbcb4cd4ef1d548b (diff) | |
| download | vyos-documentation-d75de0c24b971a2d87e817ac3cb8e09cb816aac0.tar.gz vyos-documentation-d75de0c24b971a2d87e817ac3cb8e09cb816aac0.zip | |
ci(ai-validation): skip validation on Mergify-authored PRs
The upstream anthropics/claude-code-action rejects bot-initiated
workflow runs with:
Workflow initiated by non-human actor: mergify (type: Bot).
Add bot to allowed_bots list or use '*' to allow all bots.
This leaves a failing required check on every Mergify backport PR
(#2025 today is the proximate symptom — circinus backport of #2016 is
blocked from auto-merging by this).
Per org policy, Mergify-authored PRs skip the bot-review flow entirely:
the underlying change was already reviewed on the source PR. Re-running
validation on the backport would also post duplicate findings.
Fix: extend the existing `secrets-check` step (kept the id for backwards
compat with the cascade of `if: steps.secrets-check.outputs.skip != 'true'`
gates) to also short-circuit when `github.event.pull_request.user.login`
equals `mergify[bot]`. Add a `skip_reason` output so the notify-on-PR
step can distinguish bot-skip (silent) from missing-secrets (post a
notice comment).
Implementation notes:
- PR author is bound through env: PR_AUTHOR (same pattern as secret
bindings) so the template-expansion happens before bash sees the
value as an already-quoted env variable.
- Uses `printf '...%s\n' "$PR_AUTHOR"` for the notice line so the
shell — not the YAML template engine — interpolates the author into
the workflow log output.
- Renamed step name to "Decide whether to skip validation" to reflect
the broader scope; step id stays `secrets-check`.
🤖 Generated by [robots](https://vyos.io)
Diffstat (limited to 'docs/_templates/layout.html')
0 files changed, 0 insertions, 0 deletions
