Skip to content

Commit

Permalink
podman: install "rich" dependency
Browse files Browse the repository at this point in the history
This dependency was added to podman-py in version 4.8.0, but not added
properly to their requirements. Install it explicitly for our tox
and integration testing as a workaround.

[0] containers/podman-py#350

Change-Id: I61a5fdfc4e505f2577185f0c0f1297cf2709be2c
  • Loading branch information
osfrickler authored and mnasiadka committed Nov 29, 2023
1 parent 0b7d9c3 commit 0b1a59d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
3 changes: 3 additions & 0 deletions tests/templates/globals-default.j2
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@ kolla_container_engine: "{{ container_engine }}"
{% if container_engine == 'podman' %}
podman_registry: "primary:4000"
podman_registry_insecure: "yes"
podman_sdk_pip_packages:
- "podman>=4.7.0"
- rich
{% endif %}
docker_restart_policy: "no"
docker_custom_config:
Expand Down
1 change: 1 addition & 0 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ deps = -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/up
-r{toxinidir}/requirements.txt
-r{toxinidir}/test-requirements.txt
podman>=4.3.0,<5
rich
passenv = http_proxy,HTTP_PROXY,https_proxy,HTTPS_PROXY,no_proxy,NO_PROXY, \
OS_STDOUT_CAPTURE,OS_STDERR_CAPTURE,OS_LOG_CAPTURE,OS_TEST_TIMEOUT, \
PYTHON,OS_TEST_PATH,LISTOPT,IDOPTION
Expand Down

0 comments on commit 0b1a59d

Please sign in to comment.