-
Notifications
You must be signed in to change notification settings - Fork 528
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
functionaltests: add 8.17 tests #15310
base: main
Are you sure you want to change the base?
Conversation
This pull request does not have a backport label. Could you fix it @endorama? 🙏
|
|
46c0d08
to
1d9a6e7
Compare
Include also a prefer ILM value to be checked in assertion.
This is necessary to keep test duration within the 30 minutes timeout.
The logic in assertDocCount was allowing an edge case where none of the expected data streams were found. This is now amended by checking that the retrieved docCounts map contains all the expected data streams.
1d9a6e7
to
38166ef
Compare
This pull request is now in conflicts. Could you fix it @endorama? 🙏
|
Motivation/summary
Add 8.17 tests to functional tests:
apm-data
plugin.Tests will be executed in parallel to keep the timeout to 30 minutes. This introduces code duplication in the Terraform code that will be addressed later on.
Checklist
How to test these changes
GitHub workflow
functionaltests
should be green.Initial test run https://github.com/elastic/apm-server/actions/runs/12889138214 (failed)
Related issues
#14100