Skip to content

Allow the test to also accept partial, this seems to be normal#132614

Merged
smalyshev merged 2 commits intoelastic:mainfrom
smalyshev:unmute-128545
Aug 9, 2025
Merged

Allow the test to also accept partial, this seems to be normal#132614
smalyshev merged 2 commits intoelastic:mainfrom
smalyshev:unmute-128545

Conversation

@smalyshev
Copy link
Copy Markdown
Contributor

Sometimes when failure on remote happens, it returns "partial" as status with the error in "failures" instead of full failure.

Closes #128545

@smalyshev smalyshev requested a review from dnhatn August 9, 2025 04:41
@smalyshev smalyshev added >test Issues or PRs that are addressing/adding tests >test-failure Triaged test failures from CI :Analytics/ES|QL AKA ESQL v9.2.0 labels Aug 9, 2025
@smalyshev smalyshev marked this pull request as ready for review August 9, 2025 04:41
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) needs:risk Requires assignment of a risk label (low, medium, blocker) labels Aug 9, 2025
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

Copy link
Copy Markdown
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks Stas!

@smalyshev smalyshev merged commit 71ae0cc into elastic:main Aug 9, 2025
33 checks passed
@smalyshev
Copy link
Copy Markdown
Contributor Author

💚 All backports created successfully

Status Branch Result
9.1

Questions ?

Please refer to the Backport tool documentation

@smalyshev
Copy link
Copy Markdown
Contributor Author

💚 All backports created successfully

Status Branch Result
8.19

Questions ?

Please refer to the Backport tool documentation

smalyshev added a commit to smalyshev/elasticsearch that referenced this pull request Aug 14, 2025
…ic#132614)

(cherry picked from commit 71ae0cc)

# Conflicts:
#	muted-tests.yml
elasticsearchmachine pushed a commit that referenced this pull request Aug 14, 2025
…) (#132949)

(cherry picked from commit 71ae0cc)

# Conflicts:
#	muted-tests.yml
szybia added a commit to szybia/elasticsearch that referenced this pull request Aug 15, 2025
* upstream/8.19:
  [8.19] Fix MergeWithLowDiskSpaceIT testRelocationWhileForceMerging (elastic#132913)
  Allow the test to also accept partial, this seems to be normal (elastic#132614) (elastic#132949)
  Add `complete` attribute to .fleet-agents docs (elastic#127651) (elastic#132903)
  [8.19] docs: add 429 semantic_text troubleshooting (elastic#132747)
  Reduce logging levels for meter usage tests (elastic#131935) (elastic#131942)
  Add 8.19.2 Connectors release notes (elastic#132714)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL needs:risk Requires assignment of a risk label (low, medium, blocker) Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests >test-failure Triaged test failures from CI v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] CrossClusterQueryWithPartialResultsIT testFailToStartRequestOnRemoteCluster failing

3 participants