StarlingX 3.x branch fails to download due to mock upgrade to python3

Bug #1884944 reported by Saul Wold
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Critical
Davlet Panech

Bug Description

The stx.3.0 branch fails to build due to the same issue that was encountered in master with the recent mock upgrade in CentOS repos. This patch [0] was applied to master but not yet backported to the stx.3.0 branch.

Severity is High due to blocking the stx.3.0 build.

Reproduce by running build-pkgs with an freshly created build Docker container.

[0] https://review.opendev.org/#/c/736840/4

Revision history for this message
Saul Wold (sgw-starlingx) wrote :
Revision history for this message
Ghada Khalil (gkhalil) wrote :

There appears to be a review posted to address this issue in stx.3.0:
https://review.opendev.org/#/c/737823/

tags: added: stx.build
Changed in starlingx:
assignee: nobody → Davlet Panech (dpanech)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tools (r/stx.3.0)

Reviewed: https://review.opendev.org/737823
Committed: https://git.openstack.org/cgit/starlingx/tools/commit/?id=8332f664612c3ade63c8ed7b373c1cef7ce249e6
Submitter: Zuul
Branch: r/stx.3.0

commit 8332f664612c3ade63c8ed7b373c1cef7ce249e6
Author: Davlet Panech <email address hidden>
Date: Thu Jun 18 17:20:15 2020 -0400

    Dockerfile: pin centos & epel repo versions

    Avoid different outcomes every time we run "docker build":

    - Use centos 7.8 base image
    - Use fixed versions of CentOS & EPEL repos
    - Pin mock & its dependencies to 1.4.16
    - Don't install unnecessary packages
    - Don't install unnecessary/conflicting repo files

    Partial-Bug: 1884944
    Change-Id: Ic9a52f373809ed66a0e61675f4d6276fd62e6fa0
    Signed-off-by: Davlet Panech <email address hidden>
    (cherry picked from commit 78263e8784c5f596af2a20b9c6e214fc6fea4878)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.opendev.org/739092
Committed: https://git.openstack.org/cgit/starlingx/tools/commit/?id=ec379155ae11b482ae59c6e5bafc75bd20720b2d
Submitter: Zuul
Branch: r/stx.3.0

commit ec379155ae11b482ae59c6e5bafc75bd20720b2d
Author: Davlet Panech <email address hidden>
Date: Mon Jun 29 10:45:31 2020 -0400

    Dockerfile: add mirror repos to global yum config

    Copy mirror repo files to /etc/yum.repos.d because tb.sh needs them.
    This change fixes a problem introduced by

      Commit: 78263e8784c5f596af2a20b9c6e214fc6fea4878
      Dockerfile: pin centos & epel repo versions

    that caused centos-mirror-tools/download_mirror.sh to fail when executed
    from a docker container created by tb.sh -- "rpm not found" for most
    packages being downloaded.

    This is because download_mirror.sh without additional arguments expects
    toCOPY/yum.repos.d/ files to have been copied to /etc/yum.repos.d/ in the
    docker container by default.

    Partial-Bug: 1884944
    Depends-On: https://review.opendev.org/#/c/737823/
    Change-Id: If214aea6560632837d0f196c21801fc2d06c8259
    Signed-off-by: Davlet Panech <email address hidden>
    (cherry picked from commit aa23b041b133c032efce238afb529e840fac58a8)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.opendev.org/739093
Committed: https://git.openstack.org/cgit/starlingx/tools/commit/?id=7509ba668c51b5cadfbdc7d4ef6d3ba46bb5d2ad
Submitter: Zuul
Branch: r/stx.3.0

commit 7509ba668c51b5cadfbdc7d4ef6d3ba46bb5d2ad
Author: Poornima <email address hidden>
Date: Fri Jun 26 03:27:02 2020 +0530

    Update the docker image tag to 7.8

    Since the base centos images now is 7.8.2003, updated the
    TC_CONTAINER_TAG variable in the tb.sh script to reflect the same.

    Partial-Bug: 1884944
    Depends-On: https://review.opendev.org/#/c/737823/
    Change-Id: I5455867660d787dc7eba3f07db11ce6c56fb0bca
    Signed-off-by: Poornima <email address hidden>
    (cherry picked from commit fb25b76817e60fbc9bd533c84b6d6ea2571b7991)
    Signed-off-by: Davlet Panech <email address hidden>

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.opendev.org/739094
Committed: https://git.openstack.org/cgit/starlingx/tools/commit/?id=4d5abdc6748256750d7d78e5a47db50cea79ec1b
Submitter: Zuul
Branch: r/stx.3.0

commit 4d5abdc6748256750d7d78e5a47db50cea79ec1b
Author: Don Penney <email address hidden>
Date: Wed May 20 14:20:35 2020 -0400

    Update 7.7.1908 repo to point to vault

    The Centos 7.7.1908 repo has been deprecated and moved to the vault.
    This commit updates the repo config file to point to the new baseurl
    locations.

    Partial-Bug: 1884944
    Depends-On: https://review.opendev.org/#/c/737823/
    Change-Id: I0d9b98eea925ad05544c7a8bd626ee5dcc48d103
    Signed-off-by: Don Penney <email address hidden>
    (cherry picked from commit d923525441f034c31f0231d15e0f55e3ac5c37b6)
    Signed-off-by: Davlet Panech <email address hidden>

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tools (r/stx.3.0)

Fix proposed to branch: r/stx.3.0
Review: https://review.opendev.org/739721

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tools (r/stx.3.0)

Reviewed: https://review.opendev.org/739721
Committed: https://git.openstack.org/cgit/starlingx/tools/commit/?id=dd9bb4313cf4dda365debce2be63c74813672b4b
Submitter: Zuul
Branch: r/stx.3.0

commit dd9bb4313cf4dda365debce2be63c74813672b4b
Author: Davlet Panech <email address hidden>
Date: Mon Jul 6 23:01:02 2020 +0000

    download_mirror.sh needs sudo to run some commands

    If download_mirror.sh is not run by root or under sudo,
    and if -n was omitted, then chown and some rm commands
    performed internally by download_mirror.sh will need sudo.

    The non-root user would need to be permitted to use those
    commands in sudoers.

    Partial-Bug: 1884944
    Change-Id: I93d4569bae23a5601dc2a37f03b1f1dde0fcc6a5
    Signed-off-by: Scott Little <email address hidden>
    (cherry picked from commit 29d71c1559866eb7cb36d201987461657a9a9418)
    Signed-off-by: Davlet Panech <email address hidden>

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to root (r/stx.3.0)

Fix proposed to branch: r/stx.3.0
Review: https://review.opendev.org/740338

Davlet Panech (dpanech)
Changed in starlingx:
status: In Progress → Fix Committed
Davlet Panech (dpanech)
Changed in starlingx:
status: Fix Committed → Fix Released
Ghada Khalil (gkhalil)
tags: added: in-r-stx30
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.