ensure-virtualenv: Don't support on CentOS 9-stream
It looks like 9-stream don't want to support virtualenv. Fail the ensure-virtualenv role on this platform, and account for this in testing. People should use ensure-pip and venv which is portable everywhere. Change-Id: Ifae93c1eeb96792aa26a624574d595d77cb58c4b
This commit is contained in:
parent
ab4042e60a
commit
239ace0ebf
@ -1,6 +1,16 @@
|
|||||||
|
# It looks like CentOS 9 has no plans to support this,
|
||||||
|
# and we should be using "python3 -m venv". [1]
|
||||||
|
#
|
||||||
|
# ensure-pip already exports 'ensure_pip_virtualenv_command'
|
||||||
|
#
|
||||||
|
# [1] https://lists.centos.org/pipermail/centos-devel/2021-August/098470.html
|
||||||
|
- name: Fail on CentOS 9
|
||||||
|
fail:
|
||||||
|
msg: 'This element not supported on CentOS >= 9'
|
||||||
|
when: (ansible_facts['distribution'] == 'CentOS' and ansible_facts['distribution_major_version']|int >= 9)
|
||||||
|
|
||||||
- name: Install virtualenv
|
- name: Install virtualenv
|
||||||
package:
|
package:
|
||||||
name:
|
name:
|
||||||
- python3-virtualenv
|
- python3-virtualenv
|
||||||
become: yes
|
become: yes
|
||||||
|
|
||||||
|
@ -36,25 +36,29 @@
|
|||||||
$run_pip wheel --no-deps .
|
$run_pip wheel --no-deps .
|
||||||
ls zuul-*.whl || exit 1
|
ls zuul-*.whl || exit 1
|
||||||
|
|
||||||
# ensure-virtualenv
|
- name: Test virtualenv
|
||||||
- name: Include ensure-virtualenv
|
# NOTE(ianw) 2022-02-03 : not supported on 9-stream, see inline comments
|
||||||
include_role:
|
when: not (ansible_facts['distribution'] == 'CentOS' and ansible_facts['distribution_major_version']|int >= 9)
|
||||||
name: ensure-virtualenv
|
block:
|
||||||
|
# ensure-virtualenv
|
||||||
|
- name: Include ensure-virtualenv
|
||||||
|
include_role:
|
||||||
|
name: ensure-virtualenv
|
||||||
|
|
||||||
- name: Sanity check virtualenv command works
|
- name: Sanity check virtualenv command works
|
||||||
shell: |
|
shell: |
|
||||||
tmp_venv=$(mktemp -d -t venv-XXXXXXXXXX)
|
tmp_venv=$(mktemp -d -t venv-XXXXXXXXXX)
|
||||||
trap "rm -rf $tmp_venv" EXIT
|
trap "rm -rf $tmp_venv" EXIT
|
||||||
virtualenv $tmp_venv
|
virtualenv $tmp_venv
|
||||||
$tmp_venv/bin/pip install tox
|
$tmp_venv/bin/pip install tox
|
||||||
failed_when: false
|
failed_when: false
|
||||||
register: _virtualenv_sanity
|
register: _virtualenv_sanity
|
||||||
|
|
||||||
- name: Assert sanity check
|
- name: Assert sanity check
|
||||||
fail:
|
fail:
|
||||||
msg: 'The virtualenv command does not appear to work!'
|
msg: 'The virtualenv command does not appear to work!'
|
||||||
when:
|
when:
|
||||||
- _virtualenv_sanity.rc != 0
|
- _virtualenv_sanity.rc != 0
|
||||||
|
|
||||||
# NOTE(ianw) : this does not play nicely with pip-and-virtualenv which
|
# NOTE(ianw) : this does not play nicely with pip-and-virtualenv which
|
||||||
# has already installed from source. We might be able to test this
|
# has already installed from source. We might be able to test this
|
||||||
|
Loading…
Reference in New Issue
Block a user