Change in ovirt-engine[ovirt-engine-4.1]: core: ImagesHandler - computeCowImageNeededSize()

Code Review gerrit at ovirt.org
Mon Apr 3 20:54:23 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: core: ImagesHandler - computeCowImageNeededSize()
......................................................................


core: ImagesHandler - computeCowImageNeededSize()

When vdsm creates a COW volume with provided initial size the size is
multiplied by 1.1 to prevent a case in which we won't have enough space.
If the source is already COW we don't need the additional space.

determineImageInitialSize() is used to calculate the initial size
and in case that source/dest are both COW divides the space by 1.1 as we
know that the source used space already includes the COW header.

in change I9358478d9f the check for the source image format was
mistakenly removed, leading to that even if the source image format is
RAW we'll divide the initial size by 1.1. As some of the space is used
by the QCOW header we may fail during the copy as we run out of space.

Change-Id: I54923ca1526c39ee793270c28ace6402a558b04a
Bug-Url: https://bugzilla.redhat.com/1438260
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/ImagesHandler.java
1 file changed, 10 insertions(+), 7 deletions(-)

Approvals:
  Tal Nisan: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/75097
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I54923ca1526c39ee793270c28ace6402a558b04a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>


More information about the Engine-commits mailing list