EC2 CreateVolumes/DescribeVolumes strange status

[Attila Fazekas]

In the EC2 CreateVolume and DescribeVolumes response status part I see strange additional characters:
For example: 'creating (, , None, )'

Expected result:
Statuses from this set: 'creating', 'available', 'in-use', 'deleting', 'deleted', 'error'

Version info:
openstack-cinder-2012.2-1.fc18
openstack-nova-api-2012.2-1.fc18

Note: Looks like all statuses have the same additional suffix.

[Attila Fazekas (afazekas)]

It is visible even in the command line.

euca-create-volume -z nova -s 1
VOLUME vol-00000010 1 nova creating (, , None, ) 2012-11-15T09:21:37.557659

[Attila Fazekas (afazekas)]

Reproducibility depends on the user/tenant.

I got correct responses with the demo user, but incorrect with the admin user/tenant.

[Attila Fazekas (afazekas)]

"attaching" not a valid EC2 status

[Attila Fazekas (afazekas)]

"attaching" and "detaching" can be showed as "in-use".

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/27098

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/27245

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/27098
Committed: http://github.com/openstack/nova/commit/c62d7636b6ba1c36a96119a87fff8495c5a1dc21
Submitter: Jenkins
Branch: master

commit c62d7636b6ba1c36a96119a87fff8495c5a1dc21
Author: Chris Yeoh <email address hidden>
Date: Thu Apr 18 15:44:59 2013 +0930

    Fix ec2 CreateVolumes/DescribeVolumes status

    In admin context CreateVolumes/DescribeVolumes returns
    extraneous information such as project_id, host, instance
    data etc. This results in output which is not compatible
    with the ec2 API. This change removes the addition
    of the extra information.

    This only partially fixes bug #1074901 as mapping the internal
    volume states to ones specified by the ec2 api results in the devstack
    euca.sh exercises to fail. But that can't be fixed until the fix
    in this patch is through as the extraneous information returned by DescribeVolumes
    causes euca-describe-volumes to not to show the attachment state for
    a volume which is needed by the euca exercises.

    Change-Id: I48a96fbfe4bb2e3da9514d688638ecdf3a168303

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/27245
Committed: http://github.com/openstack/nova/commit/fd6fa90d5f293be7fac4f1e612958cc8e55ee796
Submitter: Jenkins
Branch: master

commit fd6fa90d5f293be7fac4f1e612958cc8e55ee796
Author: Chris Yeoh <email address hidden>
Date: Mon Apr 22 10:51:00 2013 +0930

    ec2 CreateVolumes/DescribeVolumes status mapping

    'attaching' and 'detaching' are not valid ec2 API volume status
    values. Instead map them both to 'in-use'.

    Note this will not pass the devstack exercises until
    https://review.openstack.org/#/c/27244/ goes through

    Fixes bug #1074901

    Change-Id: Id3dfd525d24d622aab5e350e42713fdb7671b31e

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/grizzly
Review: https://review.openstack.org/27781

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/grizzly
Review: https://review.openstack.org/27782

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/27781
Committed: http://github.com/openstack/nova/commit/c18519501af57e91deb35792c53e542116ccba64
Submitter: Jenkins
Branch: stable/grizzly

commit c18519501af57e91deb35792c53e542116ccba64
Author: Chris Yeoh <email address hidden>
Date: Thu Apr 18 15:44:59 2013 +0930

    Fix ec2 CreateVolumes/DescribeVolumes status

    In admin context CreateVolumes/DescribeVolumes returns
    extraneous information such as project_id, host, instance
    data etc. This results in output which is not compatible
    with the ec2 API. This change removes the addition
    of the extra information.

    This only partially fixes bug #1074901 as mapping the internal
    volume states to ones specified by the ec2 api results in the devstack
    euca.sh exercises to fail. But that can't be fixed until the fix
    in this patch is through as the extraneous information returned by DescribeVolumes
    causes euca-describe-volumes to not to show the attachment state for
    a volume which is needed by the euca exercises.

    Change-Id: I48a96fbfe4bb2e3da9514d688638ecdf3a168303
    (cherry picked from commit c62d7636b6ba1c36a96119a87fff8495c5a1dc21)