Comment 37 for bug 1743688

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to devstack (master)

Reviewed: https://review.opendev.org/619973
Committed: https://git.openstack.org/cgit/openstack/devstack/commit/?id=f0dd9996cc635709276f51e4d94c5ebbf5d5b49f
Submitter: Zuul
Branch: master

commit f0dd9996cc635709276f51e4d94c5ebbf5d5b49f
Author: ghanshyam <email address hidden>
Date: Mon Nov 26 07:38:54 2018 +0000

    Fix DevStack to configure tempest's service_availability

    Tempest's service_availability config option includes all the service
    availability which is further used by tests to take decision of skip
    or run the test.

    For example, [service_availability].nova is true then, compute test will run
    or if [service_availability].aodh is false then, all aodh related tests either
    in aodh tempest plugin or any other plugins will be skipped.

    Now question is what is the best way to set the each service availability for
    tempest or tempest plugins tests. We have 2 category of service here-
    1. Service tested by Tempest (nova, cinder, keystone, glance, swift, neutron)
       (let's say type1 service)
    2. Services tested by Tempest plugins (all other than above list)
       (let's say type2 service)

    We need the standard way to set both type of service so that we can maintain
    the setting of service_availability config options in consistent way.

    As discussed on bug#1743688/ and review https://review.openstack.org/#/c/536723/,
    we will use devstack lib/tempest to set the type1 service which is services test
    owned by Tempest and type2 service setting will be done by devstack plugins of
    those service.

    For example - [service_availability].ironic will be set by ironic's devstack plugin.
    because that is best place we know ironic is installed and available.

    To do that we need:
    1. Add setting of [service_availability].* in devstack plugins
    2. Remove setting of type2 service from devstack lib/tempest

    This commit does the second part and all depends-on patches handle the first part.

    Related-Bug: #1743688

    Change-Id: If3aec9fd1c61e2bb53233be437b97b811dc82414