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

CI: refactor cloud release ci #4246

Closed
wants to merge 4 commits into from
Closed

Conversation

lingdie
Copy link
Collaborator

@lingdie lingdie commented Nov 3, 2023

🤖 Generated by Copilot at 3452739

Summary

☁️♻️📦

This pull request adds a new workflow for releasing the cloud components of sealos using the release tag as the image tag. It also refactors the existing workflows for building the individual components by making them reusable and accepting inputs for pushing and tagging the images. The new workflow uses the workflow_call syntax to invoke the component workflows as sub-workflows.

workflow_call used
to reuse cloud workflows
autumn of refactoring

Walkthrough

  • Add a new workflow for releasing the cloud components of sealos (link)
  • Make the existing workflows for building the cloud components reusable by other workflows (link, link, link, link)

@sealos-ci-robot
Copy link
Member

🤖 Generated by lychee action

Summary

Status Count
🔍 Total 981
✅ Successful 378
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 601
❓ Unknown 0
🚫 Errors 1

Errors per input

Errors in README.md

Full action output

Full Github Actions output

Copy link

sweep-ai bot commented Nov 3, 2023

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.

Copy link

codecov bot commented Nov 3, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!.

@lingdie lingdie changed the title Release ci CI: refactor cloud release ci Nov 3, 2023
@lingdie lingdie closed this Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants