Change in ovirt-engine[master]: core: handle error when iso domain in maintenance

mlipchuk at redhat.com mlipchuk at redhat.com
Sun Feb 3 13:22:01 UTC 2013


Maor Lipchuk has submitted this change and it was merged.

Change subject: core: handle error when iso domain in maintenance
......................................................................


core: handle error when iso domain in maintenance

The bug started from code inserted in change I9d24c94c.
Currently, when iso domain cannot be accessed for any reason, it's considered a
problematic domain and is mentioned in the event log.
This patch adds a check that if the domain is not active,
it is not attempted to be refreshed, hence not adding in event log
the error (because it's expected it cannot be accessed)

Change-Id: Ide53f42c087ffbc9240cccc033e809ff8117ec8c
Bug-Url: https://bugzilla.redhat.com/902246
Signed-off-by: Alissa Bonas <abonas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java
1 file changed, 26 insertions(+), 25 deletions(-)

Approvals:
  Maor Lipchuk: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/11449
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ide53f42c087ffbc9240cccc033e809ff8117ec8c
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>



More information about the Engine-commits mailing list