Skip to content

Add shard routing and state transitions logging to testFilterCacheStats #125836

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 28, 2025
Merged

Add shard routing and state transitions logging to testFilterCacheStats #125836

merged 1 commit into from
Mar 28, 2025

Conversation

@andreidan
Copy link
Contributor

This unmutes and adds extra logging for the testFilterCacheStats test.

The test sporadically fails on

assertThat(
                        indexShard.getLastSyncedGlobalCheckpoint(),
                        equalTo(indexShard.getLastKnownGlobalCheckpoint())
                    );

whenever the index under test has a high primary shards count (8).

We'd want to monitor the cluster state transitions and the shard routing
when this test fails again.

Relates to #124447

@andreidan andreidan added >test Issues or PRs that are addressing/adding tests :Search Foundations/Search Catch all for Search Foundations v9.1.0 labels Mar 28, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch label Mar 28, 2025
@elasticsearchmachine
Copy link
Collaborator

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

original-brownbear
original-brownbear approved these changes Mar 28, 2025
View reviewed changes
Copy link
Contributor

@original-brownbear original-brownbear left a comment

Choose a reason for hiding this comment

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

LGTM

@andreidan andreidan merged commit b942e58 into elastic:main Mar 28, 2025
17 checks passed
omricohenn pushed a commit to omricohenn/elasticsearch that referenced this pull request Mar 28, 2025
@andreidan @omricohenn
Add shard routing and state transitions logging to testFilterCacheSta…
40637c6 
…ts (elastic#125836)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@original-brownbear original-brownbear original-brownbear approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@andreidan @elasticsearchmachine @original-brownbear