Comment 6 for bug 1848786

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

Reviewed: https://review.opendev.org/690131
Committed: https://git.openstack.org/cgit/starlingx/config/commit/?id=b2881d90df736d7423bbecb237b1702891678bff
Submitter: Zuul
Branch: master

commit b2881d90df736d7423bbecb237b1702891678bff
Author: Al Bailey <email address hidden>
Date: Tue Oct 22 11:39:22 2019 -0500

    Fix tox in other repos that import sysinv

    sysinv.common.utils is imported in cgcs-patch.

    When keyring was added to that file, the tox and
    zuul jobs in cgcs-patch failed.

    The root reason for this is that when those other repos
    add sysinv as a dependency, only the requirements.txt is
    used when constructing the egg for import by tox.

    By properly relocating runtime required files from
    test-requirements.txt to requirements.txt, those other
    repos are able to import the additional required components.

    This does not impact rpms since those are driven by the
    Requires directives in the spec files.

    Change-Id: I8d9d84345a0810e311d2a50cdcbdb49056f80c2c
    Fixes-Bug: 1848786
    Signed-off-by: Al Bailey <email address hidden>