Skip to content
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

[BUG] org.opensearch.action.admin.cluster.tasks.PendingTasksBlocksIT > testPendingTasksWithClusterNotRecoveredBlock FAILED #3273

Open
dblock opened this issue May 10, 2022 · 1 comment
Assignees
Labels
bug Something isn't working flaky-test Random test failure that succeeds on second run

Comments

@dblock
Copy link
Member

dblock commented May 10, 2022

org.opensearch.action.admin.cluster.tasks.PendingTasksBlocksIT > testPendingTasksWithClusterNotRecoveredBlock FAILED
    java.lang.AssertionError: timed out waiting for green state
        at __randomizedtesting.SeedInfo.seed([F19D942E1E4110F4:5F2076E19A98425]:0)
        at org.junit.Assert.fail(Assert.java:89)
        at org.opensearch.test.OpenSearchIntegTestCase.ensureColor(OpenSearchIntegTestCase.java:979)
        at org.opensearch.test.OpenSearchIntegTestCase.ensureGreen(OpenSearchIntegTestCase.java:918)
        at org.opensearch.test.OpenSearchIntegTestCase.ensureGreen(OpenSearchIntegTestCase.java:907)
        at org.opensearch.action.admin.cluster.tasks.PendingTasksBlocksIT.testPendingTasksWithClusterNotRecoveredBlock(PendingTasksBlocksIT.java:111)

#3259 (comment)

@dbwiddis
Copy link
Member

dbwiddis commented Jan 4, 2023

This is one of a few different tests where I've seen an unreadable (truncated with large length) deprecation log message in the failure trace:

[2022-12-29T09:26:34,355][INFO ][o.o.a.a.c.t.PendingTasksBlocksIT] [testPendingTasksWithClusterNotRecoveredBlock] [PendingTasksBlocksIT#testPendingTasksWithClusterNotRecoveredBlock]: setting up test
[2022-12-29T09:26:34,536][DEPRECATION][o.o.d.c.s.Settings       ] [node_s0] [index.force_memory_term_dictionary] setting was dep
...[truncated 1376351 chars]...
eS8W-VsYrMCZy5A}{Ur513a1IRy24HkBPUMqLPw}{127.0.0.1}{127.0.0.1:42445}{imr}{shard_indexing_pressure_enabled=true}}, term: 4, version: 14, reason: ApplyCommitRequest{term=4, version=14, sourceNode={node_s0}

The setting comes from Store.java:

/**
* This is an escape hatch for lucenes internal optimization that checks if the IndexInput is an instance of ByteBufferIndexInput
* and if that's the case doesn't load the term dictionary into ram but loads it off disk iff the fields is not an ID like field.
* Since this optimization has been added very late in the release processes we add this setting to allow users to opt-out of
* this by exploiting lucene internals and wrapping the IndexInput in a simple delegate.
*/
public static final Setting<Boolean> FORCE_RAM_TERM_DICT = Setting.boolSetting(
"index.force_memory_term_dictionary",
false,
Property.IndexScope,
Property.Deprecated
);

Flakiness is introduced as part of setRandomIndexSettings() in OpenSearchIntegTestCase:

if (random.nextBoolean()) {
builder.put(Store.FORCE_RAM_TERM_DICT.getKey(), true);
}

But similar messages are on completely different log messages on completely unrelated tests:

[2022-11-23T10:30:18,307][INFO ][o.
...[truncated 1171668 chars]...
ansport.TransportService.sendRequest(TransportService.java:718) [main/:?]
	at org.opensearch.cluster.coordination.FollowersChecker$FollowerChecker.handleWakeUp(FollowersChecker.java:348) [main/:?]
	at org.opensearch.cluster.coordination.FollowersChecker$FollowerChecker.start(FollowersChecker.java:336) [main/:?]
	at org.opensearch.cluster.coordination.FollowersChecker.lambda$setCurrentNodes$2(FollowersChecker.java:178) [main/:?]
	at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197) [?:?]

<and more...>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working flaky-test Random test failure that succeeds on second run
Projects
None yet
Development

No branches or pull requests

5 participants