Skip to content

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

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#581
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
Contributor

There were no differences in the rendered Helm template.

Output
=== Differences when rendered with values file helm/cluster-cloud-director/ci/test-wc-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.

@tinkerers-ci tinkerers-ci Bot added the skip/ci Instructs pr-gatekeeper to ignore any required PR checks 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 d6e56b7 into main Aug 7, 2026
11 of 13 checks passed
@njuettner
njuettner deleted the release-pr-body branch August 7, 2026 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/ci Instructs pr-gatekeeper to ignore any required PR checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants