Skip to content

Commit

Permalink
Merge pull request #11169 from cevich/enable_docker_py_testing
Browse files Browse the repository at this point in the history
Enable docker-py compat. testing w/ ignored result
  • Loading branch information
openshift-ci[bot] authored Aug 16, 2021
2 parents 41fb06a + 2744366 commit 26b1ddd
Show file tree
Hide file tree
Showing 5 changed files with 24 additions and 8 deletions.
3 changes: 1 addition & 2 deletions .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -378,8 +378,7 @@ osx_alt_build_task:
always: *binary_artifacts


# This task is a stub: In the future it will be used to verify
# podman is compatible with the docker python-module.
# Verify podman is compatible with the docker python-module.
docker-py_test_task:
name: Docker-py Compat.
alias: docker-py_test
Expand Down
10 changes: 7 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -493,10 +493,12 @@ validate.completions:
if [ -x /bin/zsh ]; then /bin/zsh completions/zsh/_podman; fi
if [ -x /bin/fish ]; then /bin/fish completions/fish/podman.fish; fi

# Note: Assumes test/python/requirements.txt is installed & available
.PHONY: run-docker-py-tests
run-docker-py-tests:
$(eval testLogs=$(shell mktemp podman_tmp_XXXX))
./bin/podman run --rm --security-opt label=disable --privileged -v $(testLogs):/testLogs --net=host -e DOCKER_HOST=tcp://localhost:8080 $(DOCKERPY_IMAGE) sh -c "pytest $(DOCKERPY_TEST) "
touch test/__init__.py
pytest test/python/docker/
-rm test/__init__.py

.PHONY: localunit
localunit: test/goecho/goecho
Expand Down Expand Up @@ -841,11 +843,13 @@ clean: ## Clean all make artifacts
build \
test/checkseccomp/checkseccomp \
test/goecho/goecho \
test/__init__.py \
test/testdata/redis-image \
libpod/container_ffjson.go \
libpod/pod_ffjson.go \
libpod/container_easyjson.go \
libpod/pod_easyjson.go \
.install.goimports \
docs/build
docs/build \
venv
make -C docs clean
3 changes: 2 additions & 1 deletion contrib/cirrus/runner.sh
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,8 @@ function _run_bindings() {
}

function _run_docker-py() {
msg "This is docker-py stub, it is only a stub"
source venv/bin/activate
make run-docker-py-tests
}

function _run_endpoint() {
Expand Down
14 changes: 13 additions & 1 deletion contrib/cirrus/setup_environment.sh
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,19 @@ case "$TEST_FLAVOR" in
bigto dnf install -y glibc-minimal-langpack rpm-build
fi
;&
docker-py) ;&
docker-py)
remove_packaged_podman_files
make install PREFIX=/usr ETCDIR=/etc

# TODO: Don't install stuff at test runtime! Do this from
# cache_images/fedora_packaging.sh in containers/automation_images
# and STRONGLY prefer installing RPMs vs pip packages in venv
dnf install -y python3-virtualenv python3-pytest4
virtualenv venv
source venv/bin/activate
pip install --upgrade pip
pip install --requirement $GOSRC/test/python/requirements.txt
;;
build) make clean ;;
unit) ;;
apiv2) ;& # use next item
Expand Down
2 changes: 1 addition & 1 deletion test/python/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
docker~=4.4.3

requests-mock~=1.9.3
requests~=2.20.0
setuptools~=50.3.2
python-dateutil~=2.8.1
Expand Down

0 comments on commit 26b1ddd

Please sign in to comment.