Vince-Chenal commented on code in PR #519: URL: https://github.com/apache/flink-kubernetes-operator/pull/519#discussion_r1091603421
########## docs/content/docs/operations/helm.md: ########## @@ -74,7 +74,8 @@ The configurable parameters of the Helm chart and which default values as detail | operatorPod.dnsPolicy | DNS policy to be used by the operator pod. | | | operatorPod.dnsConfig | DNS configuration to be used by the operator pod. | | | operatorPod.nodeSelector | Custom nodeSelector to be added to the operator pod. | | -| operatorPod.resources | Custom resources block to be added to the operator pod. | | +| operatorPod.resources | Custom resources block to be added to the operator pod on main container. | | +| operatorPod.flinkWebhookResources | Custom resources block to be added to the operator pod on flink-webhook container. | | Review Comment: In fact there are two containers within the same pod, that's why I decided to keep it within operatorPod field. `operatorPod.resources` is already confusing as we don't know which container it configures. what do you think? -- 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]
