Skip to content

Conversation

@Mikep86
Copy link
Contributor

@Mikep86 Mikep86 commented Nov 19, 2024

Increase the timeout used in inference YAML tests for model download. Fixes #116899.

@Mikep86 Mikep86 added >non-issue :ml Machine learning v9.0.0 labels Nov 19, 2024
@Mikep86 Mikep86 requested a review from davidkyle November 19, 2024 19:14
@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label Nov 19, 2024
@elasticsearchmachine
Copy link
Collaborator

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

Comment on lines -234 to -236
- class: org.elasticsearch.xpack.inference.InferenceRestIT
method: test {p0=inference/30_semantic_text_inference/Calculates embeddings using the default ELSER 2 endpoint}
issue: https://github.com/elastic/elasticsearch/issues/116542
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also re-enable this test, which was fixed by #116749

@kderusso
Copy link
Member

Approved pending green CI :)

Copy link
Member

@davidkyle davidkyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dnhatn dnhatn merged commit 4e04a7b into elastic:main Nov 21, 2024
16 checks passed
@dnhatn
Copy link
Member

dnhatn commented Nov 21, 2024

I've merged this PR to unblock an ongoing release.

alexey-ivanov-es pushed a commit to alexey-ivanov-es/elasticsearch that referenced this pull request Nov 28, 2024
Increase the timeout used in inference YAML tests for model download. 

Fixes elastic#116899.
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.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] InferenceRestIT class failing

5 participants