-
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] fail R macOS CI jobs earlier when installations fail #5129
Conversation
Going to move this back to "draft" while investigating the latest error
But hey, at least in https://github.com/microsoft/LightGBM/runs/5826439843?check_suite_focus=true, |
😱 it looks like the curl -I https://cran.r-project.org/bin/macosx/R-4.1.2.pkg
But because there was no LightGBM/.ci/test_r_package.sh Line 79 in d811f3a
And those jobs just silently fell back to the R version already available from GitHub Actions (https://github.com/actions/virtual-environments/blob/main/images/macos/macos-11-Readme.md). For example, look at this R installation failed:
But the job continued, and you can see that the job used R 4.1.3, the version installed by default by GitHub Actions for this runner (link).
Pushed 43d60f6, targeting https://cran.r-project.org/bin/macosx/base/ for R 4.x builds so this doesn't happen again. |
yikes! Thanks for fixing this! Can this error be related to #4990? |
ah!!! I bet you're right! I can try that later (updating those jobs to |
This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this. |
#5128 describes an issue where:
brew install
of a required dependency failsR CMD check
for the R package) fails, because it's missing that dependency from HomebrewThis PR proposes adding some
exit -1
calls to the main CI script for for R-package jobs, so that those jobs will fail earlier when such issues occur in the future.Benefits of this: