automat...@ovirt.org has posted comments on this change.

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
......................................................................


Patch Set 3:

* Update tracker::#1200952::OK
* Check Bug-Url::OK
* Check Public Bug::#1200952::OK, public bug
* Check Product::#1200952::OK, Correct product Red Hat Enterprise 
Virtualization Manager
* Check TR::SKIP, not in a monitored branch (ovirt-engine-3.5 ovirt-engine-3.4 
ovirt-engine-3.3 engine_3.2 engine_3.1 ovirt-engine-3.5.2 ovirt-engine-3.4.0 
ovirt-engine-3.3.4 ovirt-engine-3.3.3 ovirt-engine-3.3.2 ovirt-engine-3.3.1)
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-engine-3.5', 
'ovirt-engine-3.4', 'ovirt-engine-3.3', 'ovirt-engine-3.5.2', 
'ovirt-engine-3.4.0', 'ovirt-engine-3.3.4', 'ovirt-engine-3.3.3', 
'ovirt-engine-3.3.2', 'ovirt-engine-3.3.1'])

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic8b53afcd6c0e87f4f206a60735cecf9a51480c5
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Mucha <mmu...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmu...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to