<div dir="ltr">Hi all:<div><br></div><div>    I'm working on the patch <a href="https://review.openstack.org/#/c/147048/">https://review.openstack.org/#/c/147048/</a> for bug/1408859</div><div><br></div><div>    Description of Bug:</div><div>    When the nova-scheduler can't select enough hosts for multiple creating instance, a NoValidHost exception was raised, but the part of hosts had been consumed from instance in the _schedule loop, the resource of consumed hosts was not reverted, it would result in the resource lacking in nova-scheduler.</div><div><br></div><div>    But now I'm in a dilemma, because the reviewers have different point about implementation of the patch, so I need more feedback.</div><div><br></div><div>    See patch set 36</div><div>    This is simple way, just set the 'updated' attribute of consumed host as None, and make use of the 'update_from_compute_node' logic to update the local HostState at the next scheduling request.</div><div><br></div><div><a href="http://git.openstack.org/cgit/openstack/nova/tree/nova/scheduler/host_manager.py#n185">http://git.openstack.org/cgit/openstack/nova/tree/nova/scheduler/host_manager.py#n185</a><br></div><div><br></div><div>    But like John say, it would break CachingScheduler in a refresh cycle, the resource of HostState would been fixed until the next periodic_tasks is executed.</div><div><br></div><div>    Other side, Alex and Sylvain think that it should be acceptable, because CachingSheduler use the out of date HostState in the cache according to the design, and the usage limitation had been described in class notes.</div><div><br></div><div>    I'm really really hope this patch can been merged ASAP, it has spent several months to review :(</div><div><br></div><div>    Please let me know your point, feel free to discuss it, thanks.</div><div><br></div><div>Best Regards.</div></div>