-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
chore(deps): upgrade @elastic/elasticsearch
#163822
Conversation
f829f28
to
da26748
Compare
Pinging @elastic/kibana-core (Team:Core) |
Pinging @elastic/kibana-security (Team:Security) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one nit, thanks!
version "8.3.2" | ||
resolved "https://registry.yarnpkg.com/@elastic/transport/-/transport-8.3.2.tgz#295e91f43e3a60a839f998ac3090a83ddb441592" | ||
integrity sha512-ZiBYRVPj6pwYW99fueyNU4notDf7ZPs7Ix+4T1btIJsKJmeaORIItIfs+0O7KV4vV+DcvyMhkY1FXQx7kQOODw== | ||
"@elastic/transport@^8.3.3": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: it looks like we need to join this declaration with the declaration above as they are semver-compatible (@elastic/transport@^8.3.1
). Yarn sometimes cannot do this automatically, so we need to manually remove declarations from yarn.lock
and re-run yarn kbn bootstrap
.
"@elastic/transport@^8.3.1", "@elastic/transport@^8.3.3":
version "8.3.3"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Done in bdeaf61
(#163822)
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: cc @afharo |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
(cherry picked from commit 5d2571a)
# Backport This will backport the following commits from `main` to `8.9`: - [chore(deps): upgrade `@elastic/elasticsearch` (#163822)](#163822) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Alejandro Fernández Haro","email":"alejandro.haro@elastic.co"},"sourceCommit":{"committedDate":"2023-08-16T11:09:52Z","message":"chore(deps): upgrade `@elastic/elasticsearch` (#163822)","sha":"5d2571abd5681615f648086d3d0132f5a12f2058","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","Team:Security","release_note:skip","backport:prev-minor","v8.10.0"],"number":163822,"url":"https://github.com/elastic/kibana/pull/163822","mergeCommit":{"message":"chore(deps): upgrade `@elastic/elasticsearch` (#163822)","sha":"5d2571abd5681615f648086d3d0132f5a12f2058"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/163822","number":163822,"mergeCommit":{"message":"chore(deps): upgrade `@elastic/elasticsearch` (#163822)","sha":"5d2571abd5681615f648086d3d0132f5a12f2058"}}]}] BACKPORT--> --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
This pr didn't make it into the build candidate for v8.9.1. Updating the labels. |
Summary
Upgrades the Elasticsearch client to the latest to take advantage of the changes in elastic/elastic-transport-js#69
For maintainers