Skip to content

Commit

Permalink
Switch test yaml to docker driver
Browse files Browse the repository at this point in the history
The podman driver is flakeky locally and consistently failing in github
actions now. Switching to docker fix the issues in github.

Developers need to install docker to run drenv test locally. I tried to
use the podman-docker pacakge but it does not emulate docker good
enough for minikube.

Signed-off-by: Nir Soffer <nsoffer@redhat.com>
  • Loading branch information
nirs committed Jun 22, 2023
1 parent c050ef6 commit 97a3c38
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 9 deletions.
16 changes: 9 additions & 7 deletions test/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,21 +37,23 @@ environment.
[Install clusteradm CLI tool](https://open-cluster-management.io/getting-started/installation/start-the-control-plane/#install-clusteradm-cli-tool)
for the details. Version 0.5.0 or later is required.

1. Install `podman`
1. Install `docker`

```
sudo dnf install podman
sudo dnf install docker
```

Allow yourself to run podman as root without a password by adding
this file as root:
Add yourself to the `docker` group to allow running docker as root:

```
# cat /etc/sudoers.d/podman
# Allow passwordless podman
myusername ALL=(ALL) NOPASSWD: /usr/bin/podman
sudo usermod -aG docker $USER && newgrp docker
```

For more info see [Linux post-installation steps for Docker Engine](https://docs.docker.com/engine/install/linux-postinstall/).

docker is used only for running drenv tests locally. You can use
podman for building and running containers locally.

### Testing that drenv is healthy

Run this script to make sure `drenv` works:
Expand Down
3 changes: 1 addition & 2 deletions test/envs/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
name: test
profiles:
- name: cluster
driver: podman
container_runtime: cri-o
driver: docker
memory: 2g
workers:
- addons:
Expand Down

0 comments on commit 97a3c38

Please sign in to comment.