Skip to content

Fix testCancelViaExpirationOnRemoteResultsWithMinimizeRoundtrips test#145130

Merged
smalyshev merged 4 commits intoelastic:mainfrom
smalyshev:fix-143407
Mar 30, 2026
Merged

Fix testCancelViaExpirationOnRemoteResultsWithMinimizeRoundtrips test#145130
smalyshev merged 4 commits intoelastic:mainfrom
smalyshev:fix-143407

Conversation

@smalyshev
Copy link
Copy Markdown
Contributor

We can collect expiration time directly from submit response. This way we don't need the second get which may expire prematurely and cause failure.

Fixes #143407

@smalyshev smalyshev added >test Issues or PRs that are addressing/adding tests :Search Foundations/CCS v9.4.0 v8.19.14 v9.3.3 auto-backport Automatically create backport pull requests when merged labels Mar 27, 2026
@smalyshev smalyshev marked this pull request as ready for review March 27, 2026 18:49
@elasticsearchmachine elasticsearchmachine added the Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch label Mar 27, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-search-foundations (Team:Search Foundations)

Copy link
Copy Markdown
Contributor

@chrisparrinello chrisparrinello left a comment

Choose a reason for hiding this comment

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

LGTM

@smalyshev smalyshev enabled auto-merge (squash) March 27, 2026 18:56
@elasticsearchmachine elasticsearchmachine added the serverless-linked Added by automation, don't add manually label Mar 27, 2026
@smalyshev smalyshev merged commit 6043968 into elastic:main Mar 30, 2026
36 checks passed
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

💔 Backport failed

Status Branch Result
9.3 Commit could not be cherrypicked due to conflicts
8.19 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 145130

@smalyshev
Copy link
Copy Markdown
Contributor Author

💔 Some backports could not be created

Status Branch Result
9.3
8.19 An unhandled error occurred. Please see the logs for details

Manual backport

To create the backport manually run:

backport --pr 145130

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 Mar 30, 2026
mamazzol pushed a commit to mamazzol/elasticsearch that referenced this pull request Mar 30, 2026
smalyshev added a commit that referenced this pull request Mar 30, 2026
…#145130) (#145198)

(cherry picked from commit 6043968)

# Conflicts:
#	muted-tests.yml
smalyshev added a commit that referenced this pull request Mar 30, 2026
…#145130) (#145196)

(cherry picked from commit 6043968)

# Conflicts:
#	muted-tests.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending :Search Foundations/CCS serverless-linked Added by automation, don't add manually Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch >test Issues or PRs that are addressing/adding tests v8.19.14 v9.3.3 v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] CrossClusterAsyncSearchIT testCancelViaExpirationOnRemoteResultsWithMinimizeRoundtrips failing

3 participants