]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Add some useful log to filter scheduler.
authorJohn Griffith <john.griffith@solidfire.com>
Thu, 28 Feb 2013 18:34:07 +0000 (18:34 +0000)
committerJohn Griffith <john.griffith@solidfire.com>
Fri, 1 Mar 2013 18:56:58 +0000 (18:56 +0000)
commit0c2ce65014be15430b37df83e12d970052a028b4
tree96aaf598581c7603990c5141bbfe8d46f8a30f12
parentdd010690330e8304eadddab158ba5c2e3f57d8c8
Add some useful log to filter scheduler.

When the filter scheduler is unable to find a suitable host to
deploy a volume on it simply logged a warning "not hosts found".
This isn't very helpful, so this patch adds some logging info as
far as why no valid host was available.  This just focuses on the
capacity filtering as that's where I've commonly seen this issue
particularly in tempest runs.

Also noticed that the get status was checking for "if Not capacity",
but this is wrong because free capacity==0 would be interpretted as
failing to get capacity.  Change this to "if is None".

Fixes bug: 1136147

Change-Id: Ia1ce508826ee69ae68a250ea68616909c59974e5
cinder/scheduler/filters/capacity_filter.py
cinder/scheduler/host_manager.py
cinder/scheduler/manager.py
cinder/tests/scheduler/test_host_manager.py