Skip to content

[1.0] fix(ci): restore missing build_type check in release pipeline (#504) - #511

Merged
Jan-Kazlouski-elastic merged 1 commit into
1.0from
backport/1.0/pr-504
Oct 5, 2026
Merged

Jan-Kazlouski-elastic merged 1 commit into
1.0from
backport/1.0/pr-504

Conversation

@github-actions

@github-actions github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown

Backports the following commits to 1.0:

## Summary

Fixes a regression introduced in #503 where the outer `if [[
"$BUILD_TYPE" == "release" ]]` guard was accidentally dropped from the
"Resolve build type" step.

**Bug:** The branch pattern check (`^[0-9]+\.[0-9]+$`) ran
unconditionally, so any Snapshot build triggered from `main` would fail
immediately with a misleading error:
> ERROR: Release builds require an N.N branch (for example, 1.0)

**Fix:** Restore the missing `if [[ "$BUILD_TYPE" == "release" ]]; then`
wrapper so the branch guard only runs for Release builds.

## Test plan

- [ ] Trigger the release pipeline from `main` with default Snapshot
settings → confirm SNAPSHOT image published (was broken before this fix)
- [ ] Trigger the release pipeline from `main` and choose "Release" →
confirm it is rejected
- [ ] Trigger the release pipeline from `1.0` with "Release" → confirm
release image published

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@Jan-Kazlouski-elastic
Jan-Kazlouski-elastic merged commit 18773cd into 1.0 Oct 5, 2026
2 checks passed
@Jan-Kazlouski-elastic
Jan-Kazlouski-elastic deleted the backport/1.0/pr-504 branch October 5, 2026 15:49
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