Browse Source

minio/charts: do not add minio volumes env on distributed mode (#2387)

* minio/charts: do not add minio volumes env on distributed mode

* fix migration error
Stavros Kois 1 year ago
parent
commit
d186c061bf

+ 1 - 1
library/ix-dev/charts/minio/Chart.yaml

@@ -3,7 +3,7 @@ description: High Performance, Kubernetes Native Object Storage
 annotations:
   title: MinIO
 type: application
-version: 2.0.5
+version: 2.0.6
 apiVersion: v2
 appVersion: "2023-03-13"
 kubeVersion: ">=1.16.0-0"

+ 2 - 1
library/ix-dev/charts/minio/migrations/migrate

@@ -36,7 +36,8 @@ def migrate_common_lib(values):
         'minioConfig': {
             'rootUser': values['accessKey'],
             'rootPassword': values['secretKey'],
-            'domain': values.get('minioDomain', ''),
+            # We don't want to allow null value here
+            'domain': values.get('minioDomain', '') or '',
             'extraArgs': values.get('extraArgs', []),
             'additionalEnvs': values.get('environmentVariables', []),
         },

+ 2 - 0
library/ix-dev/charts/minio/templates/_minio.tpl

@@ -17,8 +17,10 @@ workload:
             runAsUser: 473
             runAsGroup: 473
             # readOnlyRootFilesystem: false
+          {{- if not .Values.minioStorage.distributedMode }}
           env:
             MINIO_VOLUMES: /export
+          {{- end }}
           args:
             - server
             - --console-address