Jiří Moskovčák has posted comments on this change.

Change subject: When all filters fail, return all hosts
......................................................................


Patch Set 3: Code-Review+1

(1 comment)

....................................................
File src/ovirtscheduler/request_handler.py
Line 191:         log_adapter.debug("Aggregating results")
Line 192:         results = self.aggregate_filter_results(filterRunners, 
request_id)
Line 193:         if results is None:
Line 194:             log_adapter.info('All filters failed, return the full 
list')
Line 195:             results = hostIDs
mixing " and '
Line 196:         log_adapter.info('returning: %s' % str(results))
Line 197:         return results
Line 198: 
Line 199:     #accumalate the results


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3e88e22795d17dbd6c9dd03e7cc84da31feb483e
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-scheduler-proxy
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msi...@redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmosk...@redhat.com>
Gerrit-Reviewer: Martin Sivák <msi...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to