Remove duplicate resources entries in values.yaml; Document new initC…

…ontainerResources param in helm chart's readme
k8tz · Jun 1, 2024 · 9b7148a · 9b7148a
1 parent 0e4a7a4
commit 9b7148a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/charts/k8tz/README.md b/charts/k8tz/README.md
@@ -80,6 +80,7 @@ If you want to use the helm built-in namespace (`helm --namespace`), you can com
 | service.type                       | Admission controller service type                                                                                                                                             | ClusterIP         |
 | service.port                       | Admission controller service port                                                                                                                                             | 443               |
 | resources                          | Resource requests and limitations for the admission controller                                                                                                                | {}                |
+| initContainerResources             | Resource requests and limitations for the bootstrap init container                                                                                                                 | {}                |
 | nodeSelector                       | Node selector for the admission controller                                                                                                                                    | {}                |
 | tolerations                        | Tolerations for the admission controller                                                                                                                                      | {}                |
 | topologySpreadConstraints          | TopologySpreadConstraints for the admission controller                                                                                                                        | []                |

diff --git a/charts/k8tz/values.yaml b/charts/k8tz/values.yaml
@@ -11,8 +11,6 @@ injectedInitContainerName: k8tz
 injectAll: true
 cronJobTimeZone: false  # requires kubernetes >=1.24.0-beta.0 with 'CronJobTimeZone' feature gate enabled (alpha)
 verbose: false
-initContainerResources: {} # resources claimed for the bootstrap init container
-resources: {} # resources claimed for the k8tz controller
 
 # Labels to apply to all resources
 labels: {}
@@ -68,6 +66,7 @@ service:
   type: ClusterIP
   port: 443
 
+# resources claimed for the k8tz controller
 resources: {}
   # We usually recommend not to specify default resources and to leave this as a conscious
   # choice for the user. This also increases chances charts run on environments with little
@@ -80,6 +79,15 @@ resources: {}
   #   cpu: 100m
   #   memory: 128Mi
 
+# resources claimed for the bootstrap init container
+initContainerResources: {} 
+  # limits:
+  #   cpu: 100m
+  #   memory: 128Mi
+  # requests:
+  #   cpu: 100m
+  #   memory: 128Mi
+
 nodeSelector: {}
 
 tolerations: []