Skip to content

[ML] Skip download check for non-packaged models#136747

Merged
davidkyle merged 1 commit intoelastic:mainfrom
davidkyle:dont-check-if-not-packaged
Oct 20, 2025
Merged

[ML] Skip download check for non-packaged models#136747
davidkyle merged 1 commit intoelastic:mainfrom
davidkyle:dont-check-if-not-packaged

Conversation

@davidkyle
Copy link
Member

While investigating #133603 I noticed that when creating a ML trained model the check for an existing download of the model always runs. Only pre-packaged models can be downloaded, the test can be skipped for models being uploaded to Elasticsearch

@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label Oct 17, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@davidkyle davidkyle merged commit ce6a080 into elastic:main Oct 20, 2025
34 checks passed
@davidkyle davidkyle deleted the dont-check-if-not-packaged branch October 20, 2025 15:56
chrisparrinello pushed a commit to chrisparrinello/elasticsearch that referenced this pull request Oct 24, 2025
fzowl pushed a commit to voyage-ai/elasticsearch that referenced this pull request Nov 3, 2025
prwhelan added a commit to prwhelan/elasticsearch that referenced this pull request Dec 18, 2025
We believe elastic#136747 fixed this test and/or the test was transitory and is
now fixed.
prwhelan added a commit that referenced this pull request Dec 18, 2025
We believe #136747 reduced the chance for this to happen.

Resolve #133603
prwhelan added a commit to prwhelan/elasticsearch that referenced this pull request Dec 18, 2025
We believe elastic#136747 reduced the chance for this to happen.

Resolve elastic#133603
elasticsearchmachine pushed a commit that referenced this pull request Dec 18, 2025
We believe #136747 reduced the chance for this to happen.

Resolve #133603
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:ml Machine learning >non-issue Team:ML Meta label for the ML team v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants