[Search] Fix indices route and remove obsolete code#226595

Merged

sphilipse merged 9 commits intoelastic:mainfrom

sphilipse:es3-fix-indices-route

Jul 9, 2025

Member

sphilipse commented Jul 4, 2025 •

edited

Loading

Summary

This fixes the Serverless search indices route, which was broken: the stats API it called never worked on Serverless, so that breaks the entire route.

Additionally, this removes an used documents component that was calling that route. That's probably the only route that calls that API, but we should double check. We will follow up on that.


          [ES3] Fix indices route and remove obsolete code

f811ac0

sphilipse requested a review from a team as a code owner

July 4, 2025 14:33

sphilipse added release_note:skip backport:skip Team:Search v9.2.0 labels

sphilipse changed the title ~~[ES3] Fix indices route and remove obsolete code~~ [Search] Fix indices route and remove obsolete code

kibanamachine and others added 2 commits

July 4, 2025 14:57


          [CI] Auto-commit changed files from 'node scripts/eslint_all_files --…

fc9afe3

…no-cache --fix'


          [ES3] Fix indices route and remove obsolete code

dc5cab9

saikatsarkar056 approved these changes

View reviewed changes


          Merge branch 'main' into es3-fix-indices-route

aaa1cf0

cnasikas added bug v9.1.0 v8.19.0 backport:version and removed backport:skip labels


          Merge branch 'main' into es3-fix-indices-route

59af1a7

sphilipse removed v9.1.0 v8.19.0 labels

sphilipse and others added 4 commits

July 8, 2025 10:29


          Fix tests

06395fa


          Remove route const

5740e84


          [CI] Auto-commit changed files from 'node scripts/eslint_all_files --…

83c9afb

…no-cache --fix'


          fix test

48bc8c7

Contributor

elasticmachine commented Jul 8, 2025

💛 Build succeeded, but was flaky

Buildkite Build
Commit: 48bc8c7

Failed CI Steps

FTR Configs #20

Test Failures

[job] [logs] FTR Configs #20 / management index management index templates simulate should simulate an index template by name with a related data stream

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
`serverlessSearch`	410	381	-29

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
`serverlessSearch`	346.5KB	323.0KB	-23.6KB

Page load bundle

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

id	before	after	diff
`serverlessSearch`	26.2KB	25.7KB	-585.0B

Unknown metric groups

async chunk count

id	before	after	diff
`serverlessSearch`	11	10	-1

ESLint disabled line counts

id	before	after	diff
`serverlessSearch`	7	6	-1

Total ESLint disabled count

id	before	after	diff
`serverlessSearch`	7	6	-1

History

sphilipse merged commit 596ce61 into elastic:main

13 checks passed

kibanamachine added the backport missing label

Contributor

kibanamachine commented Jul 11, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

3 similar comments

Contributor

kibanamachine commented Jul 14, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 15, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 16, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 17, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

5 similar comments

Contributor

kibanamachine commented Jul 18, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 21, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 22, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 23, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 24, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

kertal pushed a commit to kertal/kibana that referenced this pull request


          [Search] Fix indices route and remove obsolete code (elastic#226595)

6ba7584

## Summary

This fixes the Serverless search indices route, which was broken: the
stats API it called never worked on Serverless, so that breaks the
entire route.

Additionally, this removes an used documents component that was calling
that route. That's probably the only route that calls that API, but we
should double check. We will follow up on that.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Saikat Sarkar <132922331+saikatsarkar056@users.noreply.github.com>

Contributor

kibanamachine commented Jul 25, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

5 similar comments

Contributor

kibanamachine commented Jul 28, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 29, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 30, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Jul 31, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

Contributor

kibanamachine commented Aug 1, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226595 locally
cc: @sphilipse

sphilipse added the backport:skip label

kibanamachine removed the backport missing label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip backport:version bug release_note:skip Team:Search v9.2.0