Open
Description
In #83808, new typings were introduced for the ES client.
There were a few places where @ts-expect-error
was added to our plugins. This is likely a problem in our code that should be addressed. If we find it is a problem in the client code, it should be reported to https://github.com/elastic/elastic-client-generator/issues/new/choose.
Known places:
- ILM -
register_fetch_route.ts
// @ts-expect-error Policy doesn't contain name property
- Data streams -
register_get_route.ts
// @ts-expect-error PrivilegesFromEs incompatible with ApplicationsPrivileges
- Painless Lab -
execute.ts
// @ts-expect-error `ExecutePainlessScriptRequest.body` does not allow `string`
- Painless Lab -
execute.ts
// @ts-expect-error `ExecutePainlessScriptRequest.body` does not allow `string`
- Snapshot Restore (once [Snapshot Restore] Migrate to new ES client #95499 is merged)
See theTODO
comments in these files:
x-pack/plugins/snapshot_restore/server/routes/api/policy.ts
x-pack/plugins/snapshot_restore/server/routes/api/repositories.ts
x-pack/plugins/snapshot_restore/server/routes/api/restore.ts
x-pack/plugins/snapshot_restore/server/routes/api/policy.ts
(Update @elastic/elasticsearch to 8.0.0-canary13 #98266)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment