[ES body removal] @elastic/observability-ui

[afharo]

Summary

in preparation for 9.0, we're trying to remove the deprecated body param in the ES client.

To make it easier to review, the changes have been split into multiple PRs, trying to group them per code owner as much as possible.

However, unfortunately, due to cross-dependencies, your team may be pinged more than once. Apologies for that additional noise.

What changes?

Nothing changes internally. The ES client already places everything where it should be when performing the request to ES (URL vs. query string vs. body params).

The main change is in the usage: when using the JS ES client, developers don't need to identify what goes in the body and what goes in the URL. All settings are provided at the root level. So, in summary, the change is:

const response = await client.search({
  index: 'test',
-  body: {
    query: {
      match_all: {}
    }
-  }
})

For this reason, enabling the "Hide whitespace changes" option when reviewing is recommended.

Some exceptions to this rule:

• Bulk APIs replace the body array with operations array (direct replacement)
• Index Put Settings API replace body array with settings (direct replacement)

[elasticmachine]

💔 Build Failed

• Buildkite Build
• Commit: b7afe1b

Failed CI Steps

• Jest Tests #11
• Jest Tests #11

Test Failures

• [job] [logs] Jest Tests #11 / HoverableAvatar renders the tooltip when hovering
• [job] [logs] Jest Tests #11 / useGetActionLicense unhappy path

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id	before	after	diff
observabilityShared	93.9KB	93.8KB	-7.0B

History

• 💔 Build #268605 failed 78b0834
• 💔 Build #262108 failed 26d47b2

cc @afharo

[elasticmachine]

Pinging @elastic/unified-observability (Team:Observability)

[elasticmachine]

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)