ci: Increase timeout for coverage job #948
Conversation
|
I'm fine with that, but I think we also realized we were running tarpaulin with some flags that made it slower than needed. @fabriziosestito do you remember which project was fixed? I see policy-server has some different options: https://github.com/kubewarden/policy-server/blob/9737ba8cfa17f578f6e6451f938611c98be8cbd8/Makefile#L46-L49 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #948 +/- ##
=======================================
Coverage 13.16% 13.16%
=======================================
Files 25 25
Lines 2036 2036
=======================================
Hits 268 268
Misses 1768 1768
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This is in hopes of reducing tarpaulin time below 15 mins, so it fits in GHA timeout. Signed-off-by: Víctor Cuadrado Juan <[email protected]>
viccuad
force-pushed
the
coverage-timeout
branch
from
f78c053 to
83058be
Compare
|
Good point. I have force-pushed, removing the timeout increase and setting the same args as we use for policy-server instead. |
|
It still got killed, maybe we have to raise the limit too, but to something higher than 30 minutes |
|
Note that the CI run takes the code from main, not the PR though. |
Description
This is in hopes of reducing tarpaulin time below 15 mins,
so it fits in GHA timeout.
Right now it times out after 15 mins:
https://github.com/kubewarden/kwctl/actions/runs/11442780601
Relates to #852.
Test
Tested locally, but cannot know about GHA timeout.
Additional Information
Tradeoff
Potential improvement