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: add update operation request #3802

Merged
merged 3 commits into from Sep 4, 2023
Merged

Conversation

lingdie
Copy link
Collaborator

@lingdie lingdie commented Aug 30, 2023

πŸ€– Generated by Copilot at 4fe137b

Summary

πŸ”„πŸ—‘οΈπŸŒ

This pull request adds multi-arch support for the cloud images and implements rolebinding update logic for the user controller. It modifies the .github/workflows/cloud.yml file to enable cross-platform builds and the controllers/user/controllers/operationrequest_controller.go file to handle operation requests with the Update action.

We are the sealos, we rule the cloud
We update roles with power and sound
We delete and create, we record our fate
We build for all platforms, we manifest our wrath

Walkthrough

  • Add --platform flag to sealos build command for multi-arch support (link, link)
  • Implement rolebinding update logic for user controller (link)

@sealos-ci-robot
Copy link
Member

sealos-ci-robot commented Aug 30, 2023

πŸ€– Generated by lychee action

Summary

Status Count
πŸ” Total 909
βœ… Successful 372
⏳ Timeouts 0
πŸ”€ Redirected 0
πŸ‘» Excluded 536
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

@codecov
Copy link

codecov bot commented Aug 30, 2023

Codecov Report

Patch has no changes to coverable lines.

πŸ“’ Thoughts on this report? Let us know!.

@bxy4543 bxy4543 added this to the v5.0 milestone Sep 4, 2023
@bxy4543 bxy4543 merged commit cb1fefe into labring:main Sep 4, 2023
46 checks passed
@lingdie lingdie deleted the operation-req branch October 13, 2023 04:54
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.

None yet

3 participants