Skip to content

[8.19] (backport #18668) Only raise PR to bump java version when all artifacts are ready#18669

Merged
donoghuc merged 1 commit into8.19from
mergify/bp/8.19/pr-18668
Jan 29, 2026
Merged

[8.19] (backport #18668) Only raise PR to bump java version when all artifacts are ready#18669
donoghuc merged 1 commit into8.19from
mergify/bp/8.19/pr-18668

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jan 28, 2026

Previously a PR would be raised to bump the java version shipped with LS once the artifact for x86_64 linux was available. In practice we also need the artifacts for arm, macos, windows, etc. This commit adds conditions to updatecli such that only after all artifacts are available will the PR be raised to bump the version.


This is an automatic backport of pull request #18668 done by Mergify.

* Ensure all JDK are available before raising java bump PR

Previously a PR was raised when a new java version was detected for the default
x86_64 linux artifact. In practice, other platforms become available later than
the default. This commit updates updatecli to only raise a PR once all artifacts
are ready.

* use curl instead

(cherry picked from commit e2df002)
@github-actions
Copy link
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
  • /run exhaustive tests : Run the exhaustive tests Buildkite pipeline.

Copy link
Member

@donoghuc donoghuc left a comment

Choose a reason for hiding this comment

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

Appears to be working on main (packages not available yet == no PR) https://github.com/elastic/logstash/actions/runs/21459493176/job/61808437312

@elasticmachine
Copy link

@donoghuc donoghuc merged commit 220f356 into 8.19 Jan 29, 2026
12 checks passed
@donoghuc donoghuc deleted the mergify/bp/8.19/pr-18668 branch January 29, 2026 00:21
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