cert-manager-armada-app pylint jobs are failing

[Chuck Short]

Brief Description
-----------------

Submissions to the cert-manager-armada-app that invoke the zuul pylint job for cert-manager-armada-app are randomly failing.

The root of this problem is that the tox environment is set up for python2.7 while the zuul job runs on ubuntu-focal which does not support python2.7. only python3.

Severity
--------

Major

Steps to Reproduce
------------------

Submit a safe job to cert-manager-armada-app

Expected Behavior
-----------------

Zuul job should pass

Actual Behavior
---------------

Zuul job fails

Reproducibility
---------------

Intermittent

Last Pass
---------

April 9, 2021

Timestamps
----------

pylint create: /tmp/zuul_k8scertmanagertox/pylint
ERROR: InterpreterNotFound: python2.7
___________________________________ summary ____________________________________
ERROR: pylint: InterpreterNotFound: python2.7

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/cert-manager-armada-app/+/796715

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/starlingx/cert-manager-armada-app/+/796715
Committed: https://opendev.org/starlingx/cert-manager-armada-app/commit/8253ea8ed033078a2f4510f963b0ca2494b1ab65
Submitter: "Zuul (22348)"
Branch: master

commit 8253ea8ed033078a2f4510f963b0ca2494b1ab65
Author: Charles Short <email address hidden>
Date: Wed Jun 16 13:23:23 2021 -0400

    Fix pylint and py27 failures in zuul

    pylint running in python2.7 will complain that interperter
    can not be found.

    Rather than running the pylint job run in python2.7, bump
    the job to python3.6, and set the node to run on ubuntu-bionic.

    py2.7 jobs was failing randomly because pbr was being setup in the
    sdist step. It is unclear if the reason is that something is
    source packaged during tox which conflicts with another zuul job
    of a different interperter, so simply disable to the source
    dist generation.

    Most openstack projects are configured like this as well.

    Copyright was updated so that the gate will pick up this change.
    Closes-Bug: 1932186
    Signed-off-by: Charles Short <email address hidden>
    Change-Id: Ia677057690d3ffe87c23471d958dbff670db9b53

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: f/centos8
Review: https://review.opendev.org/c/starlingx/cert-manager-armada-app/+/796931

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: f/centos8
Review: https://review.opendev.org/c/starlingx/cert-manager-armada-app/+/796934

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Chuck Short <email address hidden>" on branch: f/centos8
Review: https://review.opendev.org/c/starlingx/cert-manager-armada-app/+/796931

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/starlingx/cert-manager-armada-app/+/796934
Committed: https://opendev.org/starlingx/cert-manager-armada-app/commit/93de08662df4ba301f57a17557826e2b4450bdda
Submitter: "Zuul (22348)"
Branch: f/centos8

commit 93de08662df4ba301f57a17557826e2b4450bdda
Author: Charles Short <email address hidden>
Date: Wed Jun 16 13:23:23 2021 -0400

    Fix pylint and py27 failures in zuul

    pylint running in python2.7 will complain that interperter
    can not be found.

    Rather than running the pylint job run in python2.7, bump
    the job to python3.6, and set the node to run on ubuntu-bionic.

    py2.7 jobs was failing randomly because pbr was being setup in the
    sdist step. It is unclear if the reason is that something is
    source packaged during tox which conflicts with another zuul job
    of a different interperter, so simply disable to the source
    dist generation.

    Most openstack projects are configured like this as well.

    Copyright was updated so that the gate will pick up this change.
    Closes-Bug: 1932186
    Signed-off-by: Charles Short <email address hidden>
    Change-Id: Ia677057690d3ffe87c23471d958dbff670db9b53
    (cherry picked from commit 8253ea8ed033078a2f4510f963b0ca2494b1ab65)

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: f/centos8
Review: https://review.opendev.org/c/starlingx/cert-manager-armada-app/+/797331

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/starlingx/cert-manager-armada-app/+/797331
Committed: https://opendev.org/starlingx/cert-manager-armada-app/commit/11945fec0578e54b6d65c1d10bcc916395427d9f
Submitter: "Zuul (22348)"
Branch: f/centos8

commit 8253ea8ed033078a2f4510f963b0ca2494b1ab65
Author: Charles Short <email address hidden>
Date: Wed Jun 16 13:23:23 2021 -0400

    Fix pylint and py27 failures in zuul

    pylint running in python2.7 will complain that interperter
    can not be found.

    Rather than running the pylint job run in python2.7, bump
    the job to python3.6, and set the node to run on ubuntu-bionic.

    py2.7 jobs was failing randomly because pbr was being setup in the
    sdist step. It is unclear if the reason is that something is
    source packaged during tox which conflicts with another zuul job
    of a different interperter, so simply disable to the source
    dist generation.

    Most openstack projects are configured like this as well.

    Copyright was updated so that the gate will pick up this change.
    Closes-Bug: 1932186
    Signed-off-by: Charles Short <email address hidden>
    Change-Id: Ia677057690d3ffe87c23471d958dbff670db9b53

commit 53982cdf03da1416f57997d1edd6934a357169ab
Author: Isac Souza <email address hidden>
Date: Mon Apr 5 09:37:07 2021 -0300

    Use new method for setting num of replicas

    Use the new _num_replicas_for_platform_app method from the
    helm base class to set the number of replicas in the chart.
    The new method will return the number of provisioned
    controllers with a minimum of 1.

    Tested by building an ISO and installing the armada apps.

    Depends-On: If322ff5d02996c9b853bc350244899c5e22431a2
    Signed-off-by: Isac Souza <email address hidden>
    Change-Id: Ia15ef324a634e846bd0ec8c6fdfcec15c486c780

commit a7838e73d0b25ee013150dff29111f808690fca6
Author: Suvro Ghosh <email address hidden>
Date: Thu Mar 11 10:42:02 2021 -0500

    Adding metadata to prevent some operations

    Critical apps like this are prevented from being deleted or
    removed.

    Story: 2007960
    Task: 42037
    Depends-On: I93821965184d95a00fddd3398a1c214e3d730efa
    Signed-off-by: Suvro Ghosh <email address hidden>
    Change-Id: I7d1930e047efb556d8ef58ca161b54b728f6d188

commit 0f530949ce708c114f08107dd55cd61316778967
Author: Suvro Ghosh <email address hidden>
Date: Tue Mar 9 10:38:19 2021 -0500

    Removing forbidden deletion temporarily

    This metadata will be added after force deletion functionality is added
    https://review.opendev.org/c/starlingx/config/+/778976

    Story: 2007960
    Task: 41757
    Signed-off-by: Suvro Ghosh <email address hidden>
    Change-Id: Iae866b5791ff2cf2b69e4fcff0e4c2376843c690

[Ghada Khalil (gkhalil)]

stx.6.0 / high - as this blocks code merges to the cert-manager-armada-app repo