Browse Source

Set right attach mode after migration

When migrating attached volumes we are assuming they are always in "rw" mode,
which means that a "ro" attached volume will end up as "rw" after a
migration.

Closes-Bug: #1802155
Change-Id: I374df5d294587e0e91a10525b08ea1a5911c35a1
changes/71/616271/1
Gorka Eguileor 3 years ago
parent
commit
9f95ae9dc2
  1. 2
      cinder/tests/unit/volume/test_volume_migration.py
  2. 2
      cinder/volume/manager.py

2
cinder/tests/unit/volume/test_volume_migration.py

@ -642,7 +642,7 @@ class VolumeMigrationTestCase(base.BaseVolumeTestCase):
attachment['instance_uuid'],
attachment['attached_host'],
attachment['mountpoint'],
'rw'
attachment.get('attach_mode', 'rw'),
)
self.assertIsNotNone(attachments)
self.assertEqual(attached_host,

2
cinder/volume/manager.py

@ -2319,7 +2319,7 @@ class VolumeManager(manager.CleanableManager,
attachment.instance_uuid,
attachment.attached_host,
attachment.mountpoint,
'rw')
attachment.attach_mode or 'rw')
# At this point we now have done almost all of our swapping and
# state-changes. The target volume is now marked back to
# "in-use" the destination/worker volume is now in deleting

Loading…
Cancel
Save