Skip to content

Fix flaky test runner serverless flag for Search solution#248559

Merged
saikatsarkar056 merged 1 commit intoelastic:mainfrom
saikatsarkar056:fix/flaky-test-runner-search-serverless-flag
Jan 10, 2026
Merged

Fix flaky test runner serverless flag for Search solution#248559
saikatsarkar056 merged 1 commit intoelastic:mainfrom
saikatsarkar056:fix/flaky-test-runner-search-serverless-flag

Conversation

@saikatsarkar056
Copy link
Contributor

@saikatsarkar056 saikatsarkar056 commented Jan 9, 2026

Summary

The flaky test runner was using '--serverless=search' for the Search solution, but 'search' is not a valid serverless option. The valid options are: es, oblt, oblt-logs-essentials, security, security-essentials, security-ease.

Changed '--serverless=search' to '--serverless=es' for the Search solution to match the valid CLI options defined in kbn-scout.

This change resolves similar errors found in https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/10391/steps/canvas?jid=019ba419-e514-4657-bcbb-30650621ba0b

Checklist

Check the PR satisfies following conditions.

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.

Identify risks

Does this PR introduce any risks? For example, consider risks like hard to test bugs, performance regression, potential of data loss.

Describe the risk, its severity, and mitigation for each identified risk. Invite stakeholders and evaluate how to proceed before merging.

The flaky test runner was using '--serverless=search' for the Search
solution, but 'search' is not a valid serverless option. The valid
options are: es, oblt, oblt-logs-essentials, security, security-essentials,
security-ease.

Changed '--serverless=search' to '--serverless=es' for the Search solution
to match the valid CLI options defined in kbn-scout.
@saikatsarkar056 saikatsarkar056 requested a review from a team as a code owner January 9, 2026 23:45
@saikatsarkar056 saikatsarkar056 self-assigned this Jan 10, 2026
@saikatsarkar056 saikatsarkar056 added v9.4.0 test:scout release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Jan 10, 2026
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#10398

[✅] x-pack/solutions/search/plugins/search_homepage/test/scout/ui/parallel.playwright.config.ts: 15/15 tests passed.

see run history

@saikatsarkar056 saikatsarkar056 merged commit 17fccc3 into elastic:main Jan 10, 2026
25 checks passed
devamanv pushed a commit to devamanv/kibana that referenced this pull request Jan 12, 2026
…8559)

## Summary

The flaky test runner was using '--serverless=search' for the Search
solution, but 'search' is not a [valid serverless
option](https://github.com/elastic/kibana/blob/e586a3927b42e5a13c3710e2162b6dba03cfb77c/src/platform/packages/shared/kbn-scout/src/servers/flags.ts#L34).
The valid options are: es, oblt, oblt-logs-essentials, security,
security-essentials, security-ease.

Changed '--serverless=search' to '--serverless=es' for the Search
solution to match the valid CLI options defined in kbn-scout.

This change resolves similar errors found in
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/10391/steps/canvas?jid=019ba419-e514-4657-bcbb-30650621ba0b

### Checklist

Check the PR satisfies following conditions. 

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
- [x] [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)
- [ ] 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.

### Identify risks

Does this PR introduce any risks? For example, consider risks like hard
to test bugs, performance regression, potential of data loss.

Describe the risk, its severity, and mitigation for each identified
risk. Invite stakeholders and evaluate how to proceed before merging.

- [ ] [See some risk
examples](https://github.com/elastic/kibana/blob/main/RISK_MATRIX.mdx)
- [ ] ...
mbondyra added a commit to mbondyra/kibana that referenced this pull request Jan 12, 2026
* commit 'c4304e27736c62f17af20d145770b2ae9d3fae30': (418 commits)
  skip failing suite (elastic#89079)
  [ES|QL] Update grammars (elastic#248600)
  skip failing test suite (elastic#248579)
  [ES|QL] Update function metadata (elastic#248601)
  skip failing test suite (elastic#248554)
  Fix flaky test runner serverless flag for Search solution (elastic#248559)
  [Security Solution][Attacks/Alerts][Attacks page][Table section] Remember last selected attack details tab (Summary or Alerts) (elastic#247519) (elastic#247988)
  Fix ES health check poller (elastic#248496)
  Fix collector schema ownership (elastic#241292)
  [api-docs] 2026-01-10 Daily api_docs build (elastic#248574)
  Update dependency cssstyle to v5.3.5 (main) (elastic#237637)
  Update dependency @octokit/rest to v22.0.1 (main) (elastic#243102)
  skip failing test suite (elastic#248504)
  skip failing test suite (elastic#247685)
  Remove broken ecommerce_dashboard journeys (elastic#248162)
  [Obs AI] Hide AI Insight component when there are no connectors (elastic#248542)
  skip failing suite (elastic#248433)
  [Security Solution][Attacks/Alerts][Attacks page][Table section] Hide tabs for generic attack groups (elastic#248444)
  [Agent Builder] [AI Infra] Adds product documentation tool and task evals (elastic#248370)
  [Controls Anywhere] Keep controls focused when creating + editing other panels (elastic#248021)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes test:scout v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants