[ 
https://issues.apache.org/jira/browse/JCLOUDS-515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13948419#comment-13948419
 ] 

Everett Toews commented on JCLOUDS-515:
---------------------------------------

The Volume domain object is not actually used to create volumes. Rather, it's 
the immutable object returned by Cinder when the create call is done.

>From create_volume.py we see that Cinder always sets the availability zone no 
>matter what so this value should always be present when the create call is 
>done.

>From VolumeApi.create(int , CreateVolumeOptions...) we can see that the 
>CreateVolumeOptions object is responsible for supplying the options during the 
>create call. It's here where you can optionally supply the availability zone 
>with something like CreateVolumeOptions.Builder.availabilityZone("nova").

AFAICT, this change isn't necessary unless I've mischaracterized the issue.

Jasdeep, do you have a code snippet demonstrating the problem?

> Cinder volume create does not require availability zone
> -------------------------------------------------------
>
>                 Key: JCLOUDS-515
>                 URL: https://issues.apache.org/jira/browse/JCLOUDS-515
>             Project: jclouds
>          Issue Type: Bug
>          Components: jclouds-core
>    Affects Versions: 1.7.1
>            Reporter: Jasdeep Hundal
>            Assignee: Jasdeep Hundal
>             Fix For: 1.8.0
>
>
> Openstack Cinder does not require specifying an availability zone for volume 
> creation, but the Cinder API as implemented in JClouds does. We should change 
> the check in the constructor of org.jclouds.openstack.cinder.v1.domain to 
> allow a null value for availability zone.
> The docs on the v1 API are not readily available but tracing through the 
> Cinder source code, we find this line that gets executed if we specify no 
> availability zone in either the v1 or v2 block storage APIs:
> https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L247
> It can also be noted the the Block Storage v2 request here does not require 
> an availability zone:
> http://docs.openstack.org/api/openstack-block-storage/2.0/content/POST_createVolume_v2__tenant_id__volumes_Volumes.html



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to