Skip to content

fix(ci): preserve only deployment-owned workflows - #40

Merged
theredspoon merged 1 commit into
pipeline-controlfrom
fix/selective-deployment-workflow-overlay
Jul 1, 2026
Merged

fix(ci): preserve only deployment-owned workflows#40
theredspoon merged 1 commit into
pipeline-controlfrom
fix/selective-deployment-workflow-overlay

Conversation

@theredspoon

@theredspoon theredspoon commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep source branch validation workflows in deployment mirror PRs
  • restore only deployment-target-owned workflow contracts from deployment-target
  • prevents stale target Code Style from dropping source-side Node setup for all-features Clippy

Tests

  • bash scripts/test-auto-ff-matrix-pilot-sync-workflow.sh
  • bash scripts/test-sync-upstream-workflow.sh
  • bash scripts/test-pipeline-control-sync-workflow.sh
  • git diff --check

@github-actions github-actions Bot added scope: ci size: S Changed-line size classification risk: medium Risk classification contributor: core Contributor history classification labels Jul 1, 2026
@theredspoon theredspoon added the skip-regression-check Regression check intentionally skipped with justification label Jul 1, 2026
@theredspoon
theredspoon force-pushed the fix/selective-deployment-workflow-overlay branch from 08b1efc to 78fec39 Compare July 1, 2026 15:33
@theredspoon
theredspoon merged commit 5473ec0 into pipeline-control Jul 1, 2026
16 checks passed
@theredspoon
theredspoon deleted the fix/selective-deployment-workflow-overlay branch July 1, 2026 15:34
chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Jul 1, 2026
View reviewed changes

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 08b1efc232

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/mirror-to-deployment-target.yml
Comment on lines +60 to +64
target_owned_workflows=(
.github/workflows/pilot-linux-artifact.yml
.github/workflows/regression-test-check.yml
)
git checkout "refs/remotes/deployment_mirror/${TARGET_BRANCH}" -- "${target_owned_workflows[@]}"

@chatgpt-codex-connector chatgpt-codex-connector Bot Jul 1, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Filter source schedules before mirroring

With this allowlist checkout, every other source workflow remains in the tree that git write-tree pushes. In a deployment mirror where deployment-target is the default branch, unguarded source schedules such as .github/workflows/live-canary.yml (0 */6 * * *) and nightly CI will start running in the mirror after merge instead of only PR validation workflows. Please remove or explicitly allowlist source workflows before creating the mirror commit.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments

Assignees

No one assigned

Labels

contributor: core Contributor history classification risk: medium Risk classification scope: ci size: S Changed-line size classification skip-regression-check Regression check intentionally skipped with justification

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@theredspoon