Change in vdsm[master]: Too many True sizes for a False Volume class.

danken at redhat.com danken at redhat.com
Mon Aug 5 10:52:29 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Too many True sizes for a False Volume class.
......................................................................


Too many True sizes for a False Volume class.

Since an (abstract) volume never will be instantiated alone, get
the size of the typed object instead.

Removes Volume.getVSize(), Volume.getVTrueSize() and
Volume.getVolumeTrueSize().

Reduce the number of SD produces.

Related to BZ#960952, BZ#769502.

Change-Id: I98c82bf2b48c62626b496fe16e4b83f48c189c10
Signed-off-by: Eduardo <ewarszaw at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/17509
Tested-by: Elad Ben Aharon <eladba1990 at gmail.com>
Reviewed-by: Ayal Baron <abaron at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/storage/hsm.py
M vdsm/storage/volume.py
2 files changed, 4 insertions(+), 27 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, approved
  Elad Ben Aharon: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I98c82bf2b48c62626b496fe16e4b83f48c189c10
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Elad Ben Aharon <eladba1990 at gmail.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>


More information about the vdsm-patches mailing list