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

Fixing inference service test deprecation warning in search tests #204386

Conversation

jgowdyelastic
Copy link
Member

@jgowdyelastic jgowdyelastic commented Dec 16, 2024

Adds a model_id property to the model config and renames the service to elasticsearch, as elser has been deprecated.
This should avoid triggering API deprecation warning.

@jgowdyelastic jgowdyelastic self-assigned this Dec 16, 2024
@jgowdyelastic jgowdyelastic added :ml release_note:skip Skip the PR/issue when compiling release notes v9.0.0 testing Issues related to tests. v8.18.0 labels Dec 16, 2024
@jgowdyelastic jgowdyelastic marked this pull request as ready for review December 16, 2024 15:02
@jgowdyelastic jgowdyelastic requested a review from a team as a code owner December 16, 2024 15:02
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

Copy link
Contributor

@Samiul-TheSoccerFan Samiul-TheSoccerFan left a comment

Choose a reason for hiding this comment

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

LGTM

@jgowdyelastic jgowdyelastic added the backport:version Backport to applied version labels label Dec 16, 2024
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @jgowdyelastic

@jgowdyelastic jgowdyelastic merged commit 49a5b2f into elastic:main Dec 16, 2024
11 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12360365294

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 16, 2024
…astic#204386)

Adds a `model_id` property to the model config and renames the service
to `elasticsearch`, as `elser` has been deprecated.
This should avoid triggering API deprecation warning.

(cherry picked from commit 49a5b2f)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Dec 16, 2024
…ts (#204386) (#204480)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Fixing inference service test deprecation warning in search tests
(#204386)](#204386)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"James
Gowdy","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-16T20:03:47Z","message":"Fixing
inference service test deprecation warning in search tests
(#204386)\n\nAdds a `model_id` property to the model config and renames
the service\r\nto `elasticsearch`, as `elser` has been
deprecated.\r\nThis should avoid triggering API deprecation
warning.","sha":"49a5b2ff7ee809df5bddec88c5f1c2058c1e4b55","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":[":ml","release_note:skip","v9.0.0","testing","backport:version","v8.18.0"],"title":"Fixing
inference service test deprecation warning in search
tests","number":204386,"url":"https://github.com/elastic/kibana/pull/204386","mergeCommit":{"message":"Fixing
inference service test deprecation warning in search tests
(#204386)\n\nAdds a `model_id` property to the model config and renames
the service\r\nto `elasticsearch`, as `elser` has been
deprecated.\r\nThis should avoid triggering API deprecation
warning.","sha":"49a5b2ff7ee809df5bddec88c5f1c2058c1e4b55"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/204386","number":204386,"mergeCommit":{"message":"Fixing
inference service test deprecation warning in search tests
(#204386)\n\nAdds a `model_id` property to the model config and renames
the service\r\nto `elasticsearch`, as `elser` has been
deprecated.\r\nThis should avoid triggering API deprecation
warning.","sha":"49a5b2ff7ee809df5bddec88c5f1c2058c1e4b55"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: James Gowdy <[email protected]>
JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this pull request Dec 19, 2024
…astic#204386)

Adds a `model_id` property to the model config and renames the service
to `elasticsearch`, as `elser` has been deprecated.
This should avoid triggering API deprecation warning.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels :ml release_note:skip Skip the PR/issue when compiling release notes testing Issues related to tests. v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants