Change in vdsm[ovirt-3.6]: volume: Support older engine or disks with long description

fromani at redhat.com fromani at redhat.com
Tue Sep 22 15:04:31 UTC 2015


Francesco Romani has submitted this change and it was merged.

Change subject: volume: Support older engine or disks with long description
......................................................................


volume: Support older engine or disks with long description

New engine will limit the description field size to avoid the
MetaDataOverflowError, but we must support older engine versions and
existing disks with long description. For example, copying old disk with
long description from file to block storage.

The description field has now maximum size; if the size exceeds the
limit, it is transparently truncated when creating any volume type.

Change-Id: Ie3b1d9da0084aa5db02b29295d4cd0d5d8178ca2
Backport-To: 3.6
Bug-Url: https://bugzilla.redhat.com/1258097
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/45501
Reviewed-by: Adam Litke <alitke at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-on: https://gerrit.ovirt.org/46330
Reviewed-by: Francesco Romani <fromani at redhat.com>
---
M vdsm/rpc/vdsmapi-schema.json
M vdsm/storage/volume.py
2 files changed, 55 insertions(+), 2 deletions(-)

Approvals:
  Nir Soffer: Verified
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie3b1d9da0084aa5db02b29295d4cd0d5d8178ca2
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list