Merge pull request #5413 from ywk253100/180727_chart_bugfix

Fix misc issues of Harbor chart

contrib/helm/harbor/templates/adminserver/adminserver-cm.yaml

@@ -55,4 +55,4 @@ data:
   UAA_CLIENTID: ""
   UAA_CLIENTSECRET: ""
   UAA_VERIFY_CERT: "True"
-  REGISTRY_STORAGE_PROVIDER_NAME: "filesystem"
+  REGISTRY_STORAGE_PROVIDER_NAME: "{{ .Values.registry.storage.type }}"

contrib/helm/harbor/templates/registry/registry-cm.yaml

@@ -130,16 +130,16 @@ data:
         {{- end }}
       {{- end }}
       cache:
-        layerinfo: inmemory
+        layerinfo: redis
       maintenance:
         uploadpurging:
           enabled: false
       delete:
         enabled: true
-    {{- if .Values.registry.redis }}
     redis:
-{{ toYaml .Values.registry.redis | indent 6 }}
+      addr: "{{ template "harbor.redis.host" . }}:{{ template "harbor.redis.port" . }}"
-    {{- end }}
+      password: {{ template "harbor.redis.password" . }}
+      db: {{ template "harbor.redis.databaseIndex" . }}
     http:
       addr: :5000
       # set via environment variable
@@ -152,7 +152,6 @@ data:
         realm: "{{ template "harbor.externalURL" . }}/service/token"
         rootcertbundle: /etc/registry/root.crt
         service: harbor-registry
-
     notifications:
       endpoints:
         - name: harbor

contrib/helm/harbor/templates/ui/ui-dpl.yaml

@@ -28,6 +28,8 @@ spec:
               secretKeyRef:
                 name: "{{ template "harbor.fullname" . }}-ui"
                 key: jobserviceSecret
+          - name: _REDIS_URL
+            value: {{ template "harbor.redisForUI" . }}
           - name: GODEBUG
             value: netdns=cgo
           - name: LOG_LEVEL

contrib/helm/harbor/values.yaml

@@ -291,10 +291,6 @@ registry:
   #  requests:
   #    memory: 256Mi
   #    cpu: 100m
-  ## Enable the redis caching in the registry
-  #redis:
-  #  addr: "redis-host:6379"
-  #  db: 2
   nodeSelector: {}
   tolerations: []
   affinity: {}