Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport PR #15827 to 7.17: Fix IT tests after version bumps #15838 #15839

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

dliappis
Copy link
Contributor

Backport PR #15827 to 7.17 branch, original message:


Release notes

[rn:skip]

What does this PR do?

This commit fixes IT failures that frequently occur after version bumps due to missing unified release snapshot builds for the new version.

This commit uses project specific DRA snapshot URLs for ES and Filebeat
in all cases apart from release builds.

Why is it important/What is the impact to the user?

This is a very important to fix as several pipelines (PR, exhaustive, JDK matrix, aarch64) run IT tests as a prerequisite and produce unnecessary failures shortly
after version bumps, possibly blinding us from genuine issues.

How to test this PR locally

See CI from backporting this PR to 8.12:

and compare to https://buildkite.com/elastic/logstash-exhaustive-tests-pipeline/builds/126#018d31b4-8da9-4dbf-b43e-217f2a4f7518/46-469 showing current failures on the 8.12 branch

Related issues

This commit fixes IT failures that frequently occur after
version bumps due to missing unified release snapshot builds for
the new version.

This commit uses project specific DRA snapshot URLs for ES and Filebeat
in all cases apart from release builds.

(cherry picked from commit d74fea4)

Co-authored-by: Dimitrios Liappis <dimitrios.liappis@gmail.com>
@dliappis
Copy link
Contributor Author

@andsel a clean backport from #15827 to 7.17 wasn't possible (conflicts in build.gradle). Would appreciate it if you could take a look at this manual backport.

@elasticmachine
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

cc @dliappis

Copy link
Contributor

@andsel andsel 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 dliappis merged commit 72900ae into elastic:7.17 Jan 23, 2024
3 checks passed
@dliappis dliappis deleted the backport_15827_to_7_17 branch January 23, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants