Arik Hadas has submitted this change and it was merged. Change subject: core: simplified refreshVdsmFileList unsuccessful call to vdsm as well as returning null are considered to be invalid return values. Taking advantage of it, if vds call is unsuccessful, it's pretended as if null was returned instead simplifying further pr ...................................................................... core: simplified refreshVdsmFileList unsuccessful call to vdsm as well as returning null are considered to be invalid return values. Taking advantage of it, if vds call is unsuccessful, it's pretended as if null was returned instead simplifying further processing. Bug-Url: https://bugzilla.redhat.com/1200952 Change-Id: Ic8b53afcd6c0e87f4f206a60735cecf9a51480c5 Signed-off-by: Martin Mucha <mmucha(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java 1 file changed, 17 insertions(+), 11 deletions(-) Approvals: Martin Mucha: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41275 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8b53afcd6c0e87f4f206a60735cecf9a51480c5 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com&gt; Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org