Skip to content

Commit 44b7b8f

Browse files
committed
Polishing.
Extract docker and artifactory credentials into property file. See #1151
1 parent 97331f7 commit 44b7b8f

File tree

2 files changed

+16
-11
lines changed

2 files changed

+16
-11
lines changed

Jenkinsfile

+11-11
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,13 @@ pipeline {
3232
options { timeout(time: 30, unit: 'MINUTES') }
3333

3434
environment {
35-
DOCKER_HUB = credentials('hub.docker.com-springbuildmaster')
36-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
35+
DOCKER_HUB = credentials("${p['docker.credentials']}")
36+
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
3737
}
3838

3939
steps {
4040
script {
41-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
41+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
4242
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.docker']) {
4343
sh "docker login --username ${DOCKER_HUB_USR} --password ${DOCKER_HUB_PSW}"
4444
sh "PROFILE=ci,all-dbs ci/test.sh"
@@ -65,13 +65,13 @@ pipeline {
6565
options { timeout(time: 30, unit: 'MINUTES') }
6666

6767
environment {
68-
DOCKER_HUB = credentials('hub.docker.com-springbuildmaster')
69-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
68+
DOCKER_HUB = credentials("${p['docker.credentials']}")
69+
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
7070
}
7171

7272
steps {
7373
script {
74-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
74+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
7575
docker.image(p['docker.java.next.image']).inside(p['docker.java.inside.docker']) {
7676
sh "docker login --username ${DOCKER_HUB_USR} --password ${DOCKER_HUB_PSW}"
7777
sh "PROFILE=ci,java11 ci/test.sh"
@@ -89,13 +89,13 @@ pipeline {
8989
options { timeout(time: 30, unit: 'MINUTES') }
9090

9191
environment {
92-
DOCKER_HUB = credentials('hub.docker.com-springbuildmaster')
93-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
92+
DOCKER_HUB = credentials("${p['docker.credentials']}")
93+
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
9494
}
9595

9696
steps {
9797
script {
98-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
98+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
9999
docker.image(p['docker.java.lts.image']).inside(p['docker.java.inside.docker']) {
100100
sh "docker login --username ${DOCKER_HUB_USR} --password ${DOCKER_HUB_PSW}"
101101
sh "PROFILE=ci,java11 ci/test.sh"
@@ -122,12 +122,12 @@ pipeline {
122122
options { timeout(time: 20, unit: 'MINUTES') }
123123

124124
environment {
125-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
125+
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
126126
}
127127

128128
steps {
129129
script {
130-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
130+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
131131
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.basic']) {
132132
sh 'MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" ./mvnw -s settings.xml -Pci,artifactory -Dmaven.repo.local=/tmp/jenkins-home/.m2/spring-data-jdbc-non-root ' +
133133
'-Dartifactory.server=https://repo.spring.io ' +

ci/pipeline.properties

+5
Original file line numberDiff line numberDiff line change
@@ -22,3 +22,8 @@ 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
29+
artifactory.credentials=02bd1690-b54f-4c9f-819d-a77cb7a9822c

0 commit comments

Comments
 (0)