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

[CI] TransformSchedulerTests testScheduleNowWithSystemClock failing #95445

Closed
HiDAl opened this issue Apr 21, 2023 · 9 comments · Fixed by #95456 or #103191
Closed

[CI] TransformSchedulerTests testScheduleNowWithSystemClock failing #95445

HiDAl opened this issue Apr 21, 2023 · 9 comments · Fixed by #95456 or #103191
Assignees
Labels
medium-risk An open issue or test failure that is a medium risk to future releases :ml/Transform Transform Team:ML Meta label for the ML team >test-failure Triaged test failures from CI

Comments

@HiDAl
Copy link
Contributor

HiDAl commented Apr 21, 2023

This test has been failing consistently twice per month almost the same days. Probably something related to the date
Screenshot 2023-04-21 at 12 22 41

Build scan:
https://gradle-enterprise.elastic.co/s/m44bu6kvdglmg/tests/:x-pack:plugin:transform:test/org.elasticsearch.xpack.transform.transforms.scheduling.TransformSchedulerTests/testScheduleNowWithSystemClock

Reproduction line:

./gradlew ':x-pack:plugin:transform:test' --tests "org.elasticsearch.xpack.transform.transforms.scheduling.TransformSchedulerTests.testScheduleNowWithSystemClock" -Dtests.seed=51789FCA46998A02 -Dtests.locale=uk-UA -Dtests.timezone=Etc/GMT+6 -Druntime.java=20

Applicable branches:
main

Reproduces locally?:
No

Failure history:
https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.xpack.transform.transforms.scheduling.TransformSchedulerTests&tests.test=testScheduleNowWithSystemClock

Failure excerpt:

java.lang.AssertionError: 
Expected: a collection with size <2>
     but: 

  at __randomizedtesting.SeedInfo.seed([51789FCA46998A02:4BE1BDC054FCCE5]:0)
  at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
  at org.junit.Assert.assertThat(Assert.java:956)
  at org.junit.Assert.assertThat(Assert.java:923)
  at org.elasticsearch.xpack.transform.transforms.scheduling.TransformSchedulerTests.testScheduleNowWithSystemClock(TransformSchedulerTests.java:347)
  at jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)
  at java.lang.reflect.Method.invoke(Method.java:578)
  at com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1758)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:946)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$9.evaluate(RandomizedRunner.java:982)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$10.evaluate(RandomizedRunner.java:996)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at org.apache.lucene.tests.util.TestRuleSetupTeardownChained$1.evaluate(TestRuleSetupTeardownChained.java:48)
  at org.apache.lucene.tests.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:43)
  at org.apache.lucene.tests.util.TestRuleThreadAndTestName$1.evaluate(TestRuleThreadAndTestName.java:45)
  at org.apache.lucene.tests.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:60)
  at org.apache.lucene.tests.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:44)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:390)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl.forkTimeoutingTask(ThreadLeakControl.java:843)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl$3.evaluate(ThreadLeakControl.java:490)
  at com.carrotsearch.randomizedtesting.RandomizedRunner.runSingleTest(RandomizedRunner.java:955)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$5.evaluate(RandomizedRunner.java:840)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$6.evaluate(RandomizedRunner.java:891)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$7.evaluate(RandomizedRunner.java:902)
  at org.apache.lucene.tests.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:43)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at org.apache.lucene.tests.util.TestRuleStoreClassName$1.evaluate(TestRuleStoreClassName.java:38)
  at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
  at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at org.apache.lucene.tests.util.TestRuleAssertionsRequired$1.evaluate(TestRuleAssertionsRequired.java:53)
  at org.apache.lucene.tests.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:43)
  at org.apache.lucene.tests.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:44)
  at org.apache.lucene.tests.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:60)
  at org.apache.lucene.tests.util.TestRuleIgnoreTestSuites$1.evaluate(TestRuleIgnoreTestSuites.java:47)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:390)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl.lambda$forkTimeoutingTask$0(ThreadLeakControl.java:850)
  at java.lang.Thread.run(Thread.java:1623)

@HiDAl HiDAl added Team:ML Meta label for the ML team :ml/Transform Transform >test-failure Triaged test failures from CI labels Apr 21, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@przemekwitek
Copy link
Contributor

I identified this bug as a non-monotonicity in the system clock.
Fix is already in review.

@danielmitterdorfer
Copy link
Member

@przemekwitek this happened again on a PR of mine (#96499, which changed something completely unrelated):


17:05:28 org.elasticsearch.xpack.transform.transforms.scheduling.TransformSchedulerTests > testScheduleNowWithSystemClock FAILED
17:05:28     java.lang.AssertionError: 
17:05:28     Expected: a collection with size <2>
17:05:28          but: 
17:05:28         at __randomizedtesting.SeedInfo.seed([395D394976AF3495:6C9BBD5F35797272]:0)
17:05:28         at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
17:05:28         at org.junit.Assert.assertThat(Assert.java:956)
17:05:28         at org.junit.Assert.assertThat(Assert.java:923)
17:05:28         at org.elasticsearch.xpack.transform.transforms.scheduling.TransformSchedulerTests.testScheduleNowWithSystemClock(TransformSchedulerTests.java:345)

Reproduction line:

./gradlew ':x-pack:plugin:transform:test' --tests "org.elasticsearch.xpack.transform.transforms.scheduling.TransformSchedulerTests.testScheduleNowWithSystemClock" -Dtests.seed=395D394976AF3495 -Dtests.locale=ar-DZ -Dtests.timezone=GB-Eire -Druntime.java=20

Did not reproduce locally (I've tried 100 iterations).

Build scan: https://gradle-enterprise.elastic.co/s/hoyxmrw6dozbc

@przemekwitek
Copy link
Contributor

Thanks for reporting.

@albertzaharovits
Copy link
Contributor

New failure today on main: https://gradle-enterprise.elastic.co/s/qggytn4zkqykg .

@kingherc
Copy link
Contributor

New failure on main https://gradle-enterprise.elastic.co/s/xdxhgg64vz674

@kingherc
Copy link
Contributor

Seems to fail multiple times on 8.8, 8.9, main. Will mute.

kingherc added a commit to kingherc/elasticsearch that referenced this issue Jul 19, 2023
kingherc added a commit to kingherc/elasticsearch that referenced this issue Jul 19, 2023
kingherc added a commit to kingherc/elasticsearch that referenced this issue Jul 19, 2023
felixbarny pushed a commit to felixbarny/elasticsearch that referenced this issue Aug 3, 2023
@droberts195
Copy link
Contributor

Assigning medium-risk due to loss of test coverage from muting.

@droberts195 droberts195 added the medium-risk An open issue or test failure that is a medium risk to future releases label Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
medium-risk An open issue or test failure that is a medium risk to future releases :ml/Transform Transform Team:ML Meta label for the ML team >test-failure Triaged test failures from CI
Projects
None yet
7 participants