Volodymyr Krasnikov

Results 7 comments of Volodymyr Krasnikov

Unfortunately it's failing again :( https://gradle-enterprise.elastic.co/s/kandwegr56fsy/console-log ``` REPRODUCE WITH: gradlew ':server:internalClusterTest' --tests "org.elasticsearch.indices.flush.FlushIT.testFlushOnInactive" -Dtests.seed=4441F78B2F9D464C -Dtests.locale=en-ZA -Dtests.timezone=Europe/Sofia -Druntime.java=21 org.elasticsearch.indices.flush.FlushIT > testFlushOnInactive FAILED java.lang.AssertionError: Expected: but: was at __randomizedtesting.SeedInfo.seed([4441F78B2F9D464C:A32469C2CC324580]:0) at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18) at...

One more failure instance https://gradle-enterprise.elastic.co/s/r26qd5stdyq3u

Few observation from time spent debugging this test - Repository is [blocked](https://github.com/elastic/elasticsearch/blob/4b5f0a0bb7eac6f807876a96bd5a350c505907b8/x-pack/plugin/searchable-snapshots/src/internalClusterTest/java/org/elasticsearch/xpack/searchablesnapshots/SearchableSnapshotsCanMatchOnCoordinatorIntegTests.java#L600) *before* searchable snapshot is mounted. [Reading of blocking flag](https://github.com/elastic/elasticsearch/blob/b9c29807ecc831838c398aace29157d24e51fdea/test/framework/src/main/java/org/elasticsearch/snapshots/mockstore/MockRepository.java#L501) happens *after*. Thread synchronization is correct as for me....

Memo: Calculation formula needs to be revisited and PR updated

The issue is sitting in silos until 9.0.x work is started. Development is done and is awaiting right moment for review and merge.

According to [CI stats](https://ci-stats.elastic.co/s/jenkins/goto/5dd4a9e0-c6e6-11ee-bb52-c5ce6e2712c3) this test hasn't failed for 5+ months. Closing for now. Please re-open if new failures get encountered