Skip to content

Unmute SearchWithRejectionsIT testOpenContextsAfterRejections #130610

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

drempapis
Copy link
Contributor

I have thoroughly investigated this issue, including extensive debugging, but have been unable to identify any underlying problem. The failure cannot be reproduced locally, and it appears to be muted on the main and 9.1 branches, not 8.19 as reported here.

As a precaution, I have increased the assertBusy timeout, although I do not believe this change will have a significant impact.

I am unmuting the test to allow it to run again in CI for further evaluation and will revisit if the issue persists.

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.2.0 labels Jul 4, 2025
@drempapis drempapis added >test-failure Triaged test failures from CI Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch :Search Foundations/Search Catch all for Search Foundations v9.1.0 and removed needs:triage Requires assignment of a team area label labels Jul 4, 2025
@elasticsearchmachine elasticsearchmachine added the needs:risk Requires assignment of a risk label (low, medium, blocker) label Jul 4, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@drempapis drempapis self-assigned this Jul 4, 2025
@cbuescher cbuescher added >test Issues or PRs that are addressing/adding tests and removed >test-failure Triaged test failures from CI needs:risk Requires assignment of a risk label (low, medium, blocker) labels Jul 4, 2025
Copy link
Member

@cbuescher cbuescher 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 for the investigation, lets see if this is still an issue after unmuting.

@drempapis drempapis merged commit aaf7b73 into elastic:main Jul 4, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Search Foundations/Search Catch all for Search Foundations Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch >test Issues or PRs that are addressing/adding tests v9.1.0 v9.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants