fix: use env-provided docker config

package.json

@@ -1,6 +1,6 @@
 {
   "name": "hooks",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "description": "Three projects are included - k8s: a kubernetes hook implementation that spins up pods dynamically to run a job - docker: A hook implementation of the runner's docker implementation  - A hook lib, which contains shared typescript definitions and utilities that the other packages consume",
   "main": "",
   "directories": {

packages/docker/src/dockerCommands/container.ts

@@ -314,17 +314,16 @@ export async function getContainerEnvValue(
 }
 
 export async function registryLogin(registry?: Registry): Promise<string> {
+  const configLocation =
+    env.DOCKER_CONFIG || `${env.RUNNER_TEMP}/.docker_${uuidv4()}`
   // if registry credentials are not provided, skip login and return default docker config location
   if (!registry) {
-    return env.DOCKER_CONFIG || `${env.HOME}/.docker`
+    return configLocation
   }
   const credentials = {
     username: registry.username,
     password: registry.password
   }
-
-  const configLocation = `${env.RUNNER_TEMP}/.docker_${uuidv4()}`
-  fs.mkdirSync(configLocation)
   try {
     await dockerLogin(configLocation, registry.serverUrl, credentials)
   } catch (error) {

releaseNotes.md

@@ -1,6 +1,6 @@
 ## Features
 
-- conditionally skip docker login step to fallback on default docker config if no registry credentials are provided
+- always run docker login and use env-provided docker config location
 
 <!-- ## Bugs -->