Skip to content

CI: Add Gerrit RTDv3 merge workflow - #48

Closed
ModeSevenIndustrialSolutions wants to merge 1 commit into
lfit:mainfrom
modeseven-lfit:ci/rtdv3-merge
Closed

ModeSevenIndustrialSolutions wants to merge 1 commit into
lfit:mainfrom
modeseven-lfit:ci/rtdv3-merge

Conversation

@ModeSevenIndustrialSolutions

Copy link
Copy Markdown
Contributor

Summary

Adds an organization-level required merge workflow that publishes Read
the Docs (RTDv3) documentation once a Gerrit change merges.

Part of the effort to modernise the Linux Foundation Gerrit CI and
standardise on the centrally-managed reusable workflows in
lfit/releng-reusable-workflows.

Design

Thin dispatcher (on: workflow_dispatch, invoked by the Gerrit → GitHub
integration on merge):

  1. Notify merge start — posts a start comment to Gerrit (comment-only,
    the change has already merged).
  2. Read the Docs merge — delegates the publish to the shared reusable
    gerrit-compose-required-rtdv3-merge.yaml@v0.8.1, forwarding the RTD
    API token.
  3. Report merge status — reports the aggregated conclusion back to the
    change as a comment.

Notes / follow-ups

  • All uses: references are SHA-pinned to their latest releases.
  • Passes zizmor --persona=auditor with zero findings. The
    secrets-outside-env allow-list is provided by the companion config PR
    (CI: Add zizmor auditor configuration #45), which should merge first.
  • Org provisioning required (admin): relies on LFIT_GERRIT_SERVER,
    LFIT_GERRIT_KNOWN_HOSTS, LFIT_GERRIT_SSH_REQUIRED_USER /
    LFIT_GERRIT_SSH_REQUIRED_PRIVKEY, the RTD_TOKEN secret, and the shared
    reusable's vars.GERRIT_URL. A repository Ruleset must wire this
    workflow to run on merge for the Gerrit-mirrored docs repositories.
  • Enable this only if LF wants docs published via RTDv3 (companion to the
    bypassable verify workflow in CI: Add Gerrit bypassable RTDv3 verify workflow #47).

Validation

  • actionlint → clean
  • yamllint → clean
  • zizmor --persona=auditor --offline → no findings

Add an organization-level required merge workflow that publishes
Read the Docs (RTDv3) documentation once a Gerrit change merges.

The workflow is a thin dispatcher: it posts a start comment to Gerrit,
delegates the documentation publish to the centrally-managed reusable
gerrit-compose-required-rtdv3-merge.yaml (v0.8.1) in
lfreleng-actions' releng-reusable-workflows, then reports the
aggregated conclusion back to the change as a comment. Voting steps run
comment-only since the change has already merged.

All action references are SHA-pinned to their latest releases and the
workflow passes zizmor --persona=auditor with zero findings.

Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Matthew Watkins <mwatkins@linuxfoundation.org>
@tykeal

tykeal commented Jul 24, 2026

Copy link
Copy Markdown
Member

RTD builds are handled by the RTD GitHub connector. All merges automatically cause a new build to happen. This is an unneeded job.

@tykeal tykeal closed this Jul 24, 2026
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