Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Quick fix for adding default pools on startup, and account for VolumeTypeNotAvailableInZone errors #10
Conversation
axw
reviewed
Jan 28, 2015
| + // TODO - we do this here because upgrade no longer handle new installs. | ||
| + // Fix this once we sort out what to do. | ||
| + a.startWorkerAfterUpgrade(runner, "storage pools", func() (worker.Worker, error) { | ||
| + addDefaultPools := func(_ <-chan struct{}) error { |
|
LGTM |
added a commit
that referenced
this pull request
Jan 28, 2015
wallyworld
merged commit e3d1649
into
storage-feature
Jan 28, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
wallyworld commentedJan 28, 2015
Upgrades now don't run if deploying a fresh install, so work around that to create default pools.
Account for the fact that not all EBS instances are available in every zone.
Tested live on AWS. Could deploy to ebs and ebs-ssd pools.
storage-attached hook works also.