fixed some typos and mistakes in parsing for the config file

Author: hendrikkiedrowski

deploy/charts/bloom/templates/config.yaml

@@ -5,27 +5,27 @@ metadata:
   labels:
     {{- include "bloom.labels" . | nindent 4 }}
 data:
-  config.cfg: |
+  bloom.cfg: |
     [server]
 
-    log_level = {{ .Values.bloom.server.log_level | default "error" }}
-    inet = {{ .Values.bloom.server.inet | default "[::1]:8080" }}
+    log_level = {{ .Values.bloom.server.log_level | default "error" | quote }}
+    inet = {{ .Values.bloom.server.inet | default "[::1]:8080" | quote }}
 
 
     [control]
 
-    inet = {{ .Values.bloom.control.inet | default "[::1]:8811" }}
+    inet = {{ .Values.bloom.control.inet | default "[::1]:8811" | quote }}
     tcp_timeout = {{ .Values.bloom.control.tcp_timeout | default "300" }}
 
 
     [proxy]
 
     shard_default = {{ .Values.bloom.proxy.shard_default | default 0 }}
 
-    {{ range .Values.bloom.proxy.shards | required "At least one valis list entry is required" }}
+    {{ range .Values.bloom.proxy.shards | required "At least one valid list entry is required" }}
     [[proxy.shard]]
     shard = {{ .shard }}
-    host = {{ .host }}
+    host = {{ .host | quote}}
     port = {{ .port }}
     {{end}}
 
@@ -42,7 +42,7 @@ data:
 
     [redis]
 
-    host = {{.Values.bloom.redis.host | default "redis.svc.cluster.local" }}
+    host = {{.Values.bloom.redis.host | default "redis.svc.cluster.local" | quote }}
     port = {{.Values.bloom.redis.port | default 6379 }}
 
     database = {{ .Values.bloom.redis.database |default 0 }}
@@ -53,4 +53,4 @@ data:
     connection_timeout_seconds = {{ .Values.bloom.redis.connection_timeout_seconds | default 1 }}
 
     max_key_size = {{ .Values.bloom.redis.max_key_size | default 256000 }}
-    max_key_expiration = {{ .Values.bloom.redis.max_key_expiration | default 2592000 }}
+    max_key_expiration = {{ .Values.bloom.redis.max_key_expiration | default 2592000 | int}}

deploy/charts/bloom/templates/deployment.yaml

@@ -56,8 +56,8 @@ spec:
             {{- toYaml .Values.resources | nindent 12 }}
           volumeMounts:
             - mountPath: /etc/bloom.cfg
-              name: config
               subPath: bloom.cfg
+              name: config
       {{- with .Values.nodeSelector }}
       nodeSelector:
         {{- toYaml . | nindent 8 }}

deploy/charts/bloom/templates/ingress.yaml

@@ -1,6 +1,6 @@
 {{- $fullName := include "bloom.fullname" . -}}
 {{- $labels := include "bloom.labels" . }}
-{{- $svcPort := .Values.service.port -}}
+{{- $svcPort := .Values.service.http.port -}}
 {{- $kubeversion := .Capabilities.KubeVersion.GitVersion}}
 {{ range .Values.bloom.proxy.shards }}
 {{- if .ingress.enabled -}}
@@ -31,7 +31,7 @@ metadata:
 
 spec:
   {{- if and .ingress.className (semverCompare ">=1.18-0" $kubeversion) }}
-  ingressClassName: {{ .Values.ingress.className }}
+  ingressClassName: {{ .ingress.className }}
   {{- end }}
   {{- if .ingress.tls }}
   tls:

deploy/charts/bloom/templates/service.yaml

@@ -11,9 +11,5 @@ spec:
       targetPort: http
       protocol: TCP
       name: http
-    - port: {{ .Values.service.control.port }}
-      targetPort: control
-      protocol: TCP
-      name: control
   selector:
     {{- include "bloom.selectorLabels" . | nindent 4 }}

deploy/charts/bloom/values.yaml

@@ -8,7 +8,7 @@ image:
   repository: valeriansaliou/bloom
   pullPolicy: IfNotPresent
   # Overrides the image tag whose default is the chart appVersion.
-  tag: ""
+  tag:
 
 imagePullSecrets: []
 nameOverride: ""
@@ -24,42 +24,42 @@ serviceAccount:
   name: ""
 bloom:
   server:
-    log_level: "error"
-    inet: "[::1]:8080"
+    log_level: "debug"
+    inet: "0.0.0.0:8080"
   control:
     tcp_timeout: 300
-    inet: "[::1]:8811"
+    inet: "0.0.0.0:8811"
   proxy:
     # -- sets the default shard if none is set in the request
     shard_default: 0
     shards:
       - shard: 0
-        host: ""
-        port: ""
+        host: "someservice.namespace.svc.cluster.local"
+        port: someport
         ingress:
           enabled: true
-          className: ""
-          annotations: { }
+          className: "nginx"
+          annotations:
           # kubernetes.io/ingress.class: nginx
-          # kubernetes.io/tls-acme: "true"
+            kubernetes.io/tls-acme: "true"
           hosts:
-            - host: chart-example.local
+            - host: service.example.com
               paths:
                 - path: /
                   pathType: ImplementationSpecific
-          tls: [ ]
-          #  - secretName: chart-example-tls
-          #    hosts:
-          #      - chart-example.local
+          tls:
+            - secretName: chart-example-tls
+              hosts:
+                - service.example.com
   cache:
     ttl_default: 600
     executor_pool: 64
     disable_read: false
     disable_write: false
     compress_body: true
   redis:
-    host:
-    port:
+    host: "redis.redis.svc.cluster.local"
+    port: 6379
     database: 0
     pool_size: 80
     max_lifetime_seconds: 60
@@ -84,7 +84,7 @@ securityContext: {}
 service:
   type: ClusterIP
   http:
-    port: 80
+    port: 8080
   control:
     port: 8811