[13:38:30] <andrewbogott>	 hashar: I'm trying to catch up with your comments on https://gerrit.wikimedia.org/r/c/operations/puppet/+/1282006
[13:38:44] <andrewbogott>	 the create-volume-group thing just needs the other exec to have a different name doesn't it?
[13:39:12] <andrewbogott>	 And I'm confused about the multiple parted package def since ensure_packages is supposed to permit multiple defines of the same package...
[14:51:54] <taavi>	 andrewbogott: godog: this is what I was thinking re zookeeper and cloud-privaste: https://gerrit.wikimedia.org/r/c/operations/puppet/+/1282372
[14:53:43] <andrewbogott>	 seems fine. For educational purposes I'm interested in how/if that's better than https://gerrit.wikimedia.org/r/c/operations/puppet/+/1278524
[14:55:50] <godog>	 I'm about to go afk though I'll catch up tomorrow
[14:56:21] <godog>	 andrewbogott: tl;dr we can keep the cluster definition in zookeeper_clusters with taavi's patch
[15:54:42] <andrewbogott>	 seems good