Skip to content

Commit

Permalink
[stable/node-red] node-red chart enhancements (helm#12052)
Browse files Browse the repository at this point in the history
* switching node-red chart to SatefulSet

* based on the persistent nature of this chart as well as [this discussion](helm#1863), migrating the chart to a StatefulSet instead of a deployment. As a result bumping the major version
* bumping node-red docker image to the latest stable version
* using volumeClaimTemplates for statefulSet

Signed-off-by: Jeff Billimek <jeff@billimek.com>

* using Parallel podManagementPolicy

Signed-off-by: Jeff Billimek <jeff@billimek.com>

* revert to Deployment and leverage strategy types

Signed-off-by: Jeff Billimek <jeff@billimek.com>

* hard-code replica count

Signed-off-by: Jeff Billimek <jeff@billimek.com>
  • Loading branch information
billimek authored and Moritz Pein committed Apr 26, 2019
1 parent dc253a8 commit 4e522ad
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 5 deletions.
4 changes: 2 additions & 2 deletions stable/node-red/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
apiVersion: v1
appVersion: 0.19.4
appVersion: 0.19.6
description: Node-RED is flow-based programming for the Internet of Things
name: node-red
version: 1.1.0
version: 1.2.0
keywords:
- nodered
- node-red
Expand Down
1 change: 1 addition & 0 deletions stable/node-red/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ The following tables lists the configurable parameters of the Sentry chart and t
| `image.repository` | node-red image | `nodered/node-red-docker` |
| `image.tag` | node-red image tag | `0.19.4-v8` |
| `image.pullPolicy` | node-red image pull policy | `IfNotPresent` |
| `strategyType` | Specifies the strategy used to replace old Pods by new ones | `Recreate` |
| `flows` | Default flows configuration | `flows.json` |
| `nodeOptions` | Node.js runtime arguments | `` |
| `timezone` | Default timezone | `UTC` |
Expand Down
4 changes: 3 additions & 1 deletion stable/node-red/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@ metadata:
app.kubernetes.io/instance: {{ .Release.Name }}
app.kubernetes.io/managed-by: {{ .Release.Service }}
spec:
replicas: {{ .Values.replicaCount }}
replicas: 1
strategy:
type: {{ .Values.strategyType }}
selector:
matchLabels:
app.kubernetes.io/name: {{ include "node-red.name" . }}
Expand Down
5 changes: 3 additions & 2 deletions stable/node-red/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,12 @@
# This is a YAML-formatted file.
# Declare variables to be passed into your templates.

replicaCount: 1
# upgrade strategy type (e.g. Recreate or RollingUpdate)
strategyType: Recreate

image:
repository: nodered/node-red-docker
tag: 0.19.4-v8
tag: 0.19.6-v8
pullPolicy: IfNotPresent

nameOverride: ""
Expand Down

0 comments on commit 4e522ad

Please sign in to comment.