lhotari commented on code in PR #457:
URL: https://github.com/apache/pulsar-helm-chart/pull/457#discussion_r1489722176
##########
charts/pulsar/values.yaml:
##########
@@ -1314,24 +1314,30 @@ pulsar_manager:
## If you enabled authentication support
## JWT_TOKEN: <token>
## SECRET_KEY: data:base64,<secret key>
+
+ # the pulsar manager image relies on these variables, if they are not set
the backend will keep crashing
+ # however, feel free to overwrite them
+ SPRING_CONFIGURATION_FILE:
"/pulsar-manager/pulsar-manager/application.properties"
+ PULSAR_MANAGER_OPTS: " -Dlog4j2.formatMsgNoLookups=true"
## Pulsar manager service
## templates/pulsar-manager-service.yaml
##
service:
type: LoadBalancer
Review Comment:
Configuring ingress happens in a different way. In this case, simply
changing `type: LoadBalancer` to `type: ClusterIP` will ensure that
pulsar-manager-service doesn't get exposed by default.
(We could consider using `type: ClusterIP` as the default in values file for
all services, but that could be handled separately and would be a breaking
change, related to #444.)
--
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]
