Open Stack

On 5/18/2018 5:06 AM, Sylvain Bauza wrote:
>
>
> Le ven. 18 mai 2018 à 13:59, Nadathur, Sundar 
> <sundar.nadathur at intel.com <mailto:sundar.nadathur at intel.com>> a écrit :
>
>     Hi Matt,
>
>     On 5/17/2018 3:18 PM, Matt Riedemann wrote:
>>     On 5/17/2018 3:36 PM, Nadathur, Sundar wrote:
>>>     This applies only to the resources that Nova handles, IIUC,
>>>     which does not handle accelerators. The generic method that Alex
>>>     talks about is obviously preferable but, if that is not
>>>     available in Rocky, is the filter an option?
>>
>>     If nova isn't creating accelerator resources managed by cyborg, I
>>     have no idea why nova would be doing quota checks on those types
>>     of resources. And no, I don't think adding a scheduler filter to
>>     nova for checking accelerator quota is something we'd add either.
>>     I'm not sure that would even make sense - the quota for the
>>     resource is per tenant, not per host is it? The scheduler filters
>>     work on a per-host basis.
>     Can we not extend BaseFilter.filter_all() to get all the hosts in
>     a filter?
>     https://github.com/openstack/nova/blob/master/nova/filters.py#L36
>
>     I should have made it clearer that this putative filter will be
>     out-of-tree, and needed only till better solutions become available.
>
>
> No, there are two clear parameters for a filter, and changing that 
> would mean a new paradigm for FilterScheduler.
> If you need to have a check for all the hosts, maybe it should be 
> either a pre-filter for Placement or a post-filter but we don't accept 
> out of tree yet.

Thanks, Sylvain. So, the filter approach got filtered out.

Matt had mentioned that Cinder volume quotas are not checked by Nova 
either, citing:
      https://bugs.launchpad.net/nova/+bug/1742102
That includes this comment:
     https://bugs.launchpad.net/nova/+bug/1742102/comments/4
I'll check how Cinder does it today.

Thanks to all for your valuable input.

Regards,
Sundar
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20180518/8df00324/attachment.html>