Change in ovirt-engine[ovirt-engine-4.1]: backend: move illegal Pass Discard logging to AddVmCommand

Code Review gerrit at ovirt.org
Tue Jan 10 12:47:04 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

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 at redhat.com>
---
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/69888
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I84349befab5c9752825b0667a327edddae1d3819
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list