Skip to content
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

Evaluate Regional volumes for the two ZK pods meant to migrate at zone outage #191

Merged
merged 9 commits into from
Nov 29, 2018

Conversation

solsson
Copy link
Contributor

@solsson solsson commented Jul 21, 2018

After trying I'm not sure https://cloud.google.com/kubernetes-engine/docs/concepts/persistent-volumes#regional_persistent_disks was meant to be used with automatic volume provisioning. To me it appears to be useful only with single-pod statefulsets, due to the hard coded zone names.

@solsson solsson added this to the 5.0 - Java 11 milestone Nov 28, 2018
@solsson
Copy link
Contributor Author

solsson commented Nov 28, 2018

This PR is a good fit for the breaking 5.0 release as it introduces storage classes for zoo that lets you chose persistence. The name kafka-zookeeper-regional is also OK even if you go for other types of persistence, as it indicates the idea to have volumes that are not bound to a single availability zone.

I haven't found a way to use for example emptyDir with storage classes, but you can always remove the volume template to revert to the old behavior. Ephemeral zoo has been an element of surprise (see for example #123) so it's probably not a good default.

@solsson solsson merged commit 81774cc into master Nov 29, 2018
@solsson solsson mentioned this pull request Jan 16, 2019
solsson added a commit that referenced this pull request May 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant