From 02b7149b5fb477fa449384ad3bedddd3c239d665 Mon Sep 17 00:00:00 2001 From: Kevin Subileau Date: Tue, 18 Aug 2020 17:38:43 +0200 Subject: [PATCH 1/3] Replace alpine-tiny by busybox (#71) * ez123/alpine-tini is not updated regularly. * There is no need to pull an additional image like alpine-tiny as busybox can also be used for the log writing containers as well Signed-off-by: Kevin Subileau --- README.md | 7 ++----- templates/statefulset.yaml | 2 +- values.yaml | 9 ++------- 3 files changed, 5 insertions(+), 13 deletions(-) diff --git a/README.md b/README.md index de2f54c4..d9d2829b 100644 --- a/README.md +++ b/README.md @@ -139,11 +139,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 index 41fe1b98..7210bb4b 100644 --- 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 index c575238d..0974e5bd 100644 --- 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/ From 653801492fa772de8573c248d6e6649b4159a5d4 Mon Sep 17 00:00:00 2001 From: Alexandre Nuttinck Date: Fri, 28 Aug 2020 18:37:49 +0200 Subject: [PATCH 2/3] Update README.md --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index d9d2829b..5b71da9c 100644 --- 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+ From 4407c4af571cb5c4d063410119d461d0e08789a4 Mon Sep 17 00:00:00 2001 From: Alexandre Nuttinck Date: Fri, 28 Aug 2020 18:38:28 +0200 Subject: [PATCH 3/3] Update Chart.yaml --- Chart.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Chart.yaml b/Chart.yaml index f25db45a..2d1a30d5 100644 --- 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: