apache / pekko-management

Apache Pekko Management is a suite of tools for operating Pekko Clusters.
https://pekko.apache.org/
Apache License 2.0
22 stars 14 forks source link

double timeout on flaky test #314

Closed pjfanning closed 2 months ago

pjfanning commented 2 months ago

see #216

pjfanning commented 2 months ago

The same flaky test failed even with the timefactor=2 - but the timefactor didn't seem to increase the timeout - it still looks like the same same short 1.5 second timeout.

[info] - should be able to be created *** FAILED *** (1 second, 459 milliseconds)
[info]   The future returned an exception of type: org.apache.pekko.coordination.lease.LeaseTimeoutException, with message: Timed out removing lease [lease-1]. It is not known if the remove happened. Is the API server up?. (KubernetesApiSpec.scala:104)
[info]   org.scalatest.exceptions.TestFailedException:
raboof commented 2 months ago

hmm, maybe because this test explicitly sets it to testKitSettings.DefaultTimeout.duration? What happens when we remove that? sorry for leading you down this rabbit hole :/

pjfanning commented 2 months ago

Without the explicit patience config, things get far worse. All the individual scenarios timeout in 150ms (each).

pjfanning commented 2 months ago

I'll merge the original change that just doubles the time on the patience config in this test class.