Skip to content

Ignore the reason and retry systematically#246830

Merged
gsoldevila merged 4 commits intoelastic:mainfrom
gsoldevila:retry-on-any-search-phase-execution-error
Dec 22, 2025
Merged

Ignore the reason and retry systematically#246830
gsoldevila merged 4 commits intoelastic:mainfrom
gsoldevila:retry-on-any-search-phase-execution-error

Conversation

@gsoldevila
Copy link
Contributor

@gsoldevila gsoldevila commented Dec 18, 2025 •
edited by kibanamachine
Loading

Summary

Follow-up of #246533

We don't need to be that nitpicky about the reason. The search_phase_execution_error is likely a temporary situation and retrying will be a more efficient strategy than Pod restart + retrying.

@gsoldevila gsoldevila requested a review from a team as a code owner December 18, 2025 10:39
@gsoldevila gsoldevila added Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// 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 Dec 18, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Dec 18, 2025

Pinging @elastic/kibana-core (Team:Core)

@elastic-vault-github-plugin-prod elastic-vault-github-plugin-prod bot requested a review from a team as a code owner December 18, 2025 13:12
@gsoldevila gsoldevila force-pushed the retry-on-any-search-phase-execution-error branch from 383f77e to a674011 Compare December 18, 2025 13:42
@gsoldevila gsoldevila removed request for a team, szwarckonrad and tomsonpl December 18, 2025 13:42
@elasticmachine
Copy link
Contributor

elasticmachine commented Dec 22, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #116 / Endpoint Policy @ess @serverless Creation of DOT indices for elastic defend policies "before each" hook for "should create indices when endpoint integration policy is created"

Metrics [docs]

✅ unchanged

History

@gsoldevila gsoldevila merged commit 9028f92 into elastic:main Dec 22, 2025
13 checks passed
@kibanamachine
Copy link
Contributor

kibanamachine commented Dec 22, 2025

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

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

kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Dec 22, 2025
@gsoldevila @kibanamachine
Ignore the reason and retry systematically (elastic#246830)
f561f20 
## Summary

Follow-up of elastic#246533

We don't need to be that nitpicky about the reason. The
`search_phase_execution_error` is likely a temporary situation and
retrying will be a more efficient strategy than Pod restart + retrying.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 9028f92)
@kibanamachine kibanamachine mentioned this pull request Dec 22, 2025
Merged
@kibanamachine
Copy link
Contributor

kibanamachine commented Dec 22, 2025

💔 Some backports could not be created

Status Branch Result
❌ 8.19 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.19:
- Expose isRetryableEsClientError (#228315)
❌ 9.1 Backport failed because of merge conflicts

You might need to backport the following PRs to 9.1:
- Expose isRetryableEsClientError (#228315)
❌ 9.2 Backport failed because of merge conflicts

You might need to backport the following PRs to 9.2:
- Expose isRetryableEsClientError (#228315)
✅ 9.3

Note: Successful backport PRs will be merged automatically after passing CI.

Manual backport

To create the backport manually run:

node scripts/backport --pr 246830

Questions ?

Please refer to the Backport tool documentation

mbondyra added a commit to mbondyra/kibana that referenced this pull request Dec 22, 2025
@mbondyra
Merge commit '51756d2722200a991607658d48ecda50aeb04a7d' into dashboar…
7f313c1 
…d_step_back

* commit '51756d2722200a991607658d48ecda50aeb04a7d': (76 commits)
  [Synthetics] Fix SyncGlobalParamsSpaces flaky test (elastic#246487)
  [Synthetics] Fix useSyntheticsRules test (elastic#247259)
  [ES|QL] Fix index editor flaky test (elastic#247233)
  [ResponseOps][Reporting] Fix "failed to decrypt apiKey" error while disabling/enabling scheduled reports (elastic#247236)
  [Console] Update console definitions (main) (elastic#247214)
  [ES|QL] Esql indentation shortcut on the editor (elastic#247234)
  [Streams] Use original request for rules client when in default space (elastic#247014)
  Consolidate and improve unflattenObject (elastic#246725)
  [scout] use svl mode to run api-int tests (elastic#247223)
  SIEM Readiness V2 (elastic#245776)
  [ObsPresentation][A11y] Fix asset details flyout header announcement (elastic#246872)
  [Streams] Add abort support and silent mode for stream description generation (elastic#247082)
  [SLO] Add environment context to SLO feedback button (elastic#247221)
  Ignore the reason and retry systematically (elastic#246830)
  Update dependency @types/moment-duration-format to ^2.2.7 (main) (elastic#242221)
  [Streams 🌊 ] Add explicit waits for data grid rows before clicking expand button (elastic#246919)
  [Security Solution][Entity Analytics][Risk Scoring] Handle special characters in ESQL query for risk scoring (elastic#247060)
  [ML] Data frame analytics: Updates page headers (elastic#247097)
  [ES|QL] Build function arguments suggestions from hints (elastic#246736)
  Update dependency @hey-api/openapi-ts to v0.88.1 (main) (elastic#247210)
  ...
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Dec 23, 2025
@kibanamachine
Copy link
Contributor

kibanamachine commented Dec 23, 2025

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @gsoldevila

gsoldevila added a commit to gsoldevila/kibana that referenced this pull request Dec 23, 2025
@gsoldevila @kibanamachine
Ignore the reason and retry systematically (elastic#246830)
c198a19 
## Summary

Follow-up of elastic#246533

We don't need to be that nitpicky about the reason. The
`search_phase_execution_error` is likely a temporary situation and
retrying will be a more efficient strategy than Pod restart + retrying.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
@gsoldevila gsoldevila removed backport missing Added to PRs automatically when the are determined to be missing a backport. backport:all-open Backport to all branches that could still receive a release labels Dec 23, 2025
@gsoldevila gsoldevila added the backport:skip This PR does not require backporting label Dec 23, 2025
gsoldevila added a commit to gsoldevila/kibana that referenced this pull request Dec 23, 2025
@gsoldevila @kibanamachine
Ignore the reason and retry systematically (elastic#246830)
ce8d6c1 
## Summary

Follow-up of elastic#246533

We don't need to be that nitpicky about the reason. The
`search_phase_execution_error` is likely a temporary situation and
retrying will be a more efficient strategy than Pod restart + retrying.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
gsoldevila added a commit to gsoldevila/kibana that referenced this pull request Dec 23, 2025
@gsoldevila @kibanamachine
Ignore the reason and retry systematically (elastic#246830)
8f9bee3 
Follow-up of elastic#246533

We don't need to be that nitpicky about the reason. The
`search_phase_execution_error` is likely a temporary situation and
retrying will be a more efficient strategy than Pod restart + retrying.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Jan 6, 2026
@gsoldevila @kibanamachine @CAWilson94
Ignore the reason and retry systematically (elastic#246830)
fa1c20d 
## Summary

Follow-up of elastic#246533

We don't need to be that nitpicky about the reason. The
`search_phase_execution_error` is likely a temporary situation and
retrying will be a more efficient strategy than Pod restart + retrying.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
dej611 pushed a commit to dej611/kibana that referenced this pull request Jan 8, 2026
@gsoldevila @kibanamachine @dej611
Ignore the reason and retry systematically (elastic#246830)
cad8bb7 
## Summary

Follow-up of elastic#246533

We don't need to be that nitpicky about the reason. The
`search_phase_execution_error` is likely a temporary situation and
retrying will be a more efficient strategy than Pod restart + retrying.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Feb 5, 2026
@kibanamachine @gsoldevila @afharo
[9.3] Ignore the reason and retry systematically (#246830) (#247258)
5fd9a62 
# Backport

This will backport the following commits from `main` to `9.3`:
- [Ignore the reason and retry systematically
(#246830)](#246830)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Gerard
Soldevila","email":"gerard.soldevila@elastic.co"},"sourceCommit":{"committedDate":"2025-12-22T13:04:38Z","message":"Ignore
the reason and retry systematically (#246830)\n\n## Summary\n\nFollow-up
of https://github.com/elastic/kibana/pull/246533\n\nWe don't need to be
that nitpicky about the reason. The\n`search_phase_execution_error` is
likely a temporary situation and\nretrying will be a more efficient
strategy than Pod restart + retrying.\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"9028f920c9c9f142d6125eeeff0946188f3d1305","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","release_note:skip","backport:all-open","v9.4.0"],"title":"Ignore
the reason and retry
systematically","number":246830,"url":"https://github.com/elastic/kibana/pull/246830","mergeCommit":{"message":"Ignore
the reason and retry systematically (#246830)\n\n## Summary\n\nFollow-up
of https://github.com/elastic/kibana/pull/246533\n\nWe don't need to be
that nitpicky about the reason. The\n`search_phase_execution_error` is
likely a temporary situation and\nretrying will be a more efficient
strategy than Pod restart + retrying.\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"9028f920c9c9f142d6125eeeff0946188f3d1305"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/246830","number":246830,"mergeCommit":{"message":"Ignore
the reason and retry systematically (#246830)\n\n## Summary\n\nFollow-up
of https://github.com/elastic/kibana/pull/246533\n\nWe don't need to be
that nitpicky about the reason. The\n`search_phase_execution_error` is
likely a temporary situation and\nretrying will be a more efficient
strategy than Pod restart + retrying.\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"9028f920c9c9f142d6125eeeff0946188f3d1305"}}]}]
BACKPORT-->

Co-authored-by: Gerard Soldevila <gerard.soldevila@elastic.co>
Co-authored-by: Alejandro Fernández Haro <alejandro.haro@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jloleysens jloleysens jloleysens approved these changes

Assignees

No one assigned

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// v9.3.0 v9.4.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@gsoldevila @elasticmachine @kibanamachine @jloleysens