-
Notifications
You must be signed in to change notification settings - Fork 25.1k
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
[CI] APMYamlTestSuiteIT test {yaml=/10_apm/Test template reinstallation} failing #116445
Comments
This has been muted on branch 8.x Mute Reasons:
Build Scans: |
…0_apm/Test template reinstallation} #116445
Pinging @elastic/es-data-management (Team:Data Management) |
@axw I think this one is for APM also |
This has been muted on branch main Mute Reasons:
Build Scans: |
…0_apm/Test template reinstallation} #116445
…0_apm/Test template reinstallation} elastic#116445
…0_apm/Test template reinstallation} elastic#116445
This has been muted on branch 8.16 Mute Reasons:
Build Scans: |
…0_apm/Test template reinstallation} #116445
I would like to put forth again the idea of adding the following "wait" in these type of tests:
This request is going to wait until all the queued cluster events have been processed. See https://www.elastic.co/guide/en/elasticsearch/reference/current/cluster-health.html. It's not a solution, but it can decrease the flakiness. Considering that we do not have a solution yet, months after this has been opened, I think it's worth the try. |
I saw this and I had proposed this in the past as well, but it was rejected as an approach with the reason that yaml tests are not meant to do this, and we will be introducing a feature in the yaml tests to cover for a bug in our set-up. They do have a point btw, and we do have an issue to fix it but it's not a super easy thing to do. That's why I thought maybe this could a good enough patch. |
I just checked and this was already used in the set-up of this test. |
Build Scans:
Reproduction Line:
Applicable branches:
8.16
Reproduces locally?:
N/A
Failure History:
See dashboard
Failure Message:
Issue Reasons:
Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
The text was updated successfully, but these errors were encountered: