LeastCostScheduler.weigh_hosts() doesn't return enough hosts

Hm. Wonder if this should be fixed in AbstractScheduler's schedule_run_instance()... vs what was changed in Rev 1534 for BaseScheduler. The fix for 1534 doesn't give us much, because most subclasses from Base will override weigh_hosts (like LeastCost does).

Sandy was of the opinion that this should be addressed in the various weigh_hosts() functions. His basic thesis was that weigh_hosts() in classes other than BaseScheduler are responsible for keeping an updated picture of the available resources for a given host. It wouldn't make sense to get 5 cases of a specific host if that host could only handle having 3 instances.

I have a fix underway for this: https://github.com/SandyWalsh/nova/tree/flatten_scheduler

Hope to be attempting a merge shortly.

@Sandy: or not shortly ?

I think this resolved itself. I'll test it shortly.

@cbehrens: or not shortly ? ;)

Looks like you're not working on that anymore. Please set back to InProgress and reassign to you if you're working on proposing a change for merging.

Marked this as Invalid because the code has changed _A LOT_ since this report. If you look at the current code and this bug report, it doesn't seem like they're talking about the same code base. :-)