-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[ES client] Rename deprecated params #115528
[ES client] Rename deprecated params #115528
Conversation
💔 Build Failed
Failed CI StepsTest FailuresKibana Pipeline / general / X-Pack API Integration Tests.x-pack/test/api_integration/apis/management/snapshot_restore/policies·ts.apis management Snapshot and Restore SLM policies Create should create a SLM policyStandard Out
Stack Trace
Kibana Pipeline / general / X-Pack API Integration Tests.x-pack/test/api_integration/apis/management/snapshot_restore/policies·ts.apis management Snapshot and Restore SLM policies Create "after all" hook for "should create a policy with only required fields"Standard Out
Stack Trace
Kibana Pipeline / general / X-Pack API Integration Tests.x-pack/test/api_integration/apis/management/snapshot_restore/policies·ts.apis management Snapshot and Restore SLM policies Create should create a SLM policyStandard Out
Stack Trace
and 2 more failures, only showing the first 3. Metrics [docs]
To update your PR or re-run it, just comment with: |
💚 Build Succeeded
Metrics [docs]
To update your PR or re-run it, just comment with: |
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
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.
Security solution changes LGTM.
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.
Asset Management LGTM
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.
Rubberstamping approval. @elastic/kibana-app-services You are the owners for the file that was touched: https://github.com/elastic/kibana/pull/115528/files#diff-97c352c28918de15081fb5ef9d9405e42c42fb4539e114d186f908d20bf49f58
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.
app services change lgtm
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.
Canvas changes LGTM 👍
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.
security-solution LGTM
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
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.
Reporting changes LGTM
* filterPath --> filter_path * ignoreUnavailable --> ignore_unavailable * ignoreUnavailable --> ignore_unavailable * trackScores --> track_scores * trackTotalHits --> track_total_hits * rollback unnecessary changes
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
* filterPath --> filter_path * ignoreUnavailable --> ignore_unavailable * ignoreUnavailable --> ignore_unavailable * trackScores --> track_scores * trackTotalHits --> track_total_hits * rollback unnecessary changes Co-authored-by: Mikhail Shustov <restrry@gmail.com>
Summary
Extracted from #113950
In the next major release
@elastic/elasticsearch
client dropscamelCase
support for keys.Renames the next params:
I'm going to port the changes to
7.x
to avoid merge conflicts during backports.Let me know if you find other usages in the plugin code.