Skip to content

.github/templates: exclude cilium-cli-exclusive PRs from minor changelog - #421

Merged
joestringer merged 1 commit into
mainfrom
pr/exclude-cilium-cli-exclusive-minor-template
Jul 29, 2026
Merged

.github/templates: exclude cilium-cli-exclusive PRs from minor changelog#421
joestringer merged 1 commit into
mainfrom
pr/exclude-cilium-cli-exclusive-minor-template

Conversation

@aanm

@aanm aanm commented Jul 29, 2026

Copy link
Copy Markdown
Member

The minor release template is the only one that runs 2-prepare-release
locally. The other templates (patch, pre_main, rc_branch) route that
step through the release workflow in cilium/cilium, which already passes
--exclude-labels "cilium-cli-exclusive" on every ./release start
invocation.

Without the flag, a local run of step 2 picks up PRs labeled
cilium-cli-exclusive in the generated changelog, so the output differs from
what CI would have produced for the same release.

Regenerated testdata/checklist/release_template_minor.md.golden via
make generate-golden. Only the minor golden changed — the .input files are
symlinks to the real templates and only that one template was modified.

The minor release template is the only one that runs 2-prepare-release
locally; the other templates route that step through the release
workflow in cilium/cilium, which already passes
--exclude-labels "cilium-cli-exclusive".

Without the flag, a local run picks up PRs labeled cilium-cli-exclusive
in the generated changelog, which then differs from what CI would have
produced.

Signed-off-by: André Martins <andre@cilium.io>
@aanm
aanm requested a review from joestringer July 29, 2026 08:45
@aanm
aanm marked this pull request as ready for review July 29, 2026 08:45
@joestringer
joestringer merged commit 213c357 into main Jul 29, 2026
1 check passed
@joestringer
joestringer deleted the pr/exclude-cilium-cli-exclusive-minor-template branch July 29, 2026 16:54
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