Change in ovirt-engine[master]: core: Propagate UpdateVm failure to UpdateClusterCommand

Code Review gerrit at ovirt.org
Wed Nov 9 17:01:58 UTC 2016


>From Arik Hadas <ahadas at redhat.com>:

Arik Hadas has submitted this change and it was merged.

Change subject: core: Propagate UpdateVm failure to UpdateClusterCommand
......................................................................


core: Propagate UpdateVm failure to UpdateClusterCommand

When changing cluster compatibility version with UpdateClusterCommand,
UpdateVmCommand is called for every VM in the cluster. If
UpdateVmCommand fails, failure details are not propagated to
UpdateClusterCommand and the failure is displayed to user as generic
"Internal error".

The patch fixes this issue.

Change-Id: Ib68f691db5027106c582ed6ecd2d53c7d82e3454
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1373573
Signed-off-by: Shmuel Melamud <smelamud at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateClusterCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins CI: Passed CI tests
  Shmuel Leib Melamud: Verified
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib68f691db5027106c582ed6ecd2d53c7d82e3454
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list