Skip to content

ci(e2e): trigger E2E Test pipeline on new release/* branches#814

Merged
KayMKM merged 1 commit into
mainfrom
yuesu/e2e-test-trigger-on-release-branch
Jun 5, 2026
Merged

ci(e2e): trigger E2E Test pipeline on new release/* branches#814
KayMKM merged 1 commit into
mainfrom
yuesu/e2e-test-trigger-on-release-branch

Conversation

@KayMKM
Copy link
Copy Markdown
Contributor

@KayMKM KayMKM commented Jun 5, 2026

What

Add a CI trigger to the Modelkit E2E Test pipeline so each push to a release/* branch (including the initial push that creates a new release branch) automatically runs the pipeline.

trigger:
  batch: true
  branches:
    include:
      - release/*

Why

Today the E2E Test pipeline only runs:

  • Manually (queue-time), or
  • Daily at 00:00 UTC+8 on main.

When a new release/* branch is cut, there is no automatic E2E validation against it. This change ensures every new release branch gets an E2E run as soon as it's pushed, so we catch release-specific regressions before publishing.

Behavior

  • New release branch created -> fires on the initial push (which is how a branch is created in git), giving us the "trigger when a new release branch appears" semantics.
  • Subsequent commits on a release branch -> also trigger a run. Release branches typically see few commits, so this is acceptable.
  • batch: true -> coalesces rapid successive pushes into a single run, avoiding contention on the shared self-hosted agents (QNN / OV / AMD).
  • Daily main schedule -> unchanged.
  • PRs -> still not triggered; E2E failures don't block PR merges.

Add a CI trigger so each push to a release/* branch (including the
initial push that creates a new release branch) automatically runs the
Modelkit E2E Test pipeline. The daily main schedule is unchanged. Uses
batch: true to coalesce rapid successive pushes into a single run on
the shared self-hosted agents.
@KayMKM KayMKM requested a review from a team as a code owner June 5, 2026 03:09
Comment thread .pipelines/Modelkit E2E Test.yml
Comment thread .pipelines/Modelkit E2E Test.yml
Copy link
Copy Markdown
Collaborator

@DingmaomaoBJTU DingmaomaoBJTU left a comment

Choose a reason for hiding this comment

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

LGTM — small focused change, logic is correct. Two minor points left in comments (scheduled runs on release branches and batch semantics) but neither blocks the core goal.

@KayMKM KayMKM merged commit 104f21a into main Jun 5, 2026
9 checks passed
@KayMKM KayMKM deleted the yuesu/e2e-test-trigger-on-release-branch branch June 5, 2026 03:25
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.

2 participants