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

feat. fix cloud cluster image #3087

Merged
merged 1 commit into from
May 14, 2023
Merged

Conversation

cuisongliu
Copy link
Collaborator

@cuisongliu cuisongliu commented May 14, 2023

🤖 Generated by Copilot at 1e43528

Summary

🛠️🏗️🚀

This pull request improves the deployment speed and stability of sealos-cloud by using local tar files instead of pulling images from the sealos hub registry. It also fixes a bug in the GitHub Actions workflow that prevented using docker for building the image.

sealos build fixes
docker bug with chroot flag
autumn workaround

Walkthrough

  • Add --isolation=chroot flag to sealos build command to avoid docker bug in GitHub Actions (link)
  • Use multi-stage build in Kubefile to copy sealos-cloud components from builder image and reduce final image size and complexity (link)
  • Delete unused shim files for controller, service and frontend (link, link, link)
  • Modify init.sh script to use local tar files instead of remote images for running sealos-cloud components and improve performance and reliability (link, link, link)
  • Remove sealos login command from init.sh script as it is no longer needed (link)

Signed-off-by: cuisongliu <cuisongliu@qq.com>
@codecov
Copy link

codecov bot commented May 14, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (f8224e0) 72.30% compared to head (1e43528) 72.30%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3087   +/-   ##
=======================================
  Coverage   72.30%   72.30%           
=======================================
  Files           8        8           
  Lines         603      603           
=======================================
  Hits          436      436           
  Misses        132      132           
  Partials       35       35           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Collaborator

@lingdie lingdie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀🚀🚀

@cuisongliu cuisongliu merged commit 926314a into labring:main May 14, 2023
35 checks passed
@cuisongliu cuisongliu deleted the build_tar branch May 14, 2023 09:42
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit to cuisongliu/sealos that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
cuisongliu added a commit that referenced this pull request May 14, 2023
Signed-off-by: cuisongliu <cuisongliu@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants