openstack
/
cinder
Code Issues Proposed changes

Merge "RemoteFS: Use dest vol id instead of source id in snapshot temp name"

This commit is contained in:
Zuul 2019-11-15 23:27:37 +00:00 committed by Gerrit Code Review
parent 53ab7749a3 f3ed9d436b
commit b20561141b
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/tests/unit/volume/drivers/test_remotefs.py
View File

@ -644,7 +644,7 @@ class RemoteFsSnapDriverTestCase(test.TestCase):
'status': fields.SnapshotStatus.CREATING, 'status': fields.SnapshotStatus.CREATING,
'progress': '0%', 'progress': '0%',
'volume_size': src_vref.size, 'volume_size': src_vref.size,
'display_name': 'tmp-snap-%s' % src_vref['id'], 'display_name': 'tmp-snap-%s' % volume.id,
'display_description': None, 'display_description': None,
'volume_type_id': src_vref.volume_type_id, 'volume_type_id': src_vref.volume_type_id,
'encryption_key_id': None, 'encryption_key_id': None,

2
cinder/volume/drivers/remotefs.py
View File

@ -1058,7 +1058,7 @@ class RemoteFSSnapDriverBase(RemoteFSDriver):
'status': fields.SnapshotStatus.CREATING, 'status': fields.SnapshotStatus.CREATING,
'progress': '0%', 'progress': '0%',
'volume_size': src_vref.size, 'volume_size': src_vref.size,
'display_name': 'tmp-snap-%s' % src_vref.id, 'display_name': 'tmp-snap-%s' % volume.id,
'display_description': None, 'display_description': None,
'volume_type_id': src_vref.volume_type_id, 'volume_type_id': src_vref.volume_type_id,
'encryption_key_id': src_vref.encryption_key_id, 'encryption_key_id': src_vref.encryption_key_id,