Skip to content

Commit 950a148

Browse files
authored
Merge pull request #9 from MbolotSuse/chart-upd-2
Post code review chart updates
2 parents 7ef3c3c + ac1eef9 commit 950a148

File tree

4 files changed

+10
-15
lines changed

4 files changed

+10
-15
lines changed

charts/rancher-csp-adapter/Chart.yaml

+4-2
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ annotations:
88
catalog.cattle.io/hidden: "true"
99
catalog.cattle.io/namespace: cattle-csp-adapter-system
1010
catalog.cattle.io/release-name: rancher-csp-adapter
11+
catalog.cattle.io/display-name: "Rancher CSP Adapter"
12+
catalog.cattle.io/scope: management
1113
catalog.cattle.io/os: linux
1214
catalog.cattle.io/permits-os: linux
13-
catalog.cattle.io/rancher-version: ">= 2.6.6"
14-
catalog.cattle.io/kube-version: ">= 1.22.0"
15+
catalog.cattle.io/rancher-version: '>= 2.6.6-0 <= 2.6.99-0'
16+
catalog.cattle.io/kube-version: '>= 1.22.0'

charts/rancher-csp-adapter/templates/configmap.yaml

-6
This file was deleted.

charts/rancher-csp-adapter/templates/serviceAccount.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ kind: ServiceAccount
1111
metadata:
1212
name: {{ .Chart.Name }}
1313
namespace: cattle-csp-adapter-system
14-
{{- if eq (include "csp-adapter.csp" . ) "aws" -}}
14+
{{- if eq (include "csp-adapter.csp" . ) "aws" }}
1515
annotations:
1616
eks.amazonaws.com/role-arn: arn:aws:iam::{{ .Values.aws.accountNumber }}:role/{{ .Values.aws.roleName }}
17-
{{- end }}
17+
{{- end }}

charts/rancher-csp-adapter/values.yaml

+4-5
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,11 @@ image:
88
global:
99
cattle:
1010
systemDefaultRegistry: ""
11-
hostNetwork: false
1211

1312
tolerations: []
1413

1514
# at least one csp must be enabled like below
16-
#aws:
17-
# accountNumber: ""
18-
# roleName: ""
19-
# enabled: true
15+
aws:
16+
enabled: false
17+
accountNumber: ""
18+
roleName: ""

0 commit comments

Comments
 (0)