Skip to content

chore: T8937: retire Mergify-replaceable GHA caller workflows (rolling)#43

Merged
andamasov merged 1 commit into
rollingfrom
chore/T8937-retire-mergify-replaceable-ghas-rolling
Jun 10, 2026
Merged

chore: T8937: retire Mergify-replaceable GHA caller workflows (rolling)#43
andamasov merged 1 commit into
rollingfrom
chore/T8937-retire-mergify-replaceable-ghas-rolling

Conversation

@andamasov

Copy link
Copy Markdown
Member

LTS-branch sweep for the T8937 GHA retirement (T8937, Phase B prep).

Removes the retired Mergify-replaceable caller workflows from rolling — already removed from rolling in the Phase C sweep; LTS branches are not covered by the mirror pipeline so they need direct PRs. Also removes any workflow still calling the retired central reusables (incl. the typo-named chceck-pr-message.yml where present).

Files deleted: chceck-pr-message.yml

Advances: T8937

🤖 Generated by robots

LTS-branch sweep (Phase B prep). Removes caller workflows superseded by
central Mergify rules, plus any workflow still referencing the retired
central reusables. See https://vyos.dev/T8937.

Advances: T8937

🤖 Generated by [robots](https://vyos.io)
@andamasov andamasov added the mergify-migration T8937 fleet sweep label Jun 10, 2026
@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: e3d457dd-8fe3-4257-a2f5-b34e0c07dee4

📥 Commits

Reviewing files that changed from the base of the PR and between 4fb0250 and 71eb9a1.

📒 Files selected for processing (1)
  • .github/workflows/chceck-pr-message.yml

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Removed an internal pull request validation workflow.

Walkthrough

The PR removes the .github/workflows/chceck-pr-message.yml file entirely—a 20-line GitHub Actions workflow that checked PR message format on pull_request_target events. The workflow delegated validation to a reusable workflow (check-pr-message.yml@production) from the shared vyos/.github repository.

Changes

Cohort / File(s) Summary
Workflow file removal
.github/workflows/chceck-pr-message.yml
Deleted GitHub Actions workflow file that performed PR message validation on pull_request_target for rolling, equuleus, sagitta, and circinus branches. The workflow invoked a reusable workflow from vyos/.github with inherited secrets.

Possibly related PRs

  • vyos/vyatta-bash#32: Removes other GitHub Actions pull_request_target caller workflows (add-pr-labels.yml, auto-author-assign.yml) as part of the same Mergify migration.
  • vyos/vyatta-bash#33: Modifies the same workflow's uses: reference to the shared check-pr-message.yml reusable workflow.
  • vyos/vyatta-bash#34: Updates the same workflow's pull_request_target branch filter from current to rolling.
✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch chore/T8937-retire-mergify-replaceable-ghas-rolling

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown

👍
No issues in PR Title / Commit Title

@mergify mergify Bot added the rolling label Jun 10, 2026
@andamasov andamasov marked this pull request as ready for review June 10, 2026 20:56
@andamasov andamasov merged commit 8ad5e46 into rolling Jun 10, 2026
5 of 6 checks passed
@andamasov andamasov deleted the chore/T8937-retire-mergify-replaceable-ghas-rolling branch June 10, 2026 20:57
@vyos-bot vyos-bot Bot added mirror-initiated This PR initiated for mirror sync workflow mirror-completed This PR has been mirrored successfully and removed mirror-initiated This PR initiated for mirror sync workflow labels Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergify-migration T8937 fleet sweep mirror-completed This PR has been mirrored successfully rolling

Development

Successfully merging this pull request may close these issues.

1 participant