Code review comment for lp:~axwalk/goose/nova-availability-zones

Revision history for this message
Andrew Wilkins (axwalk) wrote :

Reviewers: mp+222275_code.launchpad.net,

Message:
Please take a look.

Description:
Add support for Availability Zones

- Added ListAvailabilityZones to Nova client
- Added AvailabilityZone field to RunServerOpts
- Added AvailabilityZone field to ServerDetail
- Updated nova test-service to support all of the above

Availability zones are an OpenStack extension,
so I've made it so that ListAvailabilityZones will
ignore any 404 to the os-availability-zone URL.

https://code.launchpad.net/~axwalk/goose/nova-availability-zones/+merge/222275

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/103900045/

Affected files (+260, -64 lines):
   A [revision details]
   M nova/live_test.go
   M nova/local_test.go
   M nova/nova.go
   M testservices/novaservice/service.go
   M testservices/novaservice/service_http.go
   M testservices/novaservice/service_http_test.go

« Back to merge proposal