github / github/octocatalog-diff

Turn parallel_tests back on for CI

Open
#164 0 comments 0 reactions 0 assignees View on GitHub
CI help wanted
Dominant language
Ruby
Stars
252
Forks
95
PR merge metrics
No merged PRs in 30d

Description

Recently the CI for octocatalog-diff has been pretty flaky due to random tests getting killed. From travis CI documents, I concluded that the individual tests may be exhausting resources of the container, and as such in https://github.com/github/octocatalog-diff/pull/161 I disabled the parallel_tests gem for CI. Since then, no problems. :crossed_fingers: But, unfortunately the CI build time per ruby version increased by about 5 minutes (20 to 25) as a result of this change.

I'm entering this issue to keep track turning this back on. Things that would probably need to be done here would be to evaluate whether the tests can be effectively parallelized (perhaps the spec tests) or if the number of simultaneous processes can be reduced to keep from bumping up against any limits that exist.

Right now nobody is actively working on this (there are higher priorities). However if test parallelization is something that interests you and you'd like to have a go at making the CI faster, please comment in the issue! 😸

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.