Comment 5 for bug 1492069

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/225322
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d24633a4688b951f2b4d29b3248019588b3e198a
Submitter: Jenkins
Branch: master

commit d24633a4688b951f2b4d29b3248019588b3e198a
Author: armando-migliaccio <email address hidden>
Date: Fri Sep 18 13:13:59 2015 -0700

    Relax service module check on service providers

    service_provider is a 'special' configuration. It is a MultiStr
    option that used to be in neutron.conf, but moved into its own *-aas
    config files after the service split. We allow for the definition to
    be available 'anywhere': either in neutron.conf or in the *-aas
    service's config file.

    The list of 'service_provider' can include drivers from within the *-aas
    tree, or from elsewhere, and can apply to different service types. Due to
    the polymorphic nature of this variable it is very tricky to identify only
    the drivers that pertain a specific service module: the service module
    may as well implement more than one service type and may have support
    from drivers out of tree.

    For this reason it is best to relax this check and rely on query filters
    when the ServiceManager.get_service_providers() is invoked. Furthermore,
    without this fix there was a situation where the value returned by
    'service_providers' may be differ depending on how the configuration is
    passed on the CLI, and this inconsistency may only cause headaches.

    Closes-bug: #1492069

    Change-Id: I4db4ce0b287c984d86181227edf769531ecb7bb8