Mortom123 commented on code in PR #448:
URL: https://github.com/apache/pulsar-helm-chart/pull/448#discussion_r1469671440


##########
charts/pulsar/values.yaml:
##########
@@ -1326,11 +1327,13 @@ pulsar_manager:
     hostname: ""
     path: "/"
 
-  ## If set use existing secret with specified name to set pulsar admin 
credentials.
+  ## If set use an existing secret with specified name to set pulsar admin 
credentials
+  ## Leave empty to generate either a new secret with random passwords or 
reuse one from a previous installation
+  ## You can find out the actual password by running the following command:
+  ## kubectl get secret -l app=pulsar-manager -o=jsonpath="{.items[0].data}" | 
base64 --decode
+  ## And decode any key by using:
+  ## kubectl get secret -l app=pulsar-manager 
-o=jsonpath="{.items[0].data.KEY}" | base64 --decode
   existingSecretName:

Review Comment:
   Yes. Setting "existingSecretName" basically disables the creation of a 
secret and lets the application installer inject their own credentials by 
creating a secret beforehand. Because the section below which was used to 
enable setting username / password is removed now, this is needed. We could 
also remove it and use a section in the values to define username/password.



-- 
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]

Reply via email to