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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fleet/flaky serverless api tests #178476

Closed

Conversation

jillguyonnet
Copy link
Contributor

@jillguyonnet jillguyonnet commented Mar 12, 2024

Summary

Closes #176754
Closes #176858

Fleet serverless API integration tests tests have already experienced flakiness due to slowness as they expect a preconfigured Fleet Server host and Elasticsearch output to exist. #176808 added retry logic in the tests, which improved flakiness a lot.

However, it seems we are still experiencing failure due to Saved Object not found errors. This indicates that the test setup with retrying is not fully working as intended: if the default fleet server host or output was not found during setup, the failure should be one of:

Expected default Fleet Server id default-fleet-server to exist
Expected default Elasticsearch output id es-default-output to exist

This PR increases the timeout of the setup retry and renames the error message thrown by Fleet backend when the SO is not found.

Flaky Test Runner builds

@jillguyonnet jillguyonnet self-assigned this Mar 12, 2024
@apmmachine
Copy link
Contributor

馃 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • /oblt-deploy-serverless : Deploy a serverless Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@jillguyonnet
Copy link
Contributor Author

Just realised that #176808 was tagged for 8.14.0 and never backported... @kpollich Do you think this could be the issue? Should I add the backport: prev-minor to #176808?

@kpollich
Copy link
Member

@jillguyonnet - Sure let's try to backport that PR.

@jillguyonnet
Copy link
Contributor Author

jillguyonnet commented Mar 13, 2024

The fix was backported to 8.13 in #178508. I've run 700 successful tests with the flaky test runner (cf. PR description) for good measure, so closing this for now.

@jillguyonnet jillguyonnet deleted the fleet/flaky-serverless-api-tests branch March 13, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment