Change in ovirt-engine[master]: core: propagate failure messages to the user.

Code Review gerrit at ovirt.org
Tue Dec 13 13:04:37 UTC 2016


>From Moti Asayag <masayag at redhat.com>:

Moti Asayag has submitted this change and it was merged.

Change subject: core: propagate failure messages to the user.
......................................................................


core: propagate failure messages to the user.

currently these messages are only initialized but not presented to the
user in any flow, making their creation completely useless.

Alternative 'solution' would be removing creation of these messages,
as they are currently not used anywhere.

Change-Id: I81a42b2afc8f52b6c1e36e8a0d08d3f7f97edf50
Signed-off-by: Martin Mucha <mmucha at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java
1 file changed, 17 insertions(+), 9 deletions(-)

Approvals:
  Martin Sivák: Looks good to me, but someone else must approve
  Marcin Mirecki: Looks good to me, but someone else must approve
  Martin Mucha: Verified
  Jenkins CI: Passed CI tests
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I81a42b2afc8f52b6c1e36e8a0d08d3f7f97edf50
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list