Fix breeze failing breeze on missing docker config (apache#18330)

sekikn · Sep 18, 2021 · 87e9865 · 87e9865
1 parent bfe7726
commit 87e9865
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/scripts/ci/libraries/_build_images.sh b/scripts/ci/libraries/_build_images.sh
@@ -432,9 +432,18 @@ function build_images::configure_docker_registry() {
     else
         verbosity::print_info "Skip Login to GitHub Registry ${GITHUB_REGISTRY} as token is missing"
     fi
-    local new_config
-    new_config=$(jq '.experimental = "enabled"' "${HOME}/.docker/config.json")
-    echo "${new_config}" > "${HOME}/.docker/config.json"
+    if [[ ${CI} == "true" ]]; then
+        # we only need that on CI in order to run "docker manifest" when we check if remote image exists
+        # See function push_pull_remove_images::check_image_manifest()
+        local new_config
+        new_config=$(jq '.experimental = "enabled"' "${HOME}/.docker/config.json" || true)
+        if [[ ${new_config} != "" ]]; then
+            echo "${new_config}" > "${HOME}/.docker/config.json"
+        else
+            echo "${COLOR_YELLOW}Could not set experimental flag in ${HOME}/.docker/config.json ${COLOR_RESET}"
+            echo "${COLOR_YELLOW}docker manifest command will likely not work${COLOR_RESET}"
+        fi
+    fi
 }