upgrade to using helm3 crd procedure since crd-install hook is no lon…

…ger supported fixes #170 chart.Values.crd.install is no longer support since crd MUST be install refer: https://helm.sh/docs/topics/charts/#custom-resource-definitions-crds Signed-off-by: vincentcui <[email protected]>
jaegertracing · Nov 26, 2020 · 013fbb7 · 013fbb7
1 parent 596c29b
commit 013fbb7
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 14 deletions.
diff --git a/charts/jaeger-operator/Chart.yaml b/charts/jaeger-operator/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v1
 description: jaeger-operator Helm chart for Kubernetes
 name: jaeger-operator
-version: 2.17.0
+version: 2.18.0
 appVersion: 1.19.0
 home: https://www.jaegertracing.io/
 icon: https://www.jaegertracing.io/img/jaeger-icon-reverse-color.svg

diff --git a/charts/jaeger-operator/README.md b/charts/jaeger-operator/README.md
@@ -55,7 +55,6 @@ The following table lists the configurable parameters of the jaeger-operator cha
 | `image.pullPolicy`      | Controller container image pull policy                                                                      | `IfNotPresent`                  |
 | `jaeger.create`         | Jaeger instance will be created                                                                             | `false`                         |
 | `jaeger.spec`           | Jaeger instance specification                                                                               | `{}`                            |
-| `crd.install`           | CustomResourceDefinition will be installed                                                                  | `true`                          |
 | `rbac.create`           | All required roles and rolebindings will be created                                                         | `true`                          |
 | `serviceAccount.create` | Service account to use                                                                                      | `true`                          |
 | `rbac.pspEnabled`       | Pod security policy for pod will be created and included in rbac role                                       | `false`                         |

diff --git a/charts/jaeger-operator/crds/crd.yaml b/charts/jaeger-operator/crds/crd.yaml
@@ -2,9 +2,6 @@ apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
   name: jaegers.jaegertracing.io
-  annotations:
-    "helm.sh/hook": crd-install
-    "helm.sh/hook-delete-policy": "before-hook-creation"
   labels:
     app: jaeger-operator
 spec:

diff --git a/charts/jaeger-operator/templates/crds.yaml b/charts/jaeger-operator/templates/crds.yaml
diff --git a/charts/jaeger-operator/values.yaml b/charts/jaeger-operator/values.yaml
@@ -7,9 +7,6 @@ image:
   pullPolicy: IfNotPresent
   imagePullSecrets: []
 
-crd:
-  install: true
-
 jaeger:
   # Specifies whether Jaeger instance should be created
   create: false