[Search][Telemetry] Add data-test-subj eslint check missing Search plugins by efegurkan · Pull Request #265004 · elastic/kibana

efegurkan · 2026-04-22T11:24:49Z

Summary

Adds ESLint config for telemetry check in missing Search plugins.

Checklist

Check the PR satisfies following conditions.

Created with Claude Code instructed by @efegurkan

Reviewers should verify this PR satisfies this list as well.

~~Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support~~
~~Documentation was added for features that require explanation or tutorials~~
~~Unit or functional tests were updated or added to match the most common scenarios~~
~~If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list~~
This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The release_note:breaking label should be applied in these situations.
~~Flaky Test Runner was used on any tests changed~~
The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
Review the backport guidelines and apply applicable backport:* labels.

…erence_endpoints

elasticmachine · 2026-04-22T12:48:55Z

💛 Build succeeded, but was flaky

Buildkite Build
Commit: 98712d8

Failed CI Steps

FTR Configs #204

Metrics [docs]

Async chunks

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

id	before	after	diff
`contentConnectors`	473.8KB	475.4KB	+1.6KB
`searchInferenceEndpoints`	214.9KB	215.5KB	+617.0B
total			+2.2KB

efegurkan · 2026-04-24T12:46:21Z

@elasticmachine merge upstream

efegurkan · 2026-04-27T11:33:25Z

@elasticmachine merge upstream

kibanamachine · 2026-04-27T12:38:58Z

💛 Build succeeded, but was flaky

Buildkite Build
Commit: cdefd63

Failed CI Steps

FTR Configs #88

Metrics [docs]

Async chunks

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

id	before	after	diff
`contentConnectors`	473.8KB	475.4KB	+1.6KB
`searchInferenceEndpoints`	215.8KB	216.4KB	+617.0B
total			+2.2KB

seialkali

This may be nitpicky, but some of these names are quite long (e.g., searchInferenceEndpointsElasticInferenceServiceModelsHeaderViewCloudUsageButton at 79 characters) and become difficult to parse at a glance.

Is the general consensus that readability for these IDs isn't a concern as long as they're unique? If so, I'm happy to approve.

sphilipse · 2026-04-28T07:17:28Z

This may be nitpicky, but some of these names are quite long (e.g., searchInferenceEndpointsElasticInferenceServiceModelsHeaderViewCloudUsageButton at 79 characters) and become difficult to parse at a glance.

Is the general consensus that readability for these IDs isn't a concern as long as they're unique? If so, I'm happy to approve.

Yup, readability doesn't matter :)

sphilipse

LGTM!

efegurkan · 2026-04-28T09:54:49Z

Names are autogenerated most of the time, and readability is not important as long as they are unique they are fine.

kibanamachine · 2026-04-28T09:55:51Z

Starting backport for target branches: 8.19, 9.2, 9.3, 9.4

https://github.com/elastic/kibana/actions/runs/25046244045

kibanamachine · 2026-04-28T10:05:42Z

💔 All backports failed

Status	Branch	Result
❌	8.19	Backport failed because of merge conflicts
❌	9.2	Backport failed because of merge conflicts
❌	9.3	Backport failed because of merge conflicts You might need to backport the following PRs to 9.3: - Add `announceOnMount` to conditionally rendered `EuiCallOut` components (#265724) - [Security GenAI] Use stable IDs for Security AI Prompt saved object filenames (#265301) - Fix to the jest test - FilterLabel should render properly (#265750)
❌	9.4	Backport failed because of merge conflicts You might need to backport the following PRs to 9.4: - auto-implement: Labels in model endpoints table of the model details flyout look misaligned (#263770)

Manual backport

To create the backport manually run:

node scripts/backport --pr 265004

Questions ?

Please refer to the Backport tool documentation

@efegurkan

…ugins (elastic#265004) ## Summary Adds ESLint config for telemetry check in missing Search plugins. ### Checklist Check the PR satisfies following conditions. Created with Claude Code instructed by @efegurkan Reviewers should verify this PR satisfies this list as well. - [ ] ~Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)~ - [ ] ~[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html) was added for features that require explanation or tutorials~ - [ ] ~[Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios~ - [ ] ~If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the [docker list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker)~ - [x] This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The `release_note:breaking` label should be applied in these situations. - [ ] ~[Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed~ - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels. Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

kibanamachine · 2026-04-29T10:49:10Z

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 265004 locally
cc: @efegurkan

kibanamachine · 2026-04-30T11:49:15Z

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 265004 locally
cc: @efegurkan

kibanamachine · 2026-05-01T12:49:46Z

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 265004 locally
cc: @efegurkan

Add data-test-subj eslint check for content_connectors and search_inf…

98712d8

…erence_endpoints

efegurkan requested review from a team as code owners April 22, 2026 11:24

efegurkan added release_note:skip Skip the PR/issue when compiling release notes backport:all-open Backport to all branches that could still receive a release labels Apr 22, 2026

jbudz approved these changes Apr 23, 2026

View reviewed changes

Merge branch 'main' into telemetry-plugin-config-connectors-inference

e7f4c5b

Merge branch 'main' into telemetry-plugin-config-connectors-inference

cdefd63

seialkali reviewed Apr 27, 2026

View reviewed changes

sphilipse approved these changes Apr 28, 2026

View reviewed changes

efegurkan merged commit e4bc92e into elastic:main Apr 28, 2026
19 checks passed

kibanamachine added the v9.5.0 label Apr 28, 2026

kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 29, 2026

Conversation

efegurkan commented Apr 22, 2026 • edited by kibanamachine Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Checklist

Uh oh!

elasticmachine commented Apr 22, 2026

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Async chunks

Uh oh!

efegurkan commented Apr 24, 2026

Uh oh!

efegurkan commented Apr 27, 2026

Uh oh!

kibanamachine commented Apr 27, 2026

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Async chunks

Uh oh!

seialkali left a comment

Choose a reason for hiding this comment

Uh oh!

sphilipse commented Apr 28, 2026

Uh oh!

sphilipse left a comment

Choose a reason for hiding this comment

Uh oh!

efegurkan commented Apr 28, 2026

Uh oh!

Uh oh!

kibanamachine commented Apr 28, 2026

Uh oh!

kibanamachine commented Apr 28, 2026

💔 All backports failed

Manual backport

Questions ?

Uh oh!

kibanamachine commented Apr 29, 2026

Uh oh!

kibanamachine commented Apr 30, 2026

Uh oh!

kibanamachine commented May 1, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

efegurkan commented Apr 22, 2026 •

edited by kibanamachine

Loading