Merge lp://qastaging/~cbehrens/nova/lp844155 into lp://qastaging/~hudson-openstack/nova/trunk
Proposed by
Chris Behrens
Status: | Merged |
---|---|
Approved by: | Josh Kearney |
Approved revision: | 1535 |
Merged at revision: | 1536 |
Proposed branch: | lp://qastaging/~cbehrens/nova/lp844155 |
Merge into: | lp://qastaging/~hudson-openstack/nova/trunk |
Diff against target: |
46 lines (+22/-3) 2 files modified
nova/scheduler/abstract_scheduler.py (+0/-3) nova/tests/scheduler/test_abstract_scheduler.py (+22/-0) |
To merge this branch: | bzr merge lp://qastaging/~cbehrens/nova/lp844155 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Josh Kearney (community) | Approve | ||
Sandy Walsh (community) | Approve | ||
Review via email:
|
Description of the change
Fix lp:844155
When using an abstract scheduler class, if no compute hosts are available locally, the checking of child zones is short circuited. I've removed the broken check.
To post a comment you must log in.
makes sense