Skip to content

Commit 6bcad8a

Browse files
committed
feat: BREAKING CHANGE. docker image now requires config file at runtime
instead of inserting the API key at build time. It now must be mounted during start up time.
1 parent 39ab688 commit 6bcad8a

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

vars/runJenkinsPipeline.groovy

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -663,11 +663,11 @@ def call(){
663663
def CONFIG = readJSON(file: CONFIG_FILE)['deploy']
664664
echo "CONFIG = ${CONFIG}"
665665
def build_args = CONFIG['docker']['build']['buildArgs'].collect{"--build-arg=${it}"}.join(' ')
666-
docker.withRegistry(CONFIG['docker']['server']['registry'], 'jenkins-nexus'){
667-
def dockerImage = docker.build("${IMAGE_NAME}:${DOCKER_TAG}", "${build_args} .")
668-
dockerImage.push()
669-
dockerImage.push('latest')
670-
}
666+
def dockerImage = docker.build("${IMAGE_NAME}:${DOCKER_TAG}", "${build_args} .")
667+
// docker.withRegistry(CONFIG['docker']['server']['registry'], 'jenkins-nexus'){
668+
// dockerImage.push()
669+
// dockerImage.push('latest')
670+
// }
671671
}
672672
}
673673
}

0 commit comments

Comments
 (0)