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: PR tests take too long (17m) #1416

Open
codefromthecrypt opened this issue Feb 25, 2024 · 2 comments
Open

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

codefromthecrypt opened this issue Feb 25, 2024 · 2 comments

Comments

@codefromthecrypt
Copy link
Member

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
Copy link
Member Author

@reta wanna give this a try?

@reta
Copy link
Contributor

reta commented Feb 25, 2024

@reta wanna give this a try?

Sure, thanks @codefromthecrypt !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants