From b792d5ccbf3968b547b7e1fbd474a9ec8060be2c Mon Sep 17 00:00:00 2001 From: "Walter A. Boring IV" Date: Wed, 25 Mar 2015 09:35:51 -0700 Subject: [PATCH] Added the missing attachment to detach_volume I must have missed this driver in the multiattach patch. The attachment object is passed in to detach_volume to give drivers a chance to deal with a detach call. This patch just updates the signature of the method to be the same as the parent. Change-Id: I895c163e67d35df202a5490a7a31ee7fc13a0cb4 Closes-Bug: #1436367 --- cinder/volume/drivers/openvstorage.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cinder/volume/drivers/openvstorage.py b/cinder/volume/drivers/openvstorage.py index 7dc643d84..050ee7eeb 100644 --- a/cinder/volume/drivers/openvstorage.py +++ b/cinder/volume/drivers/openvstorage.py @@ -383,7 +383,7 @@ class OVSVolumeDriver(driver.VolumeDriver): """Callback for volume attached to instance or host.""" pass - def detach_volume(self, context, volume): + def detach_volume(self, context, volume, attachment=None): """Callback for volume detached.""" pass -- 2.45.2