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

[TEST] [ML] Revert #185012 Do not retry model deployment #186219

Closed

Conversation

spong
Copy link
Member

@spong spong commented Jun 13, 2024

Summary

Reverts #185012 to see if it was the cause of this pt_tiny_elser failure.


Looks like it passed with the revert: https://buildkite.com/elastic/kibana-pull-request/builds/215924

Now let's do a flakey test run with it reverted here:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6325

image

1 failure: same error:

Error: Expected status code 200, got 404 with body '{"error":{"root_cause":[{"type":"resource_not_found_exception","reason":"Could not find trained model [pt_tiny_elser]"}],"type":"resource_not_found_exception","reason":"Could not find trained model [pt_tiny_elser]"},"status":404}'

And of another PR now updated w/ main that was green before:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6327

image

1 failure: same error:

Error: Expected status code 200, got 404 with body '{"error":{"root_cause":[{"type":"resource_not_found_exception","reason":"Could not find trained model [pt_tiny_elser]"}],"type":"resource_not_found_exception","reason":"Could not find trained model [pt_tiny_elser]"},"status":404}'


This does not appear to be caused by this change, so going to close this PR. Will skip test for now and continue investigation w/ ML folks.

@spong spong marked this pull request as ready for review June 13, 2024 22:12
@spong spong requested review from a team as code owners June 13, 2024 22:12
@spong spong added release_note:skip Skip the PR/issue when compiling release notes v8.15.0 labels Jun 13, 2024
@kibana-ci
Copy link
Collaborator

kibana-ci commented Jun 13, 2024

💔 Build Failed

Failed CI Steps

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#6325

[❌] x-pack/test/security_solution_api_integration/test_suites/genai/nlp_cleanup_task/basic_license_essentials_tier/configs/serverless.config.ts: 12/25 tests passed.

see run history

@spong spong closed this Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants