From 01ff9d7dc7af54eaa218ef86839b0468de283c7e Mon Sep 17 00:00:00 2001 From: Patrick East Date: Wed, 2 Sep 2015 13:51:42 -0700 Subject: [PATCH] =?utf8?q?Get=20full=20volume=20model=20in=20Replication?= =?utf8?q?=20manager=20API=E2=80=99s?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Currently we pass in a dictionary of the volume properties, but not the actual volume model. This dictionary is missing some of the properties like ‘name’ which backends may required for volume operations. Change-Id: Idcb7b2c8313360a77ba336f3b7534d8fa7431a41 Closes-Bug: #1491585 --- cinder/volume/manager.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/cinder/volume/manager.py b/cinder/volume/manager.py index 293777b0e..8b413ae7a 100644 --- a/cinder/volume/manager.py +++ b/cinder/volume/manager.py @@ -3058,6 +3058,7 @@ class VolumeManager(manager.SchedulerDependentManager): :param secondary: Specifies rep target to fail over to """ try: + volume = self.db.volume_get(context, volume['id']) volume_updates = self.driver.replication_failover(context, volume, secondary) @@ -3153,6 +3154,7 @@ class VolumeManager(manager.SchedulerDependentManager): """ try: + volume = self.db.volume_get(context, volume['id']) replication_targets = self.driver.list_replication_targets(context, volume) -- 2.45.2