Michael Kublin has submitted this change and it was merged. Change subject: engine: Simplify host life cycle - removed unneeded SetNonOperational ...................................................................... engine: Simplify host life cycle - removed unneeded SetNonOperational The following code is uselles, I don't see any reason to set host again to NonOperational when its status was already NonOperational, the condiiton described in if - it is a bug and if will occurred will be fixed accordingly Change-Id: I800beee3d38c3b1036cb82440d67ef744ac5dbae Signed-off-by: Michael Kublin <mkublin(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AsyncTaskManager.java 1 file changed, 2 insertions(+), 32 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12165 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I800beee3d38c3b1036cb82440d67ef744ac5dbae Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com&gt; Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com&gt; Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com&gt; Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com&gt; Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com&gt;