From 2fdf6d7b6484c4110d06e90605dae405a6df38f8 Mon Sep 17 00:00:00 2001 From: Maxime Beauchemin Date: Wed, 13 Mar 2024 18:40:35 -0700 Subject: [PATCH] fix: docker builds in master fail (#27503) --- .github/supersetbot/src/docker.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/supersetbot/src/docker.js b/.github/supersetbot/src/docker.js index c287e6ae1..1bf9695c2 100644 --- a/.github/supersetbot/src/docker.js +++ b/.github/supersetbot/src/docker.js @@ -79,7 +79,7 @@ export function getDockerTags({ } export function getDockerCommand({ - preset, platform, isAuthenticated, buildContext, buildContextRef, forceLatest = false, + preset, platform, buildContext, buildContextRef, forceLatest = false, }) { const platforms = platform; @@ -113,6 +113,8 @@ export function getDockerCommand({ const tags = getDockerTags({ preset, platforms, sha, buildContext, buildContextRef: ref, forceLatest, }).map((tag) => `-t ${tag}`).join(' \\\n '); + const isAuthenticated = !!(process.env.DOCKERHUB_TOKEN && process.env.DOCKERHUB_USER); + const dockerArgs = isAuthenticated ? '--push' : '--load'; const targetArgument = buildTarget ? `--target ${buildTarget}` : ''; const cacheRef = `${CACHE_REPO}:${pyVer}`;