[9.0] [APM] Fix APM flaky test cleanup logic (#241310)#241362
[9.0] [APM] Fix APM flaky test cleanup logic (#241310)#241362kibanamachine wants to merge 2 commits into
Conversation
Closes elastic#234664 ## Summary This PR fixes a flaky APM test cleanup `after` issue. The test was failing because the after hook was trying to delete the Elasticsearch index apm-8.0.0-transaction without first checking if it exists. This caused an `index_not_found_exception` when the index didn't exist. The test setup in the before hook properly checks if the index exists before deleting it ([lines 35-40](https://github.com/elastic/kibana/compare/main...jennypavlova:kibana:fix-flaky-apm-test-after?expand=1#diff-88902b6ae54830e04bb08653dfda680e427fd98f5d5ae8e5eff6d6dbad455e2cL35-L40)), but the cleanup in the after hook ([line 137](https://github.com/elastic/kibana/compare/main...jennypavlova:kibana:fix-flaky-apm-test-after?expand=1#diff-88902b6ae54830e04bb08653dfda680e427fd98f5d5ae8e5eff6d6dbad455e2cL137)) was missing this same safety check. (cherry picked from commit 8b85ef3)
|
CI was triggered for this PR, but this PR targets 9.0 which should not receive a future release. CI is not supported for these branches. Please consult the release schedule, or contact The following branches are currently considered to be open:
|
1 similar comment
|
CI was triggered for this PR, but this PR targets 9.0 which should not receive a future release. CI is not supported for these branches. Please consult the release schedule, or contact The following branches are currently considered to be open:
|
|
CI was triggered for this PR, but this PR targets 9.0 which should not receive a future release. CI is not supported for these branches. Please consult the release schedule, or contact The following branches are currently considered to be open:
|
1 similar comment
|
CI was triggered for this PR, but this PR targets 9.0 which should not receive a future release. CI is not supported for these branches. Please consult the release schedule, or contact The following branches are currently considered to be open:
|
Backport
This will backport the following commits from
mainto9.0:Questions ?
Please refer to the Backport tool documentation