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

Find a way to avert the CI failure No output has been received in the last 10 minutes #810

Closed
DavidGregory084 opened this issue Jan 16, 2016 · 12 comments

Comments

@DavidGregory084
Copy link
Member

CI builds occasionally fail due to the message:

No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.

This is off-putting for newcomers as it's not immediately obvious that there might be nothing wrong with their PR.

Could we avert this by using the travis_wait command or is there some other way that we could speed up builds so they don't encounter this timeout?

@ghost
Copy link

ghost commented Jan 16, 2016

This is caused by a know bug that has been fixed in scalacheck, but not released. Might be best to wait on this for now.

@DavidGregory084
Copy link
Member Author

Ah, that makes sense. Shall we just close this, or wait for typelevel/scalacheck#195?

@paulp
Copy link
Contributor

paulp commented Mar 18, 2016

Looking at typelevel/scalacheck#195 I see this has been going on for six months. That's about 5.95 months too long when it comes to nondeterministic CI failures. They are huge time wasters.

@DavidGregory084
Copy link
Member Author

@paulp actually 1.13.0 is released now and it looks like cats is building against 1.12.5, maybe just needs a version bump?

@ceedubs
Copy link
Contributor

ceedubs commented Mar 18, 2016

@DavidGregory084 I suspect we are going to run into typelevel/scalacheck#218 if we do that.

@DavidGregory084
Copy link
Member Author

😢

@paulp
Copy link
Contributor

paulp commented Mar 18, 2016

It doesn't even compile against 1.13.0. A little excerpt.

[error] /s/scala/cats/laws/src/main/scala/cats/laws/discipline/TraverseTests.scala:37: could not find implicit value for parameter a2: org.scalacheck.Arbitrary[A => M]
[error]         "traverse derive foldMap" -> forAll(laws.foldMapDerived[A, M] _)
[error]                                            ^
[error] /s/scala/cats/laws/src/main/scala/cats/laws/discipline/TraverseTests.scala:36: could not find implicit value for parameter a2: org.scalacheck.Arbitrary[A => X[B]]
[error]         "traverse parallel composition" -> forAll(laws.traverseParallelComposition[A, B, X, Y] _),
[error]                                                  ^
[error] 50 errors found

@ceedubs
Copy link
Contributor

ceedubs commented Mar 19, 2016

@paulp I haven't really looked yet, so I could be totally wrong, but my guess is that those compile errors are in places where we've assumed that if we have an Arbitrary[A] we have an Arbitrary[X => A]. That was true with previous versions of Scalacheck, but only because it only produced arbitrary functions with a constant result value. I'm guessing we'll have to pass along those arbitrary function requirements (because we don't already have enough implicit parameters in there...).

@xuwei-k
Copy link
Contributor

xuwei-k commented Mar 19, 2016

@ceedubs
Copy link
Contributor

ceedubs commented Mar 19, 2016

Thanks, @xuwei-k.

@adelbertc
Copy link
Contributor

An attempt: #942

@ceedubs
Copy link
Contributor

ceedubs commented May 14, 2016

I haven't seen this problem after some scala.js and scalacheck config changes that were introduced in recent months. I'm going to go ahead and close this out. Feel free to reopen it if it becomes a problem again.

@ceedubs ceedubs closed this as completed May 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants