lhotari commented on code in PR #428:
URL: https://github.com/apache/pulsar-helm-chart/pull/428#discussion_r1452425956
##########
charts/pulsar/values.yaml:
##########
@@ -1151,3 +1157,28 @@ job:
ttl:
enabled: false
secondsAfterFinished: 3600
+
+# This section is intended for cluster providers where all containers,
including init containers,
+# need to provide the number of resources they are going to use.
+initContainer_resources:
+ zookeeper_ready:
+ requests:
+ memory: 128Mi
+ cpu: 0.1
+ cs_ready:
+ requests:
+ memory: 128Mi
+ cpu: 0.1
+ verify_cluster_id:
+ requests:
+ memory: 128Mi
+ cpu: 0.1
+ broker_ready:
+ requests:
+ memory: 128Mi
+ cpu: 0.1
+ bookkeeper_ready:
+ requests:
+ memory: 128Mi
+ cpu: 0.1
Review Comment:
Since it's `requests` and not `limits`, it might not be that critical. I
checked that the defaults for requests for memory in the chart for other
containers are not adding any head room over `-Xmx`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
