Change in vdsm[master]: BZ#767111 Extend volume size must be in Megabytes

danken at redhat.com danken at redhat.com
Wed Dec 14 10:44:38 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#767111 Extend volume size must be in Megabytes
......................................................................


BZ#767111 Extend volume size must be in Megabytes

In commit c24396af (Merged SPM to StoragePool and HSM) the
spm.public_extendVolume call was replaced by pool.extendVolume.
The problem is that they use different units for the size
parameter, the first one uses bytes and the second one uses
megabytes.

In this patch:
* do not convert megabytes to bytes when calling pool.extendVolume
* improve logging to record also the size received from the mailbox

Change-Id: I955da5ca52708eb9bfd51df66f2a0e13e8eaf3c8
---
M vdsm/storage/storage_mailbox.py
1 file changed, 5 insertions(+), 5 deletions(-)

Approvals:
  Federico Simoncelli: Verified
  Dan Kenigsberg: Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/579
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I955da5ca52708eb9bfd51df66f2a0e13e8eaf3c8
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list