Use podman if it is installed. Fix DOCKER_USERNAME for podman.

manadart · Oct 6, 2023 · 8fb6955 · 8fb6955
1 parent 3ffe3f7
commit 8fb6955
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Makefile b/Makefile
@@ -453,8 +453,8 @@ check-deps:
 
 
 # CAAS related targets
-export OCI_BUILDER         ?= docker
-DOCKER_USERNAME            ?= jujusolutions
+export OCI_BUILDER         ?= $(shell (which podman 2>&1 > /dev/null && echo podman) || echo docker )
+DOCKER_USERNAME            ?= docker.io/jujusolutions
 DOCKER_BUILDX_CONTEXT      ?= juju-make
 DOCKER_STAGING_DIR         ?= ${BUILD_DIR}/docker-staging
 JUJUD_STAGING_DIR          ?= ${DOCKER_STAGING_DIR}/jujud-operator

diff --git a/make_functions.sh b/make_functions.sh
@@ -12,7 +12,7 @@ JUJU_BUILD_NUMBER=${JUJU_BUILD_NUMBER:-}
 
 # Docker variables
 OCI_BUILDER=${OCI_BUILDER:-docker}
-DOCKER_USERNAME=${DOCKER_USERNAME:-jujusolutions}
+DOCKER_USERNAME=${DOCKER_USERNAME:-docker.io/jujusolutions}
 DOCKER_BUILDX_CONTEXT=${DOCKER_BUILDX_CONTEXT:-juju-make}
 DOCKER_STAGING_DIR="${BUILD_DIR}/docker-staging"
 DOCKER_BIN=${DOCKER_BIN:-$(which ${OCI_BUILDER} || true)}