Add necessary fields to volume creation
This patch is to add the two fields in driver.py when creating volumes. Without the fix, it leads query related to the volume failed with exception. Change-Id: I0055b5bf402b0aa7b4ee78b0fb40bedea850af94 Closes-Bug: #1549673 Related-Bug: #1549669
This commit is contained in:
parent
14d7b1448b
commit
231f856ece
|
@ -6516,6 +6516,23 @@ class GenericVolumeDriverTestCase(DriverTestCase):
|
|||
backup_obj = objects.Backup.get_by_id(self.context, backup.id)
|
||||
self.assertEqual(temp_vol.id, backup_obj.temp_volume_id)
|
||||
|
||||
def test__create_temp_volume_from_snapshot(self):
|
||||
volume_dict = {'id': 'fake',
|
||||
'host': 'fakehost',
|
||||
'availability_zone': 'fakezone',
|
||||
'size': 1}
|
||||
vol = fake_volume.fake_volume_obj(self.context, **volume_dict)
|
||||
snapshot = fake_snapshot.fake_snapshot_obj(self.context)
|
||||
|
||||
with mock.patch.object(
|
||||
self.volume.driver,
|
||||
'create_volume_from_snapshot'):
|
||||
temp_vol = self.volume.driver._create_temp_volume_from_snapshot(
|
||||
self.context,
|
||||
vol, snapshot)
|
||||
self.assertEqual('detached', temp_vol['attach_status'])
|
||||
self.assertEqual('fakezone', temp_vol['availability_zone'])
|
||||
|
||||
@mock.patch.object(utils, 'brick_get_connector_properties')
|
||||
@mock.patch.object(cinder.volume.driver.VolumeDriver, '_attach_volume')
|
||||
@mock.patch.object(cinder.volume.driver.VolumeDriver, '_detach_volume')
|
||||
|
|
|
@ -1423,6 +1423,8 @@ class BaseVD(object):
|
|||
'user_id': context.user_id,
|
||||
'project_id': context.project_id,
|
||||
'status': 'creating',
|
||||
'attach_status': 'detached',
|
||||
'availability_zone': volume.availability_zone,
|
||||
}
|
||||
temp_vol_ref = self.db.volume_create(context, temp_volume)
|
||||
try:
|
||||
|
|
Loading…
Reference in New Issue