Merge pull request #13374 from cevich/fix_dupe_docker_testing

Clarify v2 API testing for podman vs docker clients
containers · Feb 28, 2022 · 86a057e · 86a057e
2 parents 4c52952 + dbf34bf
commit 86a057e
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/contrib/cirrus/runner.sh b/contrib/cirrus/runner.sh
@@ -55,7 +55,11 @@ function _run_unit() {
 }
 
 function _run_apiv2() {
-    make localapiv2 |& logformatter
+    local m="Testing of API was performed using the **PODMAN*** client"
+    warn "$m"
+    if ! make localapiv2 |& logformatter; then
+        die "$m"
+    fi
 }
 
 function _run_compose() {
@@ -96,8 +100,12 @@ function _run_bindings() {
 }
 
 function _run_docker-py() {
+    m="Testing of API was performed using the **DOCKER** client"
+    warn "$m"
     source venv/bin/activate
-    make run-docker-py-tests
+    if ! make run-docker-py-tests; then
+        die "$m"
+    fi
 }
 
 function _run_endpoint() {