From 8512678932abafd95714f59a62412d1cd695ab83 Mon Sep 17 00:00:00 2001
From: Derek Su <derek.su@suse.com>
Date: Fri, 1 Apr 2022 13:16:05 +0800
Subject: [PATCH] Remove minimum and maximum markers from CRD

Fix the helm upgrade failure on kuberenetes v1.18.
Root cause: https://github.com/kubernetes/kubernetes/issues/87675

Longhorn 3631

Signed-off-by: Derek Su <derek.su@suse.com>
---
 chart/templates/crds.yaml | 6 ------
 deploy/longhorn.yaml      | 8 +-------
 2 files changed, 1 insertion(+), 13 deletions(-)

diff --git a/chart/templates/crds.yaml b/chart/templates/crds.yaml
index 893ed56..2d7d874 100644
--- a/chart/templates/crds.yaml
+++ b/chart/templates/crds.yaml
@@ -1808,14 +1808,12 @@ spec:
                   type: object
                 type: object
               engineManagerCPURequest:
-                minimum: 0
                 type: integer
               evictionRequested:
                 type: boolean
               name:
                 type: string
               replicaManagerCPURequest:
-                minimum: 0
                 type: integer
               tags:
                 items:
@@ -2029,7 +2027,6 @@ spec:
             properties:
               concurrency:
                 description: The concurrency of taking the snapshot/backup.
-                minimum: 1
                 type: integer
               cron:
                 description: The cron setting.
@@ -2049,8 +2046,6 @@ spec:
                 type: string
               retain:
                 description: The retain count of the snapshot/backup.
-                maximum: 50
-                minimum: 1
                 type: integer
               task:
                 description: The recurring job type. Can be "snapshot" or "backup".
@@ -2626,7 +2621,6 @@ spec:
                   type: string
                 type: array
               numberOfReplicas:
-                minimum: 1
                 type: integer
               recurringJobs:
                 description: Deprecated. Replaced by a separate resource named "RecurringJob"
diff --git a/deploy/longhorn.yaml b/deploy/longhorn.yaml
index bc49737..027b352 100644
--- a/deploy/longhorn.yaml
+++ b/deploy/longhorn.yaml
@@ -1979,14 +1979,12 @@ spec:
                   type: object
                 type: object
               engineManagerCPURequest:
-                minimum: 0
                 type: integer
               evictionRequested:
                 type: boolean
               name:
                 type: string
               replicaManagerCPURequest:
-                minimum: 0
                 type: integer
               tags:
                 items:
@@ -2204,7 +2202,6 @@ spec:
             properties:
               concurrency:
                 description: The concurrency of taking the snapshot/backup.
-                minimum: 1
                 type: integer
               cron:
                 description: The cron setting.
@@ -2224,8 +2221,6 @@ spec:
                 type: string
               retain:
                 description: The retain count of the snapshot/backup.
-                maximum: 50
-                minimum: 1
                 type: integer
               task:
                 description: The recurring job type. Can be "snapshot" or "backup".
@@ -2817,7 +2812,6 @@ spec:
                   type: string
                 type: array
               numberOfReplicas:
-                minimum: 1
                 type: integer
               recurringJobs:
                 description: Deprecated. Replaced by a separate resource named "RecurringJob"
@@ -3038,7 +3032,7 @@ rules:
   verbs: ["list", "watch"]
 - apiGroups: ["admissionregistration.k8s.io"]
   resources: ["mutatingwebhookconfigurations", "validatingwebhookconfigurations"]
-  verbs: ["list", "create", "patch"]
+  verbs: ["get", "list", "create", "patch", "delete"]
 ---
 # Source: longhorn/templates/clusterrolebinding.yaml
 apiVersion: rbac.authorization.k8s.io/v1