Skip to content

Commit 49ff52c

Browse files
authored
chore: add server.enableRbac to values.yaml of karpor (#89)
<!-- Thank you for contributing to KusionStack! Note: 1. With pull requests: - Open your pull request against "main" - Your pull request should have no more than two commits, if not you should squash them. - It should pass all tests in the available continuous integration systems such as GitHub Actions. - You should add/modify tests to cover your proposed code changes. - If your pull request contains a new feature, please document it on the README. 2. Please create an issue first to describe the problem. We recommend that link the issue with the PR in the following question. For more info, check https://kusionstack.io/docs/governance/contribute/ --> #### 1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".): - [ ] N - [ ] Y <!-- You can add issue references here. e.g. fix #123, re #123, fix https://github.com/XXX/issues/44 --> #### 2. What is the scope of this PR (e.g. component or file name): <!-- You can add the scope of this change here. e.g. /src/server/core.rs, kusionstack/KCLVM/kclvm-parser --> #### 3. Provide a description of the PR(e.g. more details, effects, motivations or doc link): <!-- You can choose a brief description here --> - [ ] Affects user behaviors - [ ] Contains syntax changes - [ ] Contains variable changes - [ ] Contains experimental features - [ ] Performance regression: Consumes more CPU - [ ] Performance regression: Consumes more Memory - [ ] Other <!-- You can add more details here. e.g. Call method "XXXX" to ..... in order to ...., More details: https://XXXX.com/doc...... --> #### 4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link): - [ ] N - [ ] Y <!-- You can add more details here. e.g. Calling method "XXXX" will cause the "XXXX", "XXXX" modules to be affected. More details: https://XXXX.com/doc...... --> #### 5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links: <!-- You can choose a brief description here --> - [ ] Unit test - [ ] Integration test - [ ] Benchmark (add benchmark stats below) - [ ] Manual test (add detailed scripts or steps below) - [ ] Other <!-- You can add more details here. e.g. The test case in XXXX is used to ..... test cases in /src/tests/XXXXX test cases https://github.com/XXX/pull/44 benchmark stats: time XXX ms --> #### 6. Release note <!-- compatibility change, improvement, bugfix, and new feature need a release note --> Please refer to [Release Notes Language Style Guide](https://kusionstack.io/docs/governance/release-policy/) to write a quality release note. ```release-note None ```
1 parent f607db8 commit 49ff52c

File tree

4 files changed

+6
-1
lines changed

4 files changed

+6
-1
lines changed

charts/karpor/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
apiVersion: v2
22
name: karpor
3-
version: 0.6.19
3+
version: 0.6.20
44
type: application
55
appVersion: 0.5.9
66
description: A modern kubernetes visualization tool (Karpor).

charts/karpor/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ The Karpor Server Component is main backend server. It itself is an `apiserver`,
7777
| server.replicas | int | `1` | The number of karpor server pods to run. |
7878
| server.resources | object | `{"limits":{"cpu":"500m","ephemeral-storage":"10Gi","memory":"1Gi"},"requests":{"cpu":"250m","ephemeral-storage":"2Gi","memory":"256Mi"}}` | Resource limits and requests for the karpor server pods. |
7979
| server.serviceType | string | `"ClusterIP"` | Service type for the karpor server. The available type values list as ["ClusterIP"、"NodePort"、"LoadBalancer"]. |
80+
| server.enableRbac | bool | `false` | Enable RBAC authorization if set to true. |
8081

8182
### Karpor Syncer
8283

charts/karpor/templates/karpor-server.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,9 @@ spec:
3535
- --secure-port={{ .Values.server.port }}
3636
- --client-ca-file=/etc/karpor/ca.crt
3737
- --service-account-signing-key-file=/etc/karpor/ca.key
38+
{{- if .Values.server.enableRbac }}
39+
- --enable-rbac=true
40+
{{- end }}
3841
command:
3942
- /karpor
4043
image: {{ include "karpor.realImage" (dict "context" . "repo" .Values.server.image.repo "tag" .Values.server.image.tag "needV" (not (hasPrefix "v" .Values.server.image.tag))) }}

charts/karpor/values.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ server:
3838
cpu: 500m
3939
memory: 1Gi
4040
ephemeral-storage: 10Gi
41+
enableRbac: false
4142
# -- Service type for the karpor server. The available type values list as ["ClusterIP"、"NodePort"、"LoadBalancer"].
4243
serviceType: ClusterIP
4344

0 commit comments

Comments
 (0)