Skip to content

Commit

Permalink
fix: add annotation support for deployment manifest (#91)
Browse files Browse the repository at this point in the history
Signed-off-by: Engin Diri <[email protected]>
  • Loading branch information
Engin Diri authored Mar 30, 2022
1 parent 6f57997 commit c860b4c
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 4 deletions.
4 changes: 2 additions & 2 deletions charts/node-red/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ icon: https://nodered.org/about/resources/media/node-red-icon-2.png

type: application

version: 0.13.2
version: 0.14.0
appVersion: 2.2.2

keywords:
Expand All @@ -29,7 +29,7 @@ maintainers:
annotations:
artifacthub.io/containsSecurityUpdates: "false"
artifacthub.io/changes: |
- change port name of servicemonitor to default port
- add annotation support for deployment manifest
artifacthub.io/images: |
- name: node-red
image: docker.io/nodered/node-red:2.2.2
Expand Down
5 changes: 3 additions & 2 deletions charts/node-red/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# node-red ⚙

![Version: 0.13.2](https://img.shields.io/badge/Version-0.13.2-informational?style=for-the-badge)
![Version: 0.14.0](https://img.shields.io/badge/Version-0.14.0-informational?style=for-the-badge)
![Type: application](https://img.shields.io/badge/Type-application-informational?style=for-the-badge)
![AppVersion: 2.2.2](https://img.shields.io/badge/AppVersion-2.2.2-informational?style=for-the-badge)

Expand All @@ -26,7 +26,7 @@ helm repo update
To install the chart with the release name node-red run:

```bash
helm install node-red node-red/node-red --version 0.13.2
helm install node-red node-red/node-red --version 0.14.0
```

After a few seconds, node-red should be running.
Expand Down Expand Up @@ -55,6 +55,7 @@ The command removes all the Kubernetes components associated with the chart and
| Key | Type | Default | Description |
|-----|------|---------|-------------|
| affinity | object | `{}` | The affinity constraint |
| deploymentAnnotations | object | `{}` | Deployment annotations |
| deploymentStrategy | string | `""` | Specifies the strategy used to replace old Pods by new ones, default: `RollingUpdate` |
| env | list | `[]` | node-red env, see more environment variables in the [node-red documentation](https://nodered.org/docs/getting-started/docker) |
| extraVolumeMounts | string | `nil` | Extra Volume Mounts for the node-red pod |
Expand Down
4 changes: 4 additions & 0 deletions charts/node-red/templates/deployment.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
apiVersion: apps/v1
kind: Deployment
metadata:
{{- with .Values.deploymentAnnotations }}
annotations:
{{- toYaml . | nindent 4 }}
{{- end }}
name: {{ include "node-red.fullname" . }}
labels:
{{- include "node-red.labels" . | nindent 4 }}
Expand Down
3 changes: 3 additions & 0 deletions charts/node-red/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,9 @@ serviceAccount:
# -- Pod annotations
podAnnotations: {}

# -- Deployment annotations
deploymentAnnotations: {}

# -- Pod Security Context see [values.yaml](values.yaml)
podSecurityContext: {}
# fsGroup: 2000
Expand Down

0 comments on commit c860b4c

Please sign in to comment.