Skip to content

Fix ELAND endpoints not updating dimensions#126537

Merged
dan-rubinstein merged 2 commits intoelastic:mainfrom
dan-rubinstein:fix-dimension-updating-eland
Apr 9, 2025
Merged

Fix ELAND endpoints not updating dimensions#126537
dan-rubinstein merged 2 commits intoelastic:mainfrom
dan-rubinstein:fix-dimension-updating-eland

Conversation

@dan-rubinstein
Copy link
Member

@dan-rubinstein dan-rubinstein commented Apr 9, 2025

Description

In a previous change endpoint creation validation was added for all task types for ElasticsearchInternalService. As part of this change, a bug was introduced that stopped ELAND text embedding models from updating their dimensions after model validation causing endpoints to fail creation. This change adds back the dimension updating logic.

Testing

  • Unit tests
  • Created a text embedding ELAND endpoint and confirmed that dimensions are updated.
@dan-rubinstein dan-rubinstein added >bug :ml Machine learning Team:ML Meta label for the ML team auto-backport Automatically create backport pull requests when merged v8.19.0 v9.1.0 labels Apr 9, 2025
@elasticsearchmachine
Copy link
Collaborator

Hi @dan-rubinstein, I've created a changelog YAML for you.

@dan-rubinstein dan-rubinstein marked this pull request as ready for review April 9, 2025 14:59
@elasticsearchmachine
Copy link
Collaborator

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

@wwang500 wwang500 added the cloud-deploy Publish cloud docker image for Cloud-First-Testing label Apr 9, 2025
@dan-rubinstein dan-rubinstein merged commit 44507cc into elastic:main Apr 9, 2025
17 checks passed
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.x Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 126537

dan-rubinstein added a commit to dan-rubinstein/elasticsearch that referenced this pull request Apr 9, 2025
* Fix ELAND endpoints not updating dimensions

* Update docs/changelog/126537.yaml
dan-rubinstein added a commit that referenced this pull request Apr 9, 2025
* Fix ELAND endpoints not updating dimensions

* Update docs/changelog/126537.yaml
@dan-rubinstein dan-rubinstein deleted the fix-dimension-updating-eland branch April 9, 2025 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged >bug cloud-deploy Publish cloud docker image for Cloud-First-Testing :ml Machine learning Team:ML Meta label for the ML team v8.19.0 v9.1.0

6 participants