search

risingwavelabs / risingwave-operator

RisingWave Kubernetes Operator
https://www.risingwave.com/cloud
Apache License 2.0
77 stars 19 forks source link

feat: support license #709

Closed fuyufjh closed 2 months ago

fuyufjh commented 2 months ago

What's changed and what's your intention?

To use a license, users need to

  1. store the license (JWT Token) in K8s Secret with key license
  2. set the RisingWave CRD .license.secretName to the secret name

Checklist

Refer to a related PR or issue link (optional)

fuyufjh commented 2 months ago

@arkbriar make generate-all it will make such a diff. Is it expected or did I did anything wrong?

image
arkbriar commented 2 months ago

@arkbriar make generate-all it will make such a diff. Is it expected or did I did anything wrong?

image

Could you try remove the local bin dir and try again?

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 58.82353% with 14 lines in your changes missing coverage. Please review.

Project coverage is 55.03%. Comparing base (c6da95a) to head (25ffc48). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
apis/risingwave/v1alpha1/zz_generated.deepcopy.go 0.00% 13 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #709 +/- ## ========================================== + Coverage 55.01% 55.03% +0.01% ========================================== Files 40 40 Lines 6645 6679 +34 ========================================== + Hits 3656 3676 +20 - Misses 2864 2877 +13 - Partials 125 126 +1 ``` | [Flag](https://app.codecov.io/gh/risingwavelabs/risingwave-operator/pull/709/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=risingwavelabs) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/risingwavelabs/risingwave-operator/pull/709/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=risingwavelabs) | `55.03% <58.82%> (+0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=risingwavelabs#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.