Comment 62 for bug 1966382

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-upgrade (stable/victoria)

Reviewed: https://review.opendev.org/c/openstack/tripleo-upgrade/+/835972
Committed: https://opendev.org/openstack/tripleo-upgrade/commit/6a6d0b1e6d97d8eeef8849a960d6fb65e0422f96
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 6a6d0b1e6d97d8eeef8849a960d6fb65e0422f96
Author: Sergii Golovatiuk <email address hidden>
Date: Thu Jun 24 14:50:22 2021 +0200

    Disable auto discovery& Remove deps from precommit

    This dependency breaks Centos8/Ubuntu Focal testing as these OSes use
    python 3.8.

    Also, add .pre-commit-config.yaml in irrelevant-files, we already have
    this in wallaby and above[0]

    Also, cherry-picks 13d7a4c60ed6d40b4354370f36b916caf7e21c2a to
    Disable auto discovery

    Tripleo-ci jobs are broken after latest release of setuptools 61.0
    because of breaking changes which are not backwork compatible,
    details in related bug and [1].

    Users that don't set ``packages``, ``py_modules``, or ``configuration`` are
    still likely to observe the auto-discovery behavior, which may halt the
    build if the project contains multiple directories and/or multiple Python
    files directly under the project root.

    To disable auto discovery, one can do below in setup.py

    ~~~
    setuptools.setup(..,packages=[],..)
    ~~~

    or

    ~~~
    setuptools.setup(..,py_modules=[],..)
    ~~~

    Also, adding ^setup.*$ under irrelevant-files as content-provider
    parent[2] also have same.

    [0] https://github.com/openstack/tripleo-upgrade/commit/da2dd1e548e3236940309ea400a528602f12800b
    [1] https://github.com/pypa/setuptools/issues/3197
    [2] https://opendev.org/openstack/tripleo-ci/src/branch/master/zuul.d/base-upstream.yaml#L92

    Related-Bug: #1966382
    Releated-Bug: #1967279
    Change-Id: I518be6cdba50d9bd48b3328e19cc89432f1b89b1
    (cherry picked from commit 523d09a801947685f17fc6e43fd550efcf23e832)