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

Set a default time-out of 60 min for GitHub CI #272

Merged
merged 3 commits into from
Dec 19, 2023
Merged

Conversation

thchr
Copy link
Contributor

@thchr thchr commented Feb 8, 2021

The default time-out for Github Actions seems to be 6 hours which is quite a lot (e.g., I ran into this yesterday with two jobs that stalled overnight, eating up 12 hours cumulatively of my "private" time; ouch.)

I doubt a lot projects need CI times longer than, say, 1 hour; and if they do, they're probably aware and happy to edit the default settings. On the other hand, it would be nice to "cap" the damage a stalled job could do to something less than 6 hours. Here, I tentatively just set it to 60 minutes.

In principle, this could be an adjustable option - but I think this is a reasonable default that most people wouldn't need to change until quite far into development.

@codecov
Copy link

codecov bot commented Feb 9, 2021

Codecov Report

Merging #272 (c015474) into master (238bfc1) will decrease coverage by 1.61%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #272      +/-   ##
==========================================
- Coverage   92.59%   90.97%   -1.62%     
==========================================
  Files          19       19              
  Lines         540      532       -8     
==========================================
- Hits          500      484      -16     
- Misses         40       48       +8     
Impacted Files Coverage Δ
src/interactive.jl 73.52% <0.00%> (-11.77%) ⬇️
src/plugins/git.jl 88.88% <0.00%> (-5.56%) ⬇️
src/show.jl 89.47% <0.00%> (-5.27%) ⬇️
src/plugins/documenter.jl 75.92% <0.00%> (+3.34%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 238bfc1...8fa4e3e. Read the comment docs.

@thchr
Copy link
Contributor Author

thchr commented Feb 9, 2021

... it's kind of awful that I accidentally started four separate Github Actions on this particular pull request; apologies (was using the GitHub web interface to commit without thinking of the fact that each small commit would set off another CI run).

@DilumAluthge
Copy link
Member

Bump. Would be great to get this merged.

@christopher-dG

Copy link

codecov bot commented Dec 19, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8b012e1) 92.94% compared to head (c837d91) 92.94%.
Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #272   +/-   ##
=======================================
  Coverage   92.94%   92.94%           
=======================================
  Files          24       24           
  Lines         737      737           
=======================================
  Hits          685      685           
  Misses         52       52           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gdalle gdalle merged commit 0a1600a into JuliaCI:master Dec 19, 2023
13 checks passed
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

Successfully merging this pull request may close these issues.

3 participants