(#367) Use Cinder v2 API if available

ggiamarchi / vagrant-openstack-provider

Use Vagrant to manage OpenStack Cloud instances.

MIT License

245 stars 102 forks source link

(#367) Use Cinder v2 API if available #368

Closed danielparks closed 5 years ago

danielparks commented 5 years ago

Typically OpenStack has separate endpoints for versions 1, 2, and 3 of Cinder API. This changes the volume code to try the endpoint returned by the volumev2 endpoint, if it exists.

coveralls commented 5 years ago

Coverage increased (+4.01%) to 91.586% when pulling 7c5089998c41ec24562b77b7053d3976b056ccf4 on danielparks:cinderv2-default into 0e60bd3235e077df3aca180523f6fd5a37a04152 on ggiamarchi:master.

danielparks commented 5 years ago

I restored the test that confirms it works if volume advertises the v2 API instead of v1. Now there are tests for:

API v1 being advertised on volume
API v2 being advertised on volume
API v2 being advertised on volumev2 and anything being advertised on volume