Skip to content

Pin CI container images to v0.4 for 7.17 branch#16497

Merged
dliappis merged 2 commits intoelastic:7.17from
dliappis:pin-ci-container-to-v0.4
Oct 2, 2024
Merged

Pin CI container images to v0.4 for 7.17 branch#16497
dliappis merged 2 commits intoelastic:7.17from
dliappis:pin-ci-container-to-v0.4

Conversation

@dliappis
Copy link
Copy Markdown
Contributor

@dliappis dliappis commented Oct 2, 2024

Release notes

[rn:skip]

What does this PR do?

This commit pins the CI container images to the older version to fix failures like https://buildkite.com/elastic/logstash-exhaustive-tests-pipeline/builds/760

This problem started happening after https://github.com/elastic/ci-agent-images/pull/932 where CI images got updated to Java 21 which breaks CI for the 7.17 branch.

How to test this PR locally

https://buildkite.com/elastic/logstash-exhaustive-tests-pipeline/builds/767

Recently CI images got updated to Java 21 which breaks CI for the 7.17
branch.

This commit pins the CI container images to the older version for fix failures like https://buildkite.com/elastic/logstash-exhaustive-tests-pipeline/builds/760
Copy link
Copy Markdown
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

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

LGTM

@dliappis
Copy link
Copy Markdown
Contributor Author

dliappis commented Oct 2, 2024

Acceptance tests failed on oraclelinux-7: https://buildkite.com/elastic/logstash-exhaustive-tests-pipeline/builds/767#01924d55-5752-4e08-894d-e6def4af44d2/65-76

because the hard coded JRuby 9.4.5.0 which was worked around in https://github.com/elastic/logstash/pull/16489/files
wasn't backported to 7.17

I actually raised a clean up for main in #16499 (needs to be backported to all releasable branches) and cherry-picked it here: 65f5a1f

Triggered another run of the exhaustive tests: https://buildkite.com/elastic/logstash-exhaustive-tests-pipeline/builds/768

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

Copy link
Copy Markdown
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

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

LGTM

@dliappis
Copy link
Copy Markdown
Contributor Author

dliappis commented Oct 2, 2024

Thanks for the quick re-review @jsvd ; the oraclelinux-7 step is now running happily with the latest commit so I'll merge this PR once the exhaustive build is green.

@dliappis dliappis merged commit 0e19e42 into elastic:7.17 Oct 2, 2024
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.

3 participants