This is an automated email from the ASF dual-hosted git repository. maximebeauchemin pushed a commit to branch fix_master_once_more in repository https://gitbox.apache.org/repos/asf/superset.git
commit f6c84fa85df37367be40faa744b73ca34a163f10 Author: Maxime Beauchemin <[email protected]> AuthorDate: Wed Mar 13 23:41:23 2024 -0700 fix: master docker build is broken Following up on https://github.com/apache/superset/pull/27503 while flying blind. I'm guessing that DOCKERHUB_USER is empty (?) No way for me to check. Looking for DOCKERHUB_TOKEN should be enough though. --- .github/supersetbot/src/docker.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/supersetbot/src/docker.js b/.github/supersetbot/src/docker.js index 1bf9695c23..03eb1a315f 100644 --- a/.github/supersetbot/src/docker.js +++ b/.github/supersetbot/src/docker.js @@ -113,7 +113,7 @@ 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 isAuthenticated = !!(process.env.DOCKERHUB_TOKEN); const dockerArgs = isAuthenticated ? '--push' : '--load'; const targetArgument = buildTarget ? `--target ${buildTarget}` : '';
