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. #4389

Merged
merged 11 commits into from
Dec 11, 2023

Conversation

lingdie
Copy link
Collaborator

@lingdie lingdie commented Dec 7, 2023

πŸ€–[deprecated] Generated by Copilot at d3868ba

Summary

βœ¨πŸ”§πŸ“¦

This pull request adds a new heartbeat job to the sealos project. The heartbeat job is a Kubernetes CronJob that periodically reports the cluster resources to the license server. The pull request also updates the go.work and go.work.sum files in the controllers and service directories to manage the dependencies and sub-workspaces. The pull request also removes some unused dependencies from the go.mod file.

heartbeat job added
sends cluster resources post
go.work isolated

Walkthrough

  • Add a sub-workspace for the heartbeat job (link)
  • Define a data model for the cluster resources (link)
  • Implement the logic for the heartbeat job to get the cluster resources and send them to the license server (link, link, link)
  • Build a Docker image for the heartbeat job (link, link)
  • Deploy the heartbeat job as a Kubernetes CronJob (link)
  • Manage the dependencies and the module information for the heartbeat job (link)
  • Update the checksums of the dependencies for the controllers sub-workspace (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)
  • Remove the unused dependencies from the root go.mod file (link, link, link, link, link)
  • Update the checksums of the dependencies for the root workspace (link, link, link, link, link)
  • Remove the unused checksums from the service/go.work.sum file (link)

@sealos-ci-robot
Copy link
Member

sealos-ci-robot commented Dec 7, 2023

πŸ€– Generated by lychee action

Summary

Status Count
πŸ” Total 1148
βœ… Successful 388
⏳ Timeouts 0
πŸ”€ Redirected 0
πŸ‘» Excluded 759
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

Copy link

sweep-ai bot commented Dec 7, 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.

Copy link

codecov bot commented Dec 7, 2023

Codecov Report

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

Comparison is base (56123b8) 65.45% compared to head (016837d) 65.45%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4389   +/-   ##
=======================================
  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.

@zzjin zzjin added this to the v5.0 milestone Dec 8, 2023
Copy link

Whoa! Easy there, Partner!

This PR is too big. Please break it up into smaller PRs.

@lingdie lingdie merged commit 80c948a into labring:main Dec 11, 2023
65 checks passed
@lingdie lingdie deleted the job/heartbeat branch December 11, 2023 06:41
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.

3 participants