search

openzipkin / brave

Java distributed tracing implementation compatible with Zipkin backend services.
Apache License 2.0
2.36k stars 713 forks source link

ci: PR tests take too long (17m) #1416

Open codefromthecrypt opened 9 months ago

codefromthecrypt commented 9 months ago

Currently, PR tests take too long to verify.. 17 minutes. Most of the very long tests use docker. Besides that is dubbo which can be longer than 2 minutes to complete despite not using docker. We should bump the heavy tests into separate jobs, at least the docker ones.

Feature

Split the docker tests into separate jobs, like zipkin does.

Rationale

We should be able to get test execution speed down as they aren't all sequential. less time to re-run a flakey test.

Prior Art

codefromthecrypt commented 9 months ago

@reta wanna give this a try?

reta commented 9 months ago

@reta wanna give this a try?

Sure, thanks @codefromthecrypt !