fix bug in values.yaml

Signed-off-by: Tim Ramlot <[email protected]>
jetstack · Sep 30, 2024 · 336a802 · 336a802
1 parent 22ca1e7
commit 336a802
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 22 deletions.
diff --git a/deploy/charts/venafi-kubernetes-agent/README.md b/deploy/charts/venafi-kubernetes-agent/README.md
@@ -440,6 +440,16 @@ Control Plane.
 > ```
 
 Enable or disable the PodDisruptionBudget resource, which helps prevent downtime during voluntary disruptions such as during a Node upgrade.
+#### **podDisruptionBudget.minAvailable** ~ `number`
+
+Configure the minimum available pods for disruptions. Can either be set to an integer (e.g. 1) or a percentage value (e.g. 25%).  
+Cannot be used if `maxUnavailable` is set.
+
+#### **podDisruptionBudget.maxUnavailable** ~ `number`
+
+Configure the maximum unavailable pods for disruptions. Can either be set to an integer (e.g. 1) or a percentage value (e.g. 25%).  
+Cannot be used if `minAvailable` is set.
+
 ### CRDs
 
 
@@ -465,20 +475,5 @@ This option makes it so that the "helm.sh/resource-policy": keep annotation is a
 > ```
 
 When set to false, the rendered output does not contain the. VenafiConnection CRDs and RBAC. This is useful for when the. Venafi Connection resources are already installed separately.
-#### **podDisruptionBudget.enabled** ~ `bool`
-> Default value:
-> ```yaml
-> false
-> ```
-#### **podDisruptionBudget.minAvailable** ~ `number`
-> Default value:
-> ```yaml
-> 1
-> ```
-#### **podDisruptionBudget.maxUnavailable** ~ `number`
-> Default value:
-> ```yaml
-> 1
-> ```
 
 <!-- /AUTO-GENERATED -->
diff --git a/deploy/charts/venafi-kubernetes-agent/values.schema.json b/deploy/charts/venafi-kubernetes-agent/values.schema.json
@@ -477,14 +477,15 @@
     },
     "helm-values.podDisruptionBudget.enabled": {
       "default": false,
+      "description": "Enable or disable the PodDisruptionBudget resource, which helps prevent downtime during voluntary disruptions such as during a Node upgrade.",
       "type": "boolean"
     },
     "helm-values.podDisruptionBudget.maxUnavailable": {
-      "default": 1,
+      "description": "Configure the maximum unavailable pods for disruptions. Can either be set to an integer (e.g. 1) or a percentage value (e.g. 25%).\nCannot be used if `minAvailable` is set.",
       "type": "number"
     },
     "helm-values.podDisruptionBudget.minAvailable": {
-      "default": 1,
+      "description": "Configure the minimum available pods for disruptions. Can either be set to an integer (e.g. 1) or a percentage value (e.g. 25%).\nCannot be used if `maxUnavailable` is set.",
       "type": "number"
     },
     "helm-values.podSecurityContext": {

diff --git a/deploy/charts/venafi-kubernetes-agent/values.yaml b/deploy/charts/venafi-kubernetes-agent/values.yaml
@@ -254,11 +254,13 @@ podDisruptionBudget:
   # Configure the minimum available pods for disruptions. Can either be set to
   # an integer (e.g. 1) or a percentage value (e.g. 25%).
   # Cannot be used if `maxUnavailable` is set.
+  # +docs:property
   # minAvailable: 1
 
   # Configure the maximum unavailable pods for disruptions. Can either be set to
   # an integer (e.g. 1) or a percentage value (e.g. 25%).
   # Cannot be used if `minAvailable` is set.
+  # +docs:property
   # maxUnavailable: 1
 
 # +docs:section=CRDs
@@ -284,8 +286,3 @@ crds:
     # VenafiConnection CRDs and RBAC. This is useful for when the
     # Venafi Connection resources are already installed separately.
     include: false
-
-podDisruptionBudget:
-  enabled: false
-  minAvailable: 1
-  maxUnavailable: 1