Skip to content

Conversation

@arteam
Copy link
Contributor

@arteam arteam commented Feb 18, 2025

The underlying failure java.lang.AssertionError: Searchable snapshot directory does not support the operation [createOutput was fixed in #122006.

The automation bot was too aggressive in re-opening this issue.

Resolve #122693

…alSearchableSnapshot`

The underlying failure `java.lang.AssertionError: Searchable snapshot directory does not support the operation [createOutput`
was fixed in elastic#122006.

The automation bot was too aggressive in re-opening this issue.

Resolve elastic#122693
@arteam arteam added >test Issues or PRs that are addressing/adding tests :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. labels Feb 18, 2025
@elasticsearchmachine elasticsearchmachine added Team:Distributed Indexing Meta label for Distributed Indexing team v9.1.0 labels Feb 18, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@tlrx
Copy link
Member

tlrx commented Feb 18, 2025

I think both failures are legit and require investigation, since they both contain #122006.

Looking at the logs I can see:

Caused by: java.lang.IllegalStateException: Failed to read data from cache: cache service is not started [CLOSED]
	at org.elasticsearch.xpack.searchablesnapshots.cache.full.CacheService.ensureLifecycleStarted(CacheService.java:251) ~[main/:?]
	at org.elasticsearch.xpack.searchablesnapshots.cache.full.CacheService.get(CacheService.java:281) ~[main/:?]

So I suspect the fix in #122006 is not complete and should also consider CLOSED state along with STOPPED.

@arteam
Copy link
Contributor Author

arteam commented Feb 18, 2025

@tlrx Let me add an extra state check 👍

@arteam arteam requested a review from tlrx February 18, 2025 14:15
Copy link
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

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

LGTM, left a minor comment

…arch/xpack/searchablesnapshots/cache/full/CacheService.java

Co-authored-by: Tanguy Leroux <[email protected]>
@arteam arteam added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 18, 2025
@arteam arteam enabled auto-merge (squash) February 18, 2025 16:31
@arteam arteam merged commit b4e95dc into elastic:main Feb 18, 2025
17 checks passed
@arteam arteam deleted the unmute-122693 branch February 18, 2025 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. Team:Distributed Indexing Meta label for Distributed Indexing team >test Issues or PRs that are addressing/adding tests v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] FrozenSearchableSnapshotsIntegTests testCreateAndRestorePartialSearchableSnapshot failing

3 participants