-
Notifications
You must be signed in to change notification settings - Fork 287
CI: modify dockerfile (#21935) #21966
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: modify dockerfile (#21935) #21966
Conversation
Sometimes, the network between the runner and the apt software repository is very poor, which will cause packaging failure. We need to package all dependent software into the base image. Approved by: @XuPeng-SH, @fengttt
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨Explore these optional code suggestions:
|
|||||||||
Co-authored-by: qodo-merge-pro[bot] <151058649+qodo-merge-pro[bot]@users.noreply.github.com>
User description
Sometimes, the network between the runner and the apt software repository is very poor, which will cause packaging failure. We need to package all dependent software into the base image.
Approved by: @XuPeng-SH, @fengttt
What type of PR is this?
Which issue(s) this PR fixes:
issue https://github.com/matrixorigin/MO-Cloud/issues/5495
What this PR does / why we need it:
Sometimes, the network between the runner and the apt software repository is very poor, which will cause packaging failure. We need to package all dependent software into the base image.
PR Type
Enhancement, Documentation
Description
Refactored Dockerfile to use pre-built base images with dependencies
matrixorigin/golangandmatrixorigin/ubuntubase imagesAdded README documenting new base image dependency management
Changes walkthrough 📝
Dockerfile
Refactor Dockerfile to use pre-built images and remove apt installsoptools/images/Dockerfile
matrixorigin/golangandmatrixorigin/ubuntuimagesREADME.md
Add README for base image dependency managementoptools/images/README.md