search

openSUSE / vagrant-ceph

Builds a cluster of servers using libvirt. Supports multiple configurations.
GNU Lesser General Public License v3.0
18 stars 22 forks source link

Unify config #36

Closed marcosps closed 5 years ago

marcosps commented 5 years ago 
config.yml: Unify sleep before salt-key accept

When setting up a ceph cluster in a slower machine, salt-minions could
take more time to request the keys to master. In order to make this
process more reliable, change the sleep to 20 to all config entries,
rather having custom values for different configurations.

Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.de>
jschmid1 commented 5 years ago

This is the base PR for #37, right? They should be merged to one PR

marcosps commented 5 years ago

@jschmid1 I removed the already pushed commit, remaining only a change to increase the sleep time because in some slower machines it is necessary. At least, this now makes my machine always "provision" the VMs successfully.

jschmid1 commented 5 years ago

@marcosps I'm not convinced that this should go upstream. This may be a change that is suited to your local setup and should remain 'local'. I personally don't have problems and I think the CI is also fine with the 10 seconds

marcosps commented 5 years ago

@jschmid1 fair enough, I believe I need a new machine soon :)

Closing.