From: John Griffith <john.griffith@solidfire.com>
Date: Wed, 16 Oct 2013 01:13:45 +0000 (-0600)
Subject: Fix lvm.extend_volume to pass Gig suffix
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=ec442e41d2d243003a42ed60ae862e96142a5cde;p=openstack-build%2Fcinder-build.git

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
---

diff --git a/cinder/volume/drivers/lvm.py b/cinder/volume/drivers/lvm.py
index 3b48840c3..0ea6ba5fb 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):