Skip to content

Conversation

Copilot
Copy link

@Copilot Copilot AI commented Sep 24, 2025

This PR reverts commit 6f0ed56307a21eb2ed7d6f61219680abdcd22357 which muted the test org.elasticsearch.index.IndexingPressureIT.testWriteCanBeRejectedAtPrimaryLevel due to issue #131151.

What was changed

The original commit added a mute entry to muted-tests.yml to disable the failing test. This revert removes that mute entry, allowing the test to run again.

Removed from muted-tests.yml:

- class: org.elasticsearch.index.IndexingPressureIT
  method: testWriteCanBeRejectedAtPrimaryLevel
  issue: https://github.com/elastic/elasticsearch/issues/131151

Impact

  • The testWriteCanBeRejectedAtPrimaryLevel test in IndexingPressureIT will now be executed as part of the test suite
  • This unmutes a test that was previously disabled, potentially revealing if the underlying issue has been resolved
  • No functional code changes - only test execution behavior is affected

The revert is clean and only removes the exact lines that were added in the original muting commit.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@elasticsearchmachine
Copy link
Collaborator

@copilot-swe-agent please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation.

@elasticsearchmachine elasticsearchmachine added v9.1.5 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Sep 24, 2025
@Copilot Copilot AI changed the title [WIP] revert commit 6f0ed56307a21eb2ed7d6f61219680abdcd22357 Revert "Mute org.elasticsearch.index.IndexingPressureIT testWriteCanBeRejectedAtPrimaryLevel #131151" Sep 24, 2025
@Copilot Copilot AI requested a review from lkts September 24, 2025 20:34
Copilot finished work on behalf of lkts September 24, 2025 20:34
@lkts lkts added the >test Issues or PRs that are addressing/adding tests label Sep 24, 2025
@lkts lkts marked this pull request as ready for review September 24, 2025 20:37
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Sep 24, 2025
@lkts lkts added the :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. label Sep 24, 2025
@elasticsearchmachine elasticsearchmachine added Team:Distributed Indexing Meta label for Distributed Indexing team and removed needs:triage Requires assignment of a team area label labels Sep 24, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-indexing (Team:Distributed Indexing)

@lkts
Copy link
Contributor

lkts commented Sep 24, 2025

buildkite test this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Distributed Indexing Meta label for Distributed Indexing team >test Issues or PRs that are addressing/adding tests v9.1.6

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants