Skip to content

[ES|QL] Reports correctly the controls server side errors#263020

Merged
stratoula merged 1 commit intoelastic:mainfrom
stratoula:fix-esql-controls-server-error
Apr 14, 2026
Merged

[ES|QL] Reports correctly the controls server side errors#263020
stratoula merged 1 commit intoelastic:mainfrom
stratoula:fix-esql-controls-server-error

Conversation

@stratoula
Copy link
Copy Markdown
Contributor

@stratoula stratoula commented Apr 14, 2026 •
edited by kibanamachine
Loading

Summary

In the ES|QL controls we don't pass the server side errors correctly so they are not displayed at the editor. This PR is fixing it

image

@stratoula stratoula added Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana t// v9.4.0 v9.5.0 release_note:fix backport:version Backport to applied version labels labels Apr 14, 2026
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 14, 2026

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #180 / AgentBuilder Endpoints SML internal API POST /internal/agent_builder/sml/_attach attaches SML items and persists conversation attachment refs
  • [job] [logs] FTR Configs #92 / dashboard app - esql controls dashboard - add a value type ES|QL control should add an ES|QL value control
  • [job] [logs] affected Scout: [ platform / streams_app-stateful-classic ] plugin / local-stateful-classic - Attachments - shows empty attachments prompt when navigating to attachments tab

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
esql 827.8KB 827.8KB -7.0B

@stratoula stratoula marked this pull request as ready for review April 14, 2026 13:33
@stratoula stratoula requested a review from a team as a code owner April 14, 2026 13:33
sddonne
sddonne approved these changes Apr 14, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@sddonne sddonne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, only code review.

@stratoula stratoula merged commit 89f0c1e into elastic:main Apr 14, 2026
36 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Apr 14, 2026

Starting backport for target branches: 9.4

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Apr 14, 2026
@stratoula
[ES|QL] Reports correctly the controls server side errors (elastic#26…
f58ac8d 
…3020)

## Summary

In the ES|QL controls we don't pass the server side errors correctly so
they are not displayed at the editor. This PR is fixing it

<img width="577" height="337" alt="image"
src="https://github.com/user-attachments/assets/01bfb2ff-a1fe-4c77-89e4-a93c5dd24ec0"
/>

(cherry picked from commit 89f0c1e)
@kibanamachine kibanamachine mentioned this pull request Apr 14, 2026
Merged
@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Apr 14, 2026

💚 All backports created successfully

Status Branch Result
✅ 9.4

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

Questions ?

Please refer to the Backport tool documentation

mbondyra added a commit to mbondyra/kibana that referenced this pull request Apr 14, 2026
@mbondyra
Merge commit '11ed3645c5ededae2a6e29f2a79b31f52208b441' into pr/262450
bfea6d9 
* commit '11ed3645c5ededae2a6e29f2a79b31f52208b441': (157 commits)
  remove sync register uiAction methods (elastic#254590)
  [performance] Apply minimal auth to the search route (elastic#257497)
  [ES|QL] Reports correctly the controls server side errors (elastic#263020)
  [SecuritySolution][Navigation] Enable classic nav updates (elastic#262358)
  [Inference] Use pretty name and logo on feature settings page (elastic#262531)
  [Security Solution] fix AT-AB cypress test (elastic#262991)
  [SigEvents] Seed sigevents env script (elastic#261172)
  Adjust conditions for validating no refetch for expanded row (elastic#262978)
  [Agent Builder] update copy for the announcement modal (elastic#263034)
  [Search] Hide index management links for users without privileges (elastic#262627)
  Simplify OAS schema for GET `/api/spaces/space` query params (elastic#260831)
  Fix fleet output OAS regressions: SSL type explosion and Kafka union wrappers (elastic#260842)
  [Dashboards in chat] fix agent confusing the axes in a horizontal chat (elastic#263064)
  [One Workflow] Add alert state checkbox UI for workflow connector (elastic#259770)
  [One Workflow] Deprecate legacy Cases step types in workflow authoring (elastic#262070)
  skip failing test suite (elastic#248090)
  fix flaky test: MonitorDetails filter apply button not enabled (elastic#260788)
  fix: propagate AbortSignal to executeAsReasoningAgent for task cancellation (elastic#262811)
  [Security Solution][Alert KPI] Fix white space bug in alert KPIs (elastic#260803)
  [Streams] Move helpers and format_size_unit to utils folder (elastic#262550)
  ...

# Conflicts:
#	x-pack/platform/plugins/shared/dashboard_agent/public/attachment_types/canvas_integration/dashboard_canvas_content.test.tsx
#	x-pack/platform/plugins/shared/dashboard_agent/public/attachment_types/canvas_integration/dashboard_canvas_content.tsx
#	x-pack/platform/plugins/shared/dashboard_agent/public/attachment_types/canvas_integration/use_register_canvas_action_buttons.ts
#	x-pack/platform/plugins/shared/dashboard_agent/public/attachment_types/index.test.tsx
#	x-pack/platform/plugins/shared/dashboard_agent/public/attachment_types/index.tsx
kibanamachine added a commit that referenced this pull request Apr 14, 2026
@kibanamachine @stratoula
[9.4] [ES|QL] Reports correctly the controls server side errors (#263020
d776e2f 
) (#263101)

# Backport

This will backport the following commits from `main` to `9.4`:
- [[ES|QL] Reports correctly the controls server side errors
(#263020)](#263020)

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

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

<!--BACKPORT
[{"author":{"name":"Stratou","email":"efstratia.kalafateli@elastic.co"},"sourceCommit":{"committedDate":"2026-04-14T14:31:31Z","message":"[ES|QL]
Reports correctly the controls server side errors (#263020)\n\n##
Summary\n\nIn the ES|QL controls we don't pass the server side errors
correctly so\nthey are not displayed at the editor. This PR is fixing
it\n\n<img width=\"577\" height=\"337\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/01bfb2ff-a1fe-4c77-89e4-a93c5dd24ec0\"\n/>","sha":"89f0c1eca85fc66de25de83e2697737892f12743","branchLabelMapping":{"^v9.5.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Feature:ES|QL","Team:ESQL","backport:version","v9.4.0","v9.5.0"],"title":"[ES|QL]
Reports correctly the controls server side
errors","number":263020,"url":"https://github.com/elastic/kibana/pull/263020","mergeCommit":{"message":"[ES|QL]
Reports correctly the controls server side errors (#263020)\n\n##
Summary\n\nIn the ES|QL controls we don't pass the server side errors
correctly so\nthey are not displayed at the editor. This PR is fixing
it\n\n<img width=\"577\" height=\"337\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/01bfb2ff-a1fe-4c77-89e4-a93c5dd24ec0\"\n/>","sha":"89f0c1eca85fc66de25de83e2697737892f12743"}},"sourceBranch":"main","suggestedTargetBranches":["9.4"],"targetPullRequestStates":[{"branch":"9.4","label":"v9.4.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.5.0","branchLabelMappingKey":"^v9.5.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/263020","number":263020,"mergeCommit":{"message":"[ES|QL]
Reports correctly the controls server side errors (#263020)\n\n##
Summary\n\nIn the ES|QL controls we don't pass the server side errors
correctly so\nthey are not displayed at the editor. This PR is fixing
it\n\n<img width=\"577\" height=\"337\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/01bfb2ff-a1fe-4c77-89e4-a93c5dd24ec0\"\n/>","sha":"89f0c1eca85fc66de25de83e2697737892f12743"}}]}]
BACKPORT-->

Co-authored-by: Stratou <efstratia.kalafateli@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sddonne sddonne sddonne approved these changes

Assignees

No one assigned

Labels

backport:version Backport to applied version labels Feature:ES|QL ES|QL related features in Kibana release_note:fix Team:ESQL ES|QL related features in Kibana t// v9.4.0 v9.5.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@stratoula @elasticmachine @kibanamachine @sddonne