-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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] Use Ubuntu 22.04 as ubuntu-latest
at CI (fixes #5186)
#5288
Conversation
ubuntu-latest
at CIubuntu-latest
at CI
/gha run r-configure |
Updated this to latest But only for a few more days! From actions/runner-images#6399
|
The date for "Ubuntu 22.04 is the default |
Based on some recent comments on that upstream tracking issue, it seems like GitHub's plan is "from now until March 2023, slowly increase the number of runners for which For example:
I see evidence of this on recent builds in #5619 too...some This is a REALLY long time to not know which environment a job is going to get 😕 . So for now, to minimize disruption to LightGBM's CI, I've pushed commits here replacing all uses of |
Is the |
Yep! That's a LightGBM-specific thing, and only used on Azure. Maybe we should rethink that. Specifically, maybe we should try to switch the R jobs to running in an |
I think the issue is with clang 14, I'm able to reproduce this locally installing that version from conda: mamba install -y "clang==14.0"
conda deactivate && conda activate $ENV # to refresh the path
export CXX=$(which clang)
export CC=$CXX
mkdir build && cd build # from repo root
cmake .. I get:
I'll keep looking, just wanted to share in case this helps you. |
I think we need |
@jmoralez I don't think that's correct. The most recent runs I pushed just installing
Which means we haven't yet actually tested that installing I appreciate the help, but could you please experiment on a different branch instead of pushing commits here? I'm in the middle of testing here on this one. |
Sure, sorry. Can you please just let those two runs finish to see if they succeed? |
Sure, I won't cancel them. |
Done, I've canceled the rest. Sorry again, thought you were done for the day. |
No prob no prob! We're both here visiting a @StrikerRUS PR anyway 😂 |
Looks like just installing Builds: https://github.com/microsoft/LightGBM/actions/runs/3701050842/jobs/6270018989 I think that's preferable to using something with a version number like I'm going to open this up for review, since I know it's blocking: #5612 (comment) I'm also going to open another draft PR to work on trying to move the R package Linux jobs into containers. Will explain in that other PR why I think that's preferable. |
ubuntu-latest
at CIubuntu-latest
at CI
ubuntu-latest
at CIubuntu-latest
at CI (fixes #5186)
Ah wait sorry, the I'll move this back to draft and keep experimenting. |
ubuntu-latest
at CI (fixes #5186)ubuntu-latest
at CI (fixes #5186)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we could set an env variable in the R package workflow to hold the value of ubuntu latest and just use that everywhere (example), that way the next time we'd just change that. Approving because that's just nice to have and can be done later.
Oh, took too long to review and missed the comment. |
Ha yeah sorry! The most recent commit I pushed got us a bit closer. Now just the I see this error further up in the
Which eventually results in
Will push a fix shortly. Each time I touch one of these |
ubuntu-latest
at CI (fixes #5186)ubuntu-latest
at CI (fixes #5186)
Alright I'm gonna stop for tonight. Hopefully the CUDA, QEMU, and Appveyor jobs will end up succeeding. @jmoralez @shiyu1994 if you get back to this PR before me and see everything passing, merge it. Otherwise, I'll do that tomorrow. Thanks for the help! |
This pull request has been automatically locked since there has not been any recent activity since it was closed. |
Fixes #5186.
Depends on actions/runner-images#5490 when
ubuntu-22.04
will become the default Ubuntu on GitHub Actions.