diff options
| author | Chris Yeoh <cyeoh@au1.ibm.com> | 2013-04-22 10:51:00 +0930 |
|---|---|---|
| committer | Chris Yeoh <cyeoh@au1.ibm.com> | 2013-04-22 10:51:00 +0930 |
| commit | fd6fa90d5f293be7fac4f1e612958cc8e55ee796 (patch) | |
| tree | 0c194fd2866d9db89eb75f8f8a0fa14612c8b133 /nova/tests | |
| parent | c62d7636b6ba1c36a96119a87fff8495c5a1dc21 (diff) | |
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
Diffstat (limited to 'nova/tests')
| -rw-r--r-- | nova/tests/api/ec2/test_cinder_cloud.py | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/nova/tests/api/ec2/test_cinder_cloud.py b/nova/tests/api/ec2/test_cinder_cloud.py index 1751343ce..605f903cb 100644 --- a/nova/tests/api/ec2/test_cinder_cloud.py +++ b/nova/tests/api/ec2/test_cinder_cloud.py @@ -189,6 +189,35 @@ class CinderCloudTestCase(test.TestCase): self.cloud.delete_volume(self.context, vol1['volumeId']) self.cloud.delete_volume(self.context, vol2['volumeId']) + def test_format_volume_maps_status(self): + fake_volume = {'id': 1, + 'status': 'creating', + 'availability_zone': 'nova', + 'volumeId': 'vol-0000000a', + 'attachmentSet': [{}], + 'snapshotId': None, + 'created_at': '2013-04-18T06:03:35.025626', + 'size': 1, + 'mountpoint': None, + 'attach_status': None} + + self.assertEqual(self.cloud._format_volume(self.context, + fake_volume)['status'], + 'creating') + + fake_volume['status'] = 'attaching' + self.assertEqual(self.cloud._format_volume(self.context, + fake_volume)['status'], + 'in-use') + fake_volume['status'] = 'detaching' + self.assertEqual(self.cloud._format_volume(self.context, + fake_volume)['status'], + 'in-use') + fake_volume['status'] = 'banana' + self.assertEqual(self.cloud._format_volume(self.context, + fake_volume)['status'], + 'banana') + def test_create_volume_in_availability_zone(self): """Makes sure create_volume works when we specify an availability zone |