Commit a425be8d by prateekpandey14 Committed by Guangbo

[openebs]: remove pre-install hook from secret and pod labels

- Updated README - Updated chart version - Updated values.yaml - Update pod labels for deployments `pre-install` hook: Executes after templates are rendered, but before any resources are created in Kubernetes. This hook has been added to the secret , will only be generated on chart install, to prevent overriding the certs anytime we upgrade the chart’s released instance. Thus `pre-install hook` prevents the secret creation if someone uses the older version of charts and upgraded to new version. Signed-off-by: 's avatarprateekpandey14 <prateek.pandey@openebs.io>
parent c66f117a
...@@ -65,6 +65,7 @@ The following table lists the configurable parameters of the OpenEBS chart and t ...@@ -65,6 +65,7 @@ The following table lists the configurable parameters of the OpenEBS chart and t
| `ndm.sparse.count` | Number of sparse files to be created | `1` | | `ndm.sparse.count` | Number of sparse files to be created | `1` |
| `ndm.filters.excludeVendors` | Exclude devices with specified vendor | `CLOUDBYT,OpenEBS` | | `ndm.filters.excludeVendors` | Exclude devices with specified vendor | `CLOUDBYT,OpenEBS` |
| `ndm.filters.excludePaths` | Exclude devices with specified path patterns | `loop,fd0,sr0,/dev/ram,/dev/dm-,/dev/md` | | `ndm.filters.excludePaths` | Exclude devices with specified path patterns | `loop,fd0,sr0,/dev/ram,/dev/dm-,/dev/md` |
| `ndm.filters.includePaths` | Include devices with specified path patterns | `""` |
| `jiva.image` | Image for Jiva | `quay.io/openebs/jiva` | | `jiva.image` | Image for Jiva | `quay.io/openebs/jiva` |
| `jiva.imageTag` | Image Tag for Jiva | `0.9.0` | | `jiva.imageTag` | Image Tag for Jiva | `0.9.0` |
| `jiva.replicas` | Number of Jiva Replicas | `3` | | `jiva.replicas` | Number of Jiva Replicas | `3` |
......
...@@ -12,7 +12,7 @@ metadata: ...@@ -12,7 +12,7 @@ metadata:
component: ndm-config component: ndm-config
data: data:
# udev-probe is default or primary probe which should be enabled to run ndm # udev-probe is default or primary probe which should be enabled to run ndm
# filterconfigs contains configs of filters - in the form of include # filterconfigs contails configs of filters - in ther form fo include
# and exclude comma separated strings # and exclude comma separated strings
node-disk-manager.config: | node-disk-manager.config: |
probeconfigs: probeconfigs:
...@@ -38,6 +38,6 @@ data: ...@@ -38,6 +38,6 @@ data:
- key: path-filter - key: path-filter
name: path filter name: path filter
state: true state: true
include: "" include: "{{ .Values.ndm.filters.includePaths }}"
exclude: "{{ .Values.ndm.filters.excludePaths }}" exclude: "{{ .Values.ndm.filters.excludePaths }}"
--- ---
...@@ -8,7 +8,6 @@ metadata: ...@@ -8,7 +8,6 @@ metadata:
release: {{ .Release.Name }} release: {{ .Release.Name }}
heritage: {{ .Release.Service }} heritage: {{ .Release.Service }}
component: ndm component: ndm
openebs.io/component-name: ndm
spec: spec:
updateStrategy: updateStrategy:
type: "RollingUpdate" type: "RollingUpdate"
...@@ -23,6 +22,8 @@ spec: ...@@ -23,6 +22,8 @@ spec:
app: {{ template "openebs.name" . }} app: {{ template "openebs.name" . }}
release: {{ .Release.Name }} release: {{ .Release.Name }}
component: ndm component: ndm
openebs.io/component-name: ndm
name: openebs-ndm
openebs.io/version: {{ .Values.release.version }} openebs.io/version: {{ .Values.release.version }}
spec: spec:
serviceAccountName: {{ template "openebs.serviceAccountName" . }} serviceAccountName: {{ template "openebs.serviceAccountName" . }}
......
...@@ -8,8 +8,6 @@ metadata: ...@@ -8,8 +8,6 @@ metadata:
release: {{ .Release.Name }} release: {{ .Release.Name }}
heritage: {{ .Release.Service }} heritage: {{ .Release.Service }}
component: admission-webhook component: admission-webhook
name: admission-webhook
openebs.io/component-name: maya-apiserver
spec: spec:
replicas: {{ .Values.webhook.replicas }} replicas: {{ .Values.webhook.replicas }}
selector: selector:
...@@ -19,7 +17,9 @@ spec: ...@@ -19,7 +17,9 @@ spec:
metadata: metadata:
labels: labels:
app: admission-webhook app: admission-webhook
name: admission-webhook
openebs.io/version: {{ .Values.release.version }} openebs.io/version: {{ .Values.release.version }}
openebs.io/component-name: admission-webhook
spec: spec:
{{- if .Values.webhook.nodeSelector }} {{- if .Values.webhook.nodeSelector }}
nodeSelector: nodeSelector:
......
...@@ -8,7 +8,6 @@ metadata: ...@@ -8,7 +8,6 @@ metadata:
release: {{ .Release.Name }} release: {{ .Release.Name }}
heritage: {{ .Release.Service }} heritage: {{ .Release.Service }}
component: localpv-provisioner component: localpv-provisioner
openebs.io/component-name: openebs-localpv-provisioner
spec: spec:
replicas: {{ .Values.provisioner.replicas }} replicas: {{ .Values.provisioner.replicas }}
selector: selector:
...@@ -21,7 +20,9 @@ spec: ...@@ -21,7 +20,9 @@ spec:
app: {{ template "openebs.name" . }} app: {{ template "openebs.name" . }}
release: {{ .Release.Name }} release: {{ .Release.Name }}
component: localpv-provisioner component: localpv-provisioner
name: openebs-localpv-provisioner
openebs.io/version: {{ .Values.release.version }} openebs.io/version: {{ .Values.release.version }}
openebs.io/component-name: openebs-localpv-provisioner
spec: spec:
serviceAccountName: {{ template "openebs.serviceAccountName" . }} serviceAccountName: {{ template "openebs.serviceAccountName" . }}
containers: containers:
......
...@@ -9,7 +9,6 @@ metadata: ...@@ -9,7 +9,6 @@ metadata:
heritage: {{ .Release.Service }} heritage: {{ .Release.Service }}
component: apiserver component: apiserver
name: maya-apiserver name: maya-apiserver
openebs.io/component-name: maya-apiserver
spec: spec:
replicas: {{ .Values.apiserver.replicas }} replicas: {{ .Values.apiserver.replicas }}
selector: selector:
...@@ -23,6 +22,7 @@ spec: ...@@ -23,6 +22,7 @@ spec:
release: {{ .Release.Name }} release: {{ .Release.Name }}
component: apiserver component: apiserver
name: maya-apiserver name: maya-apiserver
openebs.io/component-name: maya-apiserver
openebs.io/version: {{ .Values.release.version }} openebs.io/version: {{ .Values.release.version }}
spec: spec:
serviceAccountName: {{ template "openebs.serviceAccountName" . }} serviceAccountName: {{ template "openebs.serviceAccountName" . }}
...@@ -48,6 +48,8 @@ spec: ...@@ -48,6 +48,8 @@ spec:
# If "true" a default cstor sparse pool will be configured, if "false" it will not be configured. # If "true" a default cstor sparse pool will be configured, if "false" it will not be configured.
- name: OPENEBS_IO_INSTALL_DEFAULT_CSTOR_SPARSE_POOL - name: OPENEBS_IO_INSTALL_DEFAULT_CSTOR_SPARSE_POOL
value: "{{ .Values.apiserver.sparse.enabled }}" value: "{{ .Values.apiserver.sparse.enabled }}"
- name: OPENEBS_IO_CSTOR_POOL_SPARSE_DIR
value: "{{ .Values.ndm.sparse.path }}"
# OPENEBS_NAMESPACE provides the namespace of this deployment as an # OPENEBS_NAMESPACE provides the namespace of this deployment as an
# environment variable # environment variable
- name: OPENEBS_NAMESPACE - name: OPENEBS_NAMESPACE
......
...@@ -8,7 +8,6 @@ metadata: ...@@ -8,7 +8,6 @@ metadata:
release: {{ .Release.Name }} release: {{ .Release.Name }}
heritage: {{ .Release.Service }} heritage: {{ .Release.Service }}
component: provisioner component: provisioner
openebs.io/component-name: openebs-provisioner
spec: spec:
replicas: {{ .Values.provisioner.replicas }} replicas: {{ .Values.provisioner.replicas }}
selector: selector:
...@@ -21,6 +20,8 @@ spec: ...@@ -21,6 +20,8 @@ spec:
app: {{ template "openebs.name" . }} app: {{ template "openebs.name" . }}
release: {{ .Release.Name }} release: {{ .Release.Name }}
component: provisioner component: provisioner
name: openebs-provisioner
openebs.io/component-name: openebs-provisioner
openebs.io/version: {{ .Values.release.version }} openebs.io/version: {{ .Values.release.version }}
spec: spec:
serviceAccountName: {{ template "openebs.serviceAccountName" . }} serviceAccountName: {{ template "openebs.serviceAccountName" . }}
......
...@@ -8,7 +8,6 @@ metadata: ...@@ -8,7 +8,6 @@ metadata:
release: {{ .Release.Name }} release: {{ .Release.Name }}
heritage: {{ .Release.Service }} heritage: {{ .Release.Service }}
component: snapshot-operator component: snapshot-operator
openebs.io/component-name: openebs-snapshot-operator
spec: spec:
replicas: {{ .Values.snapshotOperator.replicas }} replicas: {{ .Values.snapshotOperator.replicas }}
selector: selector:
...@@ -23,7 +22,9 @@ spec: ...@@ -23,7 +22,9 @@ spec:
app: {{ template "openebs.name" . }} app: {{ template "openebs.name" . }}
release: {{ .Release.Name }} release: {{ .Release.Name }}
component: snapshot-operator component: snapshot-operator
name: openebs-snapshot-operator
openebs.io/version: {{ .Values.release.version }} openebs.io/version: {{ .Values.release.version }}
openebs.io/component-name: openebs-snapshot-operator
spec: spec:
serviceAccountName: {{ template "openebs.serviceAccountName" . }} serviceAccountName: {{ template "openebs.serviceAccountName" . }}
containers: containers:
......
...@@ -42,13 +42,6 @@ metadata: ...@@ -42,13 +42,6 @@ metadata:
chart: {{ template "openebs.chart" . }} chart: {{ template "openebs.chart" . }}
release: {{ .Release.Name }} release: {{ .Release.Name }}
heritage: {{ .Release.Service }} heritage: {{ .Release.Service }}
# Helm hook annotations in order to ensure that the certs
# will only be generated on chart install. This will
# prevent overriding the certs anytime we upgrade the chart’s
# released instance.
annotations:
"helm.sh/hook": "pre-install"
"helm.sh/hook-delete-policy": "before-hook-creation"
type: Opaque type: Opaque
data: data:
{{- if .Values.webhook.generateTLS }} {{- if .Values.webhook.generateTLS }}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment