Cleanup docker build #31119
Cleanup docker build #31119
Conversation
| sudo rm -rf /usr/local/lib/android | ||
| sudo rm -rf /usr/share/dotnet | ||
| sudo du -sh /usr/local/lib/ | ||
| sudo du -sh /usr/share/ |
There was a problem hiding this comment.
we don't this any more as [intel-cpu, 8-cpu, ci] should be large enough.
There was a problem hiding this comment.
Does it hurt to keep?
There was a problem hiding this comment.
not really. But I prefer not to have it as it would implicitly suggest we need it even if we are on [intel-cpu, 8-cpu, ci]. And if we don't have it in all places, it looks inconsistent and will be confusing IMO.
(and if we have it in all places, it's a bit too much ...)
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
| sudo rm -rf /usr/local/lib/android | ||
| sudo rm -rf /usr/share/dotnet | ||
| sudo du -sh /usr/local/lib/ | ||
| sudo du -sh /usr/share/ |
There was a problem hiding this comment.
Does it hurt to keep?
IIRC, that was for CircleCI images. For that part, my comment
And in this PR, it's only about the images for GitHub Actions. |
What does this PR do?
I am staring to go back to other CI workflow (nightly, past, amd) and the first thing is to use
runs-on: [intel-cpu, 8-cpu, ci]instead ofruns-on: ubuntu-22.04so we won't have disk space issue.For
.github/workflows/build-ci-docker-images.yml, we still keepruns-on: ubuntu-22.04as we want to make sure the images there are really small (for CircleCI workflow/jobs)