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 heartbeat for sealos self-host cluster. #4383

Closed
wants to merge 1 commit into from

Conversation

lingdie
Copy link
Collaborator

@lingdie lingdie commented Dec 6, 2023 β€’

πŸ€–[deprecated] Generated by Copilot at 224257c

Summary

πŸš€πŸ› οΈπŸ”’

This pull request adds a new heartbeat controller to the ./controllers workspace, which collects and reports the cluster resources to the license server in the ./service workspace. It also updates the module management and dependency files for both workspaces, and provides the necessary files to build, test, and deploy the heartbeat controller as a Kubernetes job.

heartbeat controller
isolates its dependencies
autumn of modules

Walkthrough

  • Add a sub-workspace for the heartbeat controller in the ./controllers workspace (link)
  • Implement the heartbeat controller as a Kubernetes job that collects and reports cluster resources to the license server (link, link, link, link, link, link)
  • Build a Docker image for the heartbeat controller using the Dockerfile and the Makefile (link, link)
  • Update the checksums of the modules that are dependencies of the ./controllers code in the go.work.sum file (link, link, link, link, link, link, link, link, link)
  • Remove the unused dependencies from the ./controllers workspace in the go.work.sum file (link, link)
  • Remove the unused dependencies from the root module in the go.mod file (link, link, link, link, link)
  • Update the checksums of the modules that are dependencies of the root workspace in the go.work.sum file (link, link, link, link, link)
  • Update the checksums of the modules that are dependencies of the ./service code in the go.work.sum file (link)

@sealos-ci-robot
Copy link
Member

πŸ€– Generated by lychee action

Summary

Status Count
πŸ” Total 1081
βœ… Successful 395
⏳ Timeouts 0
πŸ”€ Redirected 0
πŸ‘» Excluded 685
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

Copy link

sweep-ai bot commented Dec 6, 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.
  • Apply: Add docstrings to all functions and file headers.

Sorry, something went wrong.

Copy link

codecov bot commented Dec 6, 2023

Codecov Report

All modified and coverable lines are covered by tests βœ…

Comparison is base (f52b174) 65.45% compared to head (224257c) 65.45%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4383   +/-   ##
=======================================
  Coverage   65.45%   65.45%           
=======================================
  Files           8        8           
  Lines         660      660           
=======================================
  Hits          432      432           
  Misses        180      180           
  Partials       48       48           

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

@lingdie lingdie closed this Dec 7, 2023
@lingdie lingdie deleted the job/heartbeat branch December 7, 2023 03:19
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

2 participants