Skip to content

Automated moxygen sync + auto-merge workflows - #56

Closed
gmarzot wants to merge 1 commit into
mainfrom
ci-workflows
Closed

Automated moxygen sync + auto-merge workflows#56
gmarzot wants to merge 1 commit into
mainfrom
ci-workflows

Conversation

@gmarzot

@gmarzot gmarzot commented Mar 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Replace the manual update moxygen submodule workflow with an event-driven
automated sync chain:

  • moxygen-sync.yml — Triggered by repository_dispatch from moxygen's
    ci main (or manual workflow_dispatch). Checks for blocking
    sync-moxygen/* PR, bumps submodule, creates PR with dual-identity
    approval (bot creates, PAT approves). Notifies Slack + email on block
    or failure.

  • auto-merge-moxygen.yml — Fires when ci pr completes on
    sync-moxygen/* branches. Merges on success, notifies on failure.
    Single job with step-level conditions (no skipped job noise).

  • ci-pr.yml — Fallback to standalone source build when prebuilt
    moxygen tarball is not available. Enables cross-repo development PRs
    where both moxygen and o-rly changes are in flight.

  • CI_OVERVIEW.md — Updated workflow descriptions and flow diagram
    showing the fully automated chain from upstream commit to Docker image.

Companion PR: openmoq/moxygen#82

Test plan

  • Merge moxygen#82 first (dispatch side)
  • Merge this PR, then trigger moxygen ci main and verify:
    • moxygen-update dispatch reaches o-rly
    • sync-moxygen/sha PR created with approval
    • ci pr runs, moxygen auto-merge merges the PR
  • Test manual workflow_dispatch trigger
  • Test blocking: create a second dispatch while a sync PR is open
  • Test standalone fallback: point submodule at a non-main commit

This change is Reviewable

@gmarzot gmarzot self-assigned this Mar 14, 2026
@gmarzot

gmarzot commented Mar 14, 2026

Copy link
Copy Markdown
Contributor Author

#52

Replace manual update-moxygen-submodule.yml with event-driven sync:

- moxygen-sync.yml: triggered by repository_dispatch (from moxygen
  ci-main) or manual workflow_dispatch. Checks for blocking
  sync-moxygen/* PR, bumps submodule, creates PR with dual-identity
  approval (bot creates, PAT approves).

- auto-merge-moxygen.yml: fires on ci-pr completion for
  sync-moxygen/* branches. Merges on success, notifies on failure.

- ci-pr.yml: fallback to standalone source build when prebuilt
  moxygen tarball not available (enables cross-repo dev PRs).

- CI_OVERVIEW.md: updated with new workflow descriptions and
  fully automated flow diagram.
@gmarzot gmarzot changed the title Add bump-moxygen workflows, rename format to check-format Automated moxygen sync + auto-merge workflows Mar 17, 2026
@gmarzot

gmarzot commented Mar 17, 2026

Copy link
Copy Markdown
Contributor Author

Reopening from devops/moxygen-sync branch (new naming convention).

@gmarzot gmarzot closed this Mar 17, 2026
@gmarzot
gmarzot deleted the ci-workflows branch March 17, 2026 02:20
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.

1 participant