Tal Nisan has submitted this change and it was merged. Change subject: backend: move illegal Pass Discard logging to AddVmCommand ...................................................................... backend: move illegal Pass Discard logging to AddVmCommand This patch moves the logging of disks with illegal Pass Discard value from AddVmFromTemplateCommand to AddVmCommand. Therefore, the log will not be shown only when creating a cloned vm from a template, but also when creating a thin vm from a template, when cloning a vm and when adding a vm from a snapshot. Change-Id: I84349befab5c9752825b0667a327edddae1d3819 Bug-Url: https://bugzilla.redhat.com/1410019 Signed-off-by: Idan Shaby <ishaby(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmFromTemplateCommand.java 2 files changed, 6 insertions(+), 5 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/69546 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I84349befab5c9752825b0667a327edddae1d3819 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com&gt; Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;