diff --git a/charts/clusterpedia/Chart.yaml b/charts/clusterpedia/Chart.yaml index bc101a6..993e700 100644 --- a/charts/clusterpedia/Chart.yaml +++ b/charts/clusterpedia/Chart.yaml @@ -15,7 +15,7 @@ type: application # This is the chart version. This version number should be incremented each time you make changes # to the chart and its templates, including the app version. # Versions are expected to follow Semantic Versioning (https://semver.org/) -version: 2.2.0 +version: 2.2.1 # This is the version number of the application being deployed. This version number should be # incremented each time you make changes to the application. Versions are not expected to diff --git a/charts/clusterpedia/templates/_helpers.tpl b/charts/clusterpedia/templates/_helpers.tpl index 40b0fed..b44b4eb 100644 --- a/charts/clusterpedia/templates/_helpers.tpl +++ b/charts/clusterpedia/templates/_helpers.tpl @@ -171,9 +171,9 @@ Return the proper Docker Image Registry Secret Names {{- define "clusterpedia.storage.password" -}} {{- if eq .Values.storageInstallMode "external" }} {{- if empty (include "clusterpedia.storage.dsn" .) -}} - {{- required "Please set correct storage password!" .Values.externalStorage.password | toString | b64enc -}} + {{- required "Please set correct storage password!" .Values.externalStorage.password | toString }} {{- else -}} - {{- .Values.externalStorage.password | toString | b64enc -}} + {{- .Values.externalStorage.password | toString -}} {{- end -}} {{- else -}} {{- if eq (include "clusterpedia.storage.type" .) "postgres" }} @@ -389,3 +389,28 @@ We truncate at 63 chars because some Kubernetes name fields are limited to this value: 'k8s.namespace.name=$(OTEL_K8S_NAMESPACE),k8s.node.name=$(OTEL_RESOURCE_ATTRIBUTES_NODE_NAME),k8s.pod.name=$(OTEL_RESOURCE_ATTRIBUTES_POD_NAME)' {{- end -}} {{- end -}} + +{{- define "clusterpedia.storage.secret" -}} +type: {{ include "clusterpedia.storage.type" . | quote }} +dsn: {{ include "clusterpedia.storage.dsn" . | quote }} +host: {{ include "clusterpedia.storage.host" . | quote }} +port: {{ include "clusterpedia.storage.port" . }} +user: {{ include "clusterpedia.storage.user" . }} +password: {{ include "clusterpedia.storage.password" . }} +database: {{ include "clusterpedia.storage.database" . }} +{{ if .Values.storageConfig.log.enabled }} +log: + stdout: {{ .Values.storageConfig.log.stdout }} + level: {{ .Values.storageConfig.log.level | default "Warn" | quote }} + slowThreshold: {{ .Values.storageConfig.log.slowThreshold }} + ignoreRecordNotFoundError: {{ .Values.storageConfig.log.ignoreRecordNotFoundError }} + logger: + filename: {{ .Values.storageConfig.log.logger.filename }} + maxsize: {{ .Values.storageConfig.log.logger.maxsize }} + maxbackups: {{ .Values.storageConfig.log.logger.maxbackups }} +{{ end }} +connPool: + maxIdleConns: {{ .Values.storageConfig.connPool.maxIdleConns | int }} + maxOpenConns: {{ .Values.storageConfig.connPool.maxOpenConns | int }} + connMaxLifetime: {{ .Values.storageConfig.connPool.connMaxLifetime }} +{{- end -}} \ No newline at end of file diff --git a/charts/clusterpedia/templates/apiserver-deployment.yaml b/charts/clusterpedia/templates/apiserver-deployment.yaml index 062f881..b2a4afb 100644 --- a/charts/clusterpedia/templates/apiserver-deployment.yaml +++ b/charts/clusterpedia/templates/apiserver-deployment.yaml @@ -56,12 +56,6 @@ spec: done; echo 'DataBase OK ✓' {{- end }} - env: - - name: {{ include "clusterpedia.storage.password.envKey" . }} - valueFrom: - secretKeyRef: - name: {{ include "clusterpedia.internalstorage.fullname" . }} - key: password volumeMounts: - name: internalstorage-config mountPath: /etc/clusterpedia/storage @@ -95,14 +89,6 @@ spec: done; echo 'DataBase OK ✓' {{- end }} - {{- if eq (include "clusterpedia.storage.type" .) "mysql" }} - env: - - name: DB_PASSWORD - valueFrom: - secretKeyRef: - name: {{ include "clusterpedia.internalstorage.fullname" . }} - key: password - {{- end }} volumeMounts: - name: internalstorage-config mountPath: /etc/clusterpedia/storage @@ -130,11 +116,6 @@ spec: resources: {{- toYaml .Values.apiserver.resources | nindent 12 }} {{- end }} env: - - name: DB_PASSWORD - valueFrom: - secretKeyRef: - name: {{ include "clusterpedia.internalstorage.fullname" . }} - key: password {{- include "clusterpedia.apiserver.tracing.env" . | nindent 8 }} {{- if .Values.apiserver.enableSHA1Cert }} - name: GODEBUG @@ -164,8 +145,8 @@ spec: {{- end }} volumes: - name: internalstorage-config - configMap: - name: {{ include "clusterpedia.internalstorage.fullname" . }} + secret: + secretName: {{ include "clusterpedia.internalstorage.fullname" . }} - name: scripts configMap: name: {{ include "clusterpedia.scripts.fullname" . }} diff --git a/charts/clusterpedia/templates/clustersynchro-manager-deployment.yaml b/charts/clusterpedia/templates/clustersynchro-manager-deployment.yaml index 8a3c5a7..fc4f847 100644 --- a/charts/clusterpedia/templates/clustersynchro-manager-deployment.yaml +++ b/charts/clusterpedia/templates/clustersynchro-manager-deployment.yaml @@ -56,12 +56,6 @@ spec: done; echo 'DataBase OK ✓' {{- end }} - env: - - name: {{ include "clusterpedia.storage.password.envKey" . }} - valueFrom: - secretKeyRef: - name: {{ include "clusterpedia.internalstorage.fullname" . }} - key: password volumeMounts: - name: internalstorage-config mountPath: /etc/clusterpedia/storage @@ -95,14 +89,6 @@ spec: done; echo 'DataBase OK ✓' {{- end }} - {{- if eq (include "clusterpedia.storage.type" .) "mysql" }} - env: - - name: DB_PASSWORD - valueFrom: - secretKeyRef: - name: {{ include "clusterpedia.internalstorage.fullname" . }} - key: password - {{- end }} volumeMounts: - name: internalstorage-config mountPath: /etc/clusterpedia/storage @@ -134,11 +120,6 @@ spec: resources: {{- toYaml .Values.clustersynchroManager.resources | nindent 12 }} {{- end }} env: - - name: DB_PASSWORD - valueFrom: - secretKeyRef: - name: {{ include "clusterpedia.internalstorage.fullname" . }} - key: password - name: GODEBUG value: netdns=go volumeMounts: @@ -157,8 +138,8 @@ spec: {{- end }} volumes: - name: internalstorage-config - configMap: - name: {{ include "clusterpedia.internalstorage.fullname" . }} + secret: + secretName: {{ include "clusterpedia.internalstorage.fullname" . }} - name: scripts configMap: name: {{ include "clusterpedia.scripts.fullname" . }} diff --git a/charts/clusterpedia/templates/internalstorage-configmap.yaml b/charts/clusterpedia/templates/internalstorage-configmap.yaml deleted file mode 100644 index 3929be6..0000000 --- a/charts/clusterpedia/templates/internalstorage-configmap.yaml +++ /dev/null @@ -1,29 +0,0 @@ -apiVersion: v1 -kind: ConfigMap -metadata: - name: {{ include "clusterpedia.internalstorage.fullname" . }} - namespace: {{ .Release.Namespace }} - labels: {{- include "common.labels.standard" . | nindent 4 }} -data: - internalstorage-config.yaml: |- - type: {{ include "clusterpedia.storage.type" . | quote }} - dsn: {{ include "clusterpedia.storage.dsn" . | quote }} - host: {{ include "clusterpedia.storage.host" . | quote }} - port: {{ include "clusterpedia.storage.port" . }} - user: {{ include "clusterpedia.storage.user" . }} - database: {{ include "clusterpedia.storage.database" . }} - {{ if .Values.storageConfig.log.enabled }} - log: - stdout: {{ .Values.storageConfig.log.stdout }} - level: {{ .Values.storageConfig.log.level | default "Warn" | quote }} - slowThreshold: {{ .Values.storageConfig.log.slowThreshold }} - ignoreRecordNotFoundError: {{ .Values.storageConfig.log.ignoreRecordNotFoundError }} - logger: - filename: {{ .Values.storageConfig.log.logger.filename }} - maxsize: {{ .Values.storageConfig.log.logger.maxsize }} - maxbackups: {{ .Values.storageConfig.log.logger.maxbackups }} - {{ end }} - connPool: - maxIdleConns: {{ .Values.storageConfig.connPool.maxIdleConns | int }} - maxOpenConns: {{ .Values.storageConfig.connPool.maxOpenConns | int }} - connMaxLifetime: {{ .Values.storageConfig.connPool.connMaxLifetime }} diff --git a/charts/clusterpedia/templates/internalstorage-secret.yaml b/charts/clusterpedia/templates/internalstorage-secret.yaml index 0a89e1c..98609b4 100644 --- a/charts/clusterpedia/templates/internalstorage-secret.yaml +++ b/charts/clusterpedia/templates/internalstorage-secret.yaml @@ -7,4 +7,4 @@ metadata: app: {{ include "clusterpedia.internalstorage.fullname" . }} internalstorage.clusterpedia.io/type: {{ include "clusterpedia.storage.type" . }} data: - password: {{ include "clusterpedia.storage.password" . }} + internalstorage-config.yaml: {{ include "clusterpedia.storage.secret" .| b64enc | quote }}