reinstalling quickstart-extras conflicts with quickstart venv

Bug #1955574 reported by Bogdan Dobrelya
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Sagi (Sergey) Shnaidman

Bug Description

Zuul reproducer with the static-libvirt provider requires both quickstart and extras in the same venv. For example the extras-common and snapshot/restore roles are required. The script quiickstart.sh installs those with --force options, and that creates conflicts in the config/ data files of the repos. Once quickstart config/releases and config/nodes get installed, quickstart-extras rewrites venv and removes those paths leaving only its own config/{environments, general_config}

Tags: ci quickstart
Changed in tripleo:
status: New → Triaged
tags: added: ci quickstart
Changed in tripleo:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-quickstart (master)
Changed in tripleo:
status: Triaged → In Progress
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :
Changed in tripleo:
assignee: nobody → Bogdan Dobrelya (bogdando)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-quickstart (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/tripleo-quickstart/+/822853

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-quickstart-extras (master)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-quickstart-extras (master)

Reviewed: https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/822917
Committed: https://opendev.org/openstack/tripleo-quickstart-extras/commit/fb4d5a10835c2c341140eb462634f239aa79a1c8
Submitter: "Zuul (22348)"
Branch: master

commit fb4d5a10835c2c341140eb462634f239aa79a1c8
Author: Sagi Shnaidman <email address hidden>
Date: Sat Dec 25 03:30:32 2021 +0200

    Fix conflict between tq and tqe directories

    When reinstalling tq and tqe there is conflict with config/
    directory, which is populated by both repos. Fix it by specifying
    exactly what to install in tqe.
    Related-bug: #1955574
    Change-Id: I0b1d85e83b51ef86f245f237dd72dd5306d2f79e

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-quickstart (master)

Change abandoned by "Shnaidman Sagi (Sergey) <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/tripleo-quickstart/+/822853
Reason: in favor of https://review.opendev.org/c/openstack/tripleo-quickstart/+/823168

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

Reviewed: https://review.opendev.org/c/openstack/tripleo-quickstart/+/823168
Committed: https://opendev.org/openstack/tripleo-quickstart/commit/453eefcf9f8265a1e0689e0c84cc55c687968c85
Submitter: "Zuul (22348)"
Branch: master

commit 453eefcf9f8265a1e0689e0c84cc55c687968c85
Author: Sagi Shnaidman <email address hidden>
Date: Wed Dec 29 19:03:38 2021 +0200

    Install wheel explicitly python3 venv is used

    [1] Switched to python3 native venv in CentOS 9,
    virtualenv created by python3 -m venv don't install
    wheel by default, so we would need to install that
    explicitly.

    Without wheel installer tries to do legacy install
    and with that uninstall/reinstall of a project
    may clean files owned by another project in shared
    directories.

    [1] https://review.opendev.org/c/openstack/tripleo-quickstart/+/813624

    Closes-Bug: #1955574
    Change-Id: Ie26a13a15c05778c572d1837af775566e99398e0

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-quickstart (master)

Change abandoned by "Bogdan Dobrelya <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/tripleo-quickstart/+/822724

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

Change abandoned by "Bogdan Dobrelya <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/tripleo-quickstart/+/822639
Reason: ack!

Changed in tripleo:
assignee: Bogdan Dobrelya (bogdando) → Sagi (Sergey) Shnaidman (sshnaidm)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-ci (master)

Reviewed: https://review.opendev.org/c/openstack/tripleo-ci/+/822722
Committed: https://opendev.org/openstack/tripleo-ci/commit/c882d18ce5855a62bc3b00a36b483c15bb6b3d71
Submitter: "Zuul (22348)"
Branch: master

commit c882d18ce5855a62bc3b00a36b483c15bb6b3d71
Author: Bogdan Dobrelya <email address hidden>
Date: Wed Dec 22 16:28:46 2021 +0100

    Allow skipping venv boostrap on reused zuul nodes

    There is cases, like zuul libvirt reproducer, when the venv
    may be retained on subnodes, but the job under test should be
    rechecked.

    Let an external var teardown (also defined for quickstart and
    reproducer repos) to control the venv bootstrapping.

    Related-bug: #1955574
    Change-Id: I6d9e5dc1160acf580e5f5c291b4e7a33c6a53559
    Signed-off-by: Bogdan Dobrelya <email address hidden>

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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