feat: adapt license for poc #3594
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🤖 Generated by lychee actionSummary
Full action output |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #3594 +/- ##
=======================================
Coverage 67.92% 67.92%
=======================================
Files 8 8
Lines 664 664
=======================================
Hits 451 451
Misses 171 171
Partials 42 42 ☔ View full report in Codecov by Sentry. |
yxxchange
force-pushed
the
license-poc
branch
5 times, most recently
from
8875f01
to
e8ae68a
Compare
lingdie
reviewed
Jul 30, 2023
.github/workflows/ci.yml
Outdated
| @@ -24,6 +24,7 @@ on: | |||
| - "service/**" | |||
| - "webhooks/**" | |||
| - "frontend/**" | |||
| - "deploy/cloud/manifests/free-license.yaml" | |||
| @@ -609,7 +609,7 @@ spec: | |||
| secretKeyRef: | |||
| key: isMonitor | |||
| name: licenseissuer-env | |||
| image: ghcr.io/labring/sealos-licenseissuer-controller:latest | |||
| image: registry.cn-hangzhou.aliyuncs.com/fckc/cloud-controller:v3.3.5 | |||
There was a problem hiding this comment.
do not use personal registry.
deploy/cloud/scripts/init.sh
Outdated
|
|
||
| # wait for admin-user ready | ||
| echo "waiting for admin-user generated" | ||
| sleep 3 |
bxy4543
reviewed
Jul 31, 2023
| @@ -0,0 +1,7 @@ | |||
| apiVersion: infostream.sealos.io/v1 | |||
There was a problem hiding this comment.
add // nosemgrep: generic.secrets.security.detected-jwt-token.detected-jwt-token to ingore semgrep ci .
lingdie
approved these changes
Jul 31, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🤖 Generated by Copilot at dfdccdc
Summary
✨🔧🔨
Added support for free license feature in sealos. This involved removing license expiration check, moving license recharge logic, creating admin user and free license resources, and applying them to the cluster.
Walkthrough
Reconcilefunction ofLicenseReconcilerstruct (link,link)Reconcilefunction ofLicenseReconcilerstruct (link)controllerpackage for readability and formatting (link)Userresource namedadminwith custom CSR expiration time toadmin-user.yamlfile indeploy/cloud/manifestsdirectory (link)Licenseresource namedlicensewith token containing free license for admin user tofree-license.yamlfile indeploy/cloud/manifestsdirectory (link)admin-user.yamlandfree-license.yamlmanifests to Kubernetes cluster ininit.shscript indeploy/cloud/scriptsdirectory (link)