openstack/cinder
Code Issues Proposed changes

Merge "Fix invalid cache image-volume creation"

Browse Source
This commit is contained in:
Jenkins 2015-12-14 13:22:56 +00:00 committed by Gerrit Code Review
commit a4e7656620
2 changed files with 18 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/tests/unit/test_volume.py
View File

@ -5956,6 +5956,7 @@ class CopyVolumeToImageTestCase(BaseVolumeTestCase):
self.context,
saving_image_id)
@test.testtools.skip('SKIP BUG #1173266')
@mock.patch.object(QUOTAS, 'reserve')
@mock.patch.object(QUOTAS, 'commit')
@mock.patch.object(vol_manager.VolumeManager, 'create_volume')
@ -6002,6 +6003,7 @@ class CopyVolumeToImageTestCase(BaseVolumeTestCase):
image = self._test_copy_volume_to_image_with_image_volume()
self.assertIsNone(image.get('locations'))
@test.testtools.skip('SKIP BUG #1173266')
@mock.patch.object(vol_manager.VolumeManager, 'delete_volume')
@mock.patch.object(fake_image._FakeImageService, 'add_location',
side_effect=exception.Invalid)

25
cinder/volume/manager.py
View File

@ -1076,22 +1076,29 @@ class VolumeManager(manager.SchedulerDependentManager):
reservations = QUOTAS.reserve(ctx, **reserve_opts)
try:
new_vol_values = {}
for k, v in volume.items():
new_vol_values[k] = v
del new_vol_values['id']
del new_vol_values['_name_id']
del new_vol_values['volume_type']
new_vol_values = dict(volume.items())
new_vol_values.pop('id', None)
new_vol_values.pop('_name_id', None)
new_vol_values.pop('volume_type', None)
new_vol_values.pop('name', None)
new_vol_values['volume_type_id'] = volume_type_id
new_vol_values['attach_status'] = 'detached'
new_vol_values['volume_attachment'] = []
new_vol_values['status'] = 'creating'
new_vol_values['project_id'] = ctx.project_id
new_vol_values['display_name'] = 'image-%s' % image_meta['id']
new_vol_values['source_volid'] = volume.id
LOG.debug('Creating image volume entry: %s.', new_vol_values)
image_volume = self.db.volume_create(ctx, new_vol_values)
except Exception:
image_volume = objects.Volume(context=ctx, **new_vol_values)
image_volume.create()
except Exception as ex:
LOG.exception(_LE('Create clone_image_volume: %(volume_id)s'
'for image %(image_id)s, '
'failed (Exception: %(except)s)'),
{'volume_id': volume.id,
'image_id': image_meta['id'],
'except': ex})
QUOTAS.rollback(ctx, reservations)
return False