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

[ML] Detect timeout when waiting for download task (#103197) #104370

Merged

Conversation

davidkyle
Copy link
Member

A list tasks timeout indicates the task exists and is in progress. Interpreting the timeout as the task not existing meant the download check would incorrectly assume the download had completed.

backport of #103197

A list tasks timeout indicates the task exists and is in progress.
Interpreting the timeout as the task not existing meant the download
check would incorrectly assume the download had completed.
# Conflicts:
#	x-pack/plugin/ml-package-loader/src/main/java/org/elasticsearch/xpack/ml/packageloader/action/TransportLoadTrainedModelPackage.java
#	x-pack/plugin/src/yamlRestTest/resources/rest-api-spec/test/ml/3rd_party_deployment.yml
@davidkyle davidkyle added >test Issues or PRs that are addressing/adding tests :ml Machine learning backport v8.12.1 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Jan 15, 2024
@elasticsearchmachine elasticsearchmachine merged commit d81117a into elastic:8.12 Jan 15, 2024
15 checks passed
@davidkyle davidkyle deleted the waiting-on-download-812 branch January 15, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :ml Machine learning >test Issues or PRs that are addressing/adding tests v8.12.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants