freeznet commented on pull request #11213: URL: https://github.com/apache/pulsar/pull/11213#issuecomment-876854749
I think we'd better make the unifying of helm install command across the whole docs. This means, for the sections `Get started in Kubernetes` and `Install Apache Pulsar using Helm` should using the same install command. So if we decide to use `helm install`, this PR is LGTM. otherwise, a new PR should be created to change `helm install` to `helm upgrade --install` in `Get started in Kubernetes`. Also, from my experience, a ready to used command for the first-time deployment is better for the user, so I suggest putting `--set initialize=true` in the code snippet by default. heard some of the users failed with the first-time deployment because they forget to set `--set initialize=true`. -- 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]
