Change in ovirt-engine[master]: core: simplified refreshVdsmFileList unsuccessful call to vd...

[ahadas at redhat.com]

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 at redhat.com>
---
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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: automation at ovirt.org