[openstack-dev] [nova] Need more suggestion about nova-scheduler patch/147048
john at johngarbutt.com
Mon Apr 27 08:43:11 UTC 2015
Thanks for your comments on the patch I see what you are trying to do
now, I think.
However, still a bit concerned about the unit tests, but its probably
easier to discuss that in gerrit.
On 24 April 2015 at 04:25, Rui Chen <chenrui.momo at gmail.com> wrote:
> Hi all:
> I'm working on the patch https://review.openstack.org/#/c/147048/ for
> Description of Bug:
> 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.
> But now I'm in a dilemma, because the reviewers have different point
> about implementation of the patch, so I need more feedback.
> See patch set 36
> 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.
> 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
> 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.
> I'm really really hope this patch can been merged ASAP, it has spent
> several months to review :(
> Please let me know your point, feel free to discuss it, thanks.
> Best Regards.
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
More information about the OpenStack-dev