Merge pull request #73 from cetic/develop

Add Warning comment in the README.md
cetic · Aug 28, 2020 · 58c84ff · 58c84ff
2 parents c803f3e + 4407c4a
commit 58c84ff
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 14 deletions.
diff --git a/Chart.yaml b/Chart.yaml
@@ -1,7 +1,7 @@
 ---
 apiVersion: v2
 name: nifi
-version: 0.5.0
+version: 0.5.1
 appVersion: 1.11.4
 description: Apache NiFi is a software project from the Apache Software Foundation designed to automate the flow of data between software systems.
 keywords:

diff --git a/README.md b/README.md
@@ -6,6 +6,8 @@
 
 This [Helm](https://github.com/kubernetes/helm) chart installs [nifi](https://nifi.apache.org/) in a Kubernetes cluster.
 
+:warning: As explained in several issues, this helm chart does not allow TLS configurations and secure Admin access. We are currently working on this feature, but any help is welcome.
+
 ## Prerequisites
 
 - Kubernetes cluster 1.10+
@@ -139,11 +141,8 @@ The following table lists the configurable parameters of the nifi chart and the
 | **jvmMemory**                                                               |
 | `jvmMemory`                                                                 | bootstrap jvm size                                                                                                 | `2g`                            |
 | **SideCar**                                                                 |
-| `sidecar.image`                                                             | Separate image for tailing each log separately                                                                     | `ez123/alpine-tini`             |
-| `sidecar.tag`                                                               | Image tag                                                                                                          | `latest`                        |
-| **BusyBox**                                                                 |
-| `busybox.image`                                                             | Separate image for initContainer that verifies zookeeper is accessible                                             | `busybox`                       |
-| `busybox.tag`                                                               | Image tag                                                                                                          | `latest`                        |
+| `sidecar.image`                                                             | Separate image for tailing each log separately and checking zookeeper connectivity                                 | `busybox`                       |
+| `sidecar.tag`                                                               | Image tag                                                                                                          | `1.32.0`                        |
 | **Resources**                                                               |
 | `resources`                                                                 | Pod resource requests and limits for logs                                                                          | `{}`                            |
 | **logResources**                                                            |

diff --git a/templates/statefulset.yaml b/templates/statefulset.yaml
@@ -74,7 +74,7 @@ spec:
       initContainers:
 {{- if .Values.properties.isNode }}
       - name: zookeeper
-        image: "{{ .Values.busybox.image }}:{{ .Values.busybox.tag }}"
+        image: "{{ .Values.sidecar.image }}:{{ .Values.sidecar.tag }}"
         command:
         - sh
         - -c

diff --git a/values.yaml b/values.yaml
@@ -131,15 +131,10 @@ ingress:
 # Amount of memory to give the NiFi java heap
 jvmMemory: 2g
 
-# Separate image for tailing each log separately
+# Separate image for tailing each log separately and checking zookeeper connectivity
 sidecar:
-  image: ez123/alpine-tini
-  tag: latest
-
-# Busybox image
-busybox:
   image: busybox
-  tag: latest
+  tag: "1.32.0"
 
 ## Enable persistence using Persistent Volume Claims
 ## ref: http://kubernetes.io/docs/user-guide/persistent-volumes/