Skip to content

Commit 4e3fee7

Browse files
committed
Polishing.
Extract docker credentials into properties file. See #2541
1 parent 52d8976 commit 4e3fee7

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

Jenkinsfile

+4-4
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ pipeline {
3535
}
3636
steps {
3737
script {
38-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
38+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
3939
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.basic']) {
4040
sh 'MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" ./mvnw -s settings.xml clean dependency:list verify -Dsort -U -B'
4141
}
@@ -63,7 +63,7 @@ pipeline {
6363
}
6464
steps {
6565
script {
66-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
66+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
6767
docker.image(p['docker.java.next.image']).inside(p['docker.java.inside.basic']) {
6868
sh 'MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" ./mvnw -s settings.xml -Pjava11 clean dependency:list verify -Dsort -U -B'
6969
}
@@ -82,7 +82,7 @@ pipeline {
8282
}
8383
steps {
8484
script {
85-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
85+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
8686
docker.image(p['docker.java.lts.image']).inside(p['docker.java.inside.basic']) {
8787
sh 'MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" ./mvnw -s settings.xml -Pjava11 clean dependency:list verify -Dsort -U -B'
8888
}
@@ -112,7 +112,7 @@ pipeline {
112112

113113
steps {
114114
script {
115-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
115+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
116116
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.basic']) {
117117
sh 'MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" ./mvnw -s settings.xml -Pci,artifactory ' +
118118
'-Dartifactory.server=https://repo.spring.io ' +

ci/pipeline.properties

+4
Original file line numberDiff line numberDiff line change
@@ -22,3 +22,7 @@ docker.cassandra.3.version=3.11.10
2222
# Docker environment settings
2323
docker.java.inside.basic=-v $HOME:/tmp/jenkins-home
2424
docker.java.inside.docker=-u root -v /var/run/docker.sock:/var/run/docker.sock -v /usr/bin/docker:/usr/bin/docker -v $HOME:/tmp/jenkins-home
25+
26+
# Credentials
27+
docker.registry=
28+
docker.credentials=hub.docker.com-springbuildmaster

0 commit comments

Comments
 (0)