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

Add Regions to info_image #232

Merged
merged 1 commit into from
Jan 25, 2016
Merged

Conversation

bmamlin
Copy link
Contributor

@bmamlin bmamlin commented Jan 25, 2016

Now that tugboat supports API 2.0 (thanks!), reporting additional image info can help scripts use tugboat to automate creation of images from snapshots. We added Min Disk Size last October. Now, I'm proposing we add Regions too, since a valid region must be known to successfully build a droplet from an snapshot.

Regions are reported in a comma-separated list:

Name:             Redmine on 14.04
ID:               12438838
Distribution:     Ubuntu
Min Disk Size:    20GB
Regions:          nyc1,ams1,sfo1,nyc2,ams2,sgp1,lon1,nyc3,ams3,fra1

I considered sorting the list, but decided against it, since, while I couldn't find documentation of the significance of the ordering of regions returned by the API, the sequence mirrors the sequence on the website for creating droplets, suggesting the regions may be in order of preference.

FYI – I'm not a Ruby programmer, so apologies if I've missed something. I did ensure bundle exec rspec tests are all passing, so hopefully this simple feature suggestion will be acceptable & easy to merge.

petems added a commit that referenced this pull request Jan 25, 2016
@petems petems merged commit 88a055c into petems:master Jan 25, 2016
@petems
Copy link
Owner

petems commented Jan 25, 2016

Hi @bmamlin, thanks for your contribution 👍

I'll see if I can get this released in a 2.2.0 version by EOD 26th Jan. If I've not, ping me to remind me! 😄

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.

2 participants