Skip to content

Conversation

@jozala
Copy link
Contributor

@jozala jozala commented Nov 24, 2025

Older version of openjdk17 are incompatible with newer ubuntu versions.
On ubuntu24 we now replace openjdk17 with adoptopenjdk17 in these cases.

The solution here is copied from #135504

The failure in Buildkite periodic build before the fix:
https://buildkite.com/elastic/elasticsearch-periodic/builds/11482#019ab0ce-0353-4908-a442-46819eb5889b

Including this fix:
https://buildkite.com/elastic/elasticsearch-periodic/builds/11487#019ab5cb-10dc-4efb-8e16-9e36b6a9d45f

It also fails in the PR builds. Example here:
#137440
https://buildkite.com/elastic/elasticsearch-pull-request/builds/105637#019ab57e-2f55-45ec-a6ce-3a41feded4b6

Older version of openjdk17 are incompatible with newer ubuntu versions.
On ubuntu24 we now replace openjdk17 with adoptopenjdk17 in these cases.
@jozala jozala added >non-issue :Delivery/Build Build or test infrastructure branch:8.19 labels Nov 24, 2025
@elasticsearchmachine elasticsearchmachine added Team:Delivery Meta label for Delivery team v8.19.8 labels Nov 24, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@jozala jozala requested a review from a team November 24, 2025 13:17
Copy link
Contributor

@breskeby breskeby left a comment

Choose a reason for hiding this comment

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

lgtm. The succesful example build you link to is on main right? which hasn't suffered from this issue. only 8.19 seems to be affected.

Longterm I wish we could just aggressively either not run those combinations of packaged java + OS. This adds so much complexity, mental load and maintability burdon. Ideally we would just run those against the last 100% supported OS but that will leave us with a lack of test coverage I'm not sure we are willing to address.

@jozala
Copy link
Contributor Author

jozala commented Nov 25, 2025

lgtm. The succesful example build you link to is on main right? which hasn't suffered from this issue. only 8.19 seems to be affected.

The successful build I've mentioned here:

Including this fix:
https://buildkite.com/elastic/elasticsearch-periodic/builds/11487#019ab5cb-10dc-4efb-8e16-9e36b6a9d45f

Is same as branch in this PR which is created from 8.19. This is to prove that the fix works on 8.19.
As you've mentioned, the issue doesn't exist on main on any other branch than 8.19 (edit) so I don't plan to add this to any of the 9.x/main branches.

@jozala jozala merged commit 2100cf5 into elastic:8.19 Nov 25, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v8.19.8

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants