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

Harmonize ELSER model ID #2993

Open
wants to merge 4 commits into
base: 8.x
Choose a base branch
from
Open

Conversation

kosabogi
Copy link
Contributor

@kosabogi kosabogi commented Mar 13, 2025

📸 Preview

Related issue: https://github.com/elastic/search-docs-team/issues/258

Changes

Deprecation message after running the PUT _inference/sparse_embedding/my-elser-model call:

PUT _inference/sparse_embedding/my-elser-model

#! Putting elasticsearch service inference endpoints (including elser service) without a model_id field is deprecated and will be removed in a future release. Please specify a model_id field.
#! The [elser] service is deprecated and will be removed in a future release. Use the [elasticsearch] service instead, with [model_id] set to [.elser_model_2_linux-x86_64] in the [service_settings]

I've updated the example to reflect this change by:

  • Replacing "service": "elser" with "service": "elasticsearch".
  • Adding `"model_id"

@kosabogi kosabogi added backport-8.17 Automated backport with mergify backport-8.18 Automated backport with mergify labels Mar 13, 2025
@kosabogi kosabogi requested a review from dlockli March 13, 2025 11:59
@kosabogi kosabogi requested a review from a team as a code owner March 13, 2025 11:59
Copy link

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@dlockli
Copy link

dlockli commented Mar 13, 2025

LGTM @kosabogi!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8.17 Automated backport with mergify backport-8.18 Automated backport with mergify
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants