Skip to content

feat: Run the full E2E suite set automatically on release PRs - #218

Merged
njuettner merged 1 commit into
mainfrom
release-pr-body
Aug 7, 2026
Merged

feat: Run the full E2E suite set automatically on release PRs#218
njuettner merged 1 commit into
mainfrom
release-pr-body

Conversation

@njuettner

Copy link
Copy Markdown
Member

Towards giantswarm/roadmap#4334

Release PRs are created with an empty body, so Tekton — which starts the tests from a /run line in the PR body — never ran anything on them. Running the E2E tests depended on someone remembering to comment, and release PRs were merged with no run at all (cluster-aws#2036, cluster-cloud-director#573, cluster-vsphere#527).

Adds .github/release-pr-body.md, which giantswarm/github-workflows#266 uses as the release PR body. cluster-test-suites already expands to every suite for this provider once it detects a release PR, so no suite parameters are needed here.

Has no effect until giantswarm/github-workflows#266 merges; until then the file is simply unused.

Release PRs are created with an empty body, so Tekton - which starts the tests from a
/run line in the PR body - never ran anything on them, and releases were merged with no
E2E run at all. giantswarm/github-workflows#266 makes the release PR body come from this
file when present.

cluster-test-suites already expands to every suite for this provider once it detects a
release PR, so no suite parameters are needed.
@njuettner
njuettner requested a review from a team as a code owner August 6, 2026 12:11
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

There were no differences in the rendered Helm template.

Output
=== Differences when rendered with values file helm/cluster-eks/ci/test-wc-minimal-values.yaml ===

No difference


=== No differences at all ===

@njuettner njuettner changed the title Run the full E2E suite set automatically on release PRs feat: Run the full E2E suite set automatically on release PRs Aug 7, 2026
@njuettner

Copy link
Copy Markdown
Member Author

/skip-ci This change does not affect what's being tested by CI.

@njuettner
njuettner enabled auto-merge (squash) August 7, 2026 09:26
@tinkerers-ci tinkerers-ci Bot added the skip/ci label Aug 7, 2026
@tinkerers-ci

tinkerers-ci Bot commented Aug 7, 2026

Copy link
Copy Markdown

🚨 CI checks were skipped

Skip reason: This change does not affect what's being tested by CI.
Skipped by: @njuettner

@njuettner
njuettner merged commit 1e4f8f8 into main Aug 7, 2026
12 of 13 checks passed
@njuettner
njuettner deleted the release-pr-body branch August 7, 2026 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants