Tal Nisan has submitted this change and it was merged. Change subject: core: limit disk alias and description for disk metadata ...................................................................... core: limit disk alias and description for disk metadata On block storage, the disk metadata is limited to 512 bytes. Since we store its alias and description in the metadata, we should limit their size or else they will be truncated by vdsm and we won't be able to read them. This patch limits the value of the metadata description field size to a maximum of 210 bytes. We do it also for file storage in order to prevent a truncation of the description field when moving a disk from file to block storage. Change-Id: I74d04291bbaacb5e642d8658ab752b1a8d32c0ac Bug-Url: https://bugzilla.redhat.com/1258547 Related-To: https://bugzilla.redhat.com/1249130 Related-To: https://bugzilla.redhat.com/1261387 Signed-off-by: Idan Shaby <ishaby@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddImageFromScratchCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MetadataDiskDescriptionHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/MetadataDiskDescriptionHandlerTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 6 files changed, 211 insertions(+), 18 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/45925 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74d04291bbaacb5e642d8658ab752b1a8d32c0ac Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Idan Shaby <ishaby@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: automation@ovirt.org