Tal Nisan has submitted this change and it was merged. Change subject: core: Don't write empty description/comment to OVF ......................................................................
core: Don't write empty description/comment to OVF Currently when we generate OVF files we write the description and comment properties of the VM even if they are null. As a result when a VM without a description or comment is re-imported it has an empty string a value (instead of null) for these properties. This patch changes the OVF writer so that it won't write the description or comment properties if they are null. Change-Id: Iffd0f7010bc32402c45882dc6087bf02a55a14d9 Bug-Url: https://bugzilla.redhat.com/1145466 Signed-off-by: Juan Hernandez <[email protected]> (cherry picked from commit cbd16554817f0373df18018bd65795412972544f) --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfWriter.java 1 file changed, 10 insertions(+), 6 deletions(-) Approvals: Juan Hernandez: Verified Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33258 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iffd0f7010bc32402c45882dc6087bf02a55a14d9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Juan Hernandez <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
