Change in vdsm[master]: volume: Support older engine or disks with long description

danken at redhat.com danken at redhat.com
Thu Sep 17 12:22:51 UTC 2015


Dan Kenigsberg 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
---
M vdsm/rpc/vdsmapi-schema.json
M vdsm/storage/volume.py
2 files changed, 55 insertions(+), 2 deletions(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Nir Soffer: Verified
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie3b1d9da0084aa5db02b29295d4cd0d5d8178ca2
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list