From bdb5d982d069ac6464c8ef857a0f95dde02005de Mon Sep 17 00:00:00 2001 From: John Griffith Date: Tue, 15 Oct 2013 19:13:45 -0600 Subject: [PATCH] Fix lvm.extend_volume to pass Gig suffix The extend function in the lvm driver was not converting the cinder size value to Gigabytes before passing the call to the vg module. The result was that we would attempt to extend a volume to "new size in Megabytes" which of course is less than the current size since we do a Gigabyte string conversion on create and everywhere else. This change makes sure we pass the integer change through the sizestr method to get the G suffix needed to work properly. Change-Id: I070962a3aa7038f612e19a93ccaa60cbc13008f6 Closes-Bug: #1240287 (cherry picked from commit ec442e41d2d243003a42ed60ae862e96142a5cde) --- cinder/volume/drivers/lvm.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cinder/volume/drivers/lvm.py b/cinder/volume/drivers/lvm.py index 862fe238f..1b37cbf8f 100644 --- a/cinder/volume/drivers/lvm.py +++ b/cinder/volume/drivers/lvm.py @@ -380,7 +380,8 @@ class LVMVolumeDriver(driver.VolumeDriver): def extend_volume(self, volume, new_size): """Extend an existing voumes size.""" - self.vg.extend_volume(volume['name'], new_size) + self.vg.extend_volume(volume['name'], + self._sizestr(new_size)) class LVMISCSIDriver(LVMVolumeDriver, driver.ISCSIDriver): -- 2.45.2