Skip to content

Commit

Permalink
Remove workaround for podman-py #350
Browse files Browse the repository at this point in the history
Remove manual installation of python-rich dependency, which was required
due to issue containers/podman-py#350 in
podman-py 4.8 and which should be fixed in podman-py 4.9

Signed-off-by: Martin Perina <[email protected]>
  • Loading branch information
mwperina committed Mar 11, 2024
1 parent 7ba0a40 commit 9abb3ca
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/integration-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,6 @@ jobs:
- name: Install unpackaged python libraries from PyPi
run: |
pip install "tmt[provision]==1.30" "tmt[report-junit]==1.30" podman pytest pyyaml paramiko
# Mitigate https://github.com/containers/podman-py/issues/350
pip install rich
- name: Checkout sources
uses: actions/checkout@v4
Expand Down
6 changes: 1 addition & 5 deletions tests/scripts/tests-setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,13 +14,11 @@ function install_worker_deps(){
python3-pytest \
python3-pyyaml \
-y
# Mitigate https://github.com/containers/podman-py/issues/350
dnf install python3-rich -y
}

function setup_worker_ssh(){
dnf install openssh-server passwd -y

# root access is needed in order to access BlueChi's API
echo "PasswordAuthentication yes" >> /etc/ssh/sshd_config
echo "PermitRootLogin yes" >> /etc/ssh/sshd_config
Expand Down Expand Up @@ -91,8 +89,6 @@ function setup_container_test(){
# Install required dependencies only when asked
if [ "$INSTALL_DEPS" == "yes" ]; then
install_worker_deps
# Mitigate https://github.com/containers/podman-py/issues/350
dnf install python3-rich -y
fi

BUILD_ARG=""
Expand Down

0 comments on commit 9abb3ca

Please sign in to comment.