On 5/18/2018 5:06 AM, Sylvain Bauza wrote:


Le ven. 18 mai 2018 à 13:59, Nadathur, Sundar <sundar.nadat...@intel.com <mailto:sundar.nadat...@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
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to