-
Notifications
You must be signed in to change notification settings - Fork 220
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
automate initialize #138
automate initialize #138
Conversation
This is going to be merged? |
Co-authored-by: Lari Hotari <lhotari@users.noreply.github.com>
Co-authored-by: Lari Hotari <lhotari@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@valeriano-manassero Thank you for the contribution. Would it be also possible to update README.md in this PR and remove the instructions of using |
done |
The helm option `--set initialize=true` is no longer needed. See: apache/pulsar-helm-chart#138 Signed-off-by: Paul Gier <paul.gier@datastax.com>
The helm option `--set initialize=true` is no longer needed. See: apache/pulsar-helm-chart#138 Signed-off-by: Paul Gier <paul.gier@datastax.com>
Fixes #114
Motivation
Having
initialize
value that must be manually set is prone to human error, there is a risk of running init again forcing the need of a rollback.Modifications
Using Helm capability to understand if it's an install or upgrade we can avoid to launch init when not needed.
Verifying this change