CI: Add zizmor auditor configuration#45
Open
ModeSevenIndustrialSolutions wants to merge 1 commit into
Open
Conversation
Establish a shared zizmor policy for the organization's Gerrit "required" workflows so they pass the strict --persona=auditor scan with zero findings. The Gerrit verification and merge workflows vote on changes using dedicated SSH service accounts, and the RTDv3 reusables require an RTD API token. These credentials are organization secrets consumed directly by the actions; they cannot be scoped to a GitHub deployment environment because environment secrets do not propagate correctly into reusable workflows (which would require "secrets: inherit", itself flagged by the secrets-inherit audit). Allow-list these credentials for the auditor-only secrets-outside-env audit, which is the remediation sanctioned by zizmor for this case. Co-authored-by: Claude <noreply@anthropic.com> Signed-off-by: Matthew Watkins <mwatkins@linuxfoundation.org>
This was referenced Jul 20, 2026
zxiiro
approved these changes
Jul 20, 2026
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.
Summary
Establishes a shared zizmor policy for the
organization's Gerrit "required" workflows so they pass the strict
--persona=auditorscan with zero findings.This is the first of a small series of atomic changes that modernise the
Linux Foundation Gerrit verification/merge CI to standardise on the
centrally-managed reusable workflows in
lfit/releng-reusable-workflows.It lands the audit configuration on its own so each subsequent workflow
change stays focused and independently reviewable.
What & why
The Gerrit verification and merge workflows vote on changes using
dedicated SSH service accounts, and the RTDv3 reusables require an RTD API
token. These are organization secrets consumed directly by the actions.
They cannot be scoped to a GitHub deployment environment because
environment secrets do not propagate correctly into reusable workflows
(that would require
secrets: inherit, which is itself flagged byzizmor's
secrets-inheritaudit).Allow-listing these credentials for the auditor-only
secrets-outside-envaudit is the remediation sanctioned by zizmor
for exactly this situation.
Allow-listed secrets (forward-looking, covering the workflows added in the
follow-up changes):
LFIT_GERRIT_SSH_REQUIRED_PRIVKEY— required-check voting accountLFIT_GERRIT_SSH_BYPASSABLE_PRIVKEY— bypassable-check voting accountRTD_TOKEN— Read the Docs v3 API tokenValidation
zizmor --persona=auditor --offline→ auto-discovers this configyamllint→ clean