Skip to content

Commit 9800786

Browse files
committed
Polishing.
Also, externalize docker registry and artifactory credentials. See #1629
1 parent 8a6cf2e commit 9800786

File tree

2 files changed

+14
-8
lines changed

2 files changed

+14
-8
lines changed

Jenkinsfile

+8-8
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,11 @@ pipeline {
2424
}
2525
options { timeout(time: 30, unit: 'MINUTES') }
2626
environment {
27-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
27+
ARTIFACTORY = credentials(String.valueOf(p['artifactory.credentials']))
2828
}
2929
steps {
3030
script {
31-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
31+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
3232
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.basic']) {
3333
sh 'MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" ./mvnw -s settings.xml clean dependency:list verify -Dsort -B'
3434
}
@@ -48,11 +48,11 @@ pipeline {
4848
}
4949
options { timeout(time: 30, unit: 'MINUTES') }
5050
environment {
51-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
51+
ARTIFACTORY = credentials(String.valueOf(p['artifactory.credentials']))
5252
}
5353
steps {
5454
script {
55-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
55+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
5656
docker.image(p['docker.java.next.image']).inside(p['docker.java.inside.basic']) {
5757
sh 'MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" ./mvnw -s settings.xml -Pjava11 clean dependency:list verify -Dsort -B'
5858
}
@@ -67,11 +67,11 @@ pipeline {
6767
}
6868
options { timeout(time: 30, unit: 'MINUTES') }
6969
environment {
70-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
70+
ARTIFACTORY = credentials(String.valueOf(p['artifactory.credentials']))
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.lts.image']).inside(p['docker.java.inside.basic']) {
7676
sh 'MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" ./mvnw -s settings.xml -Pjava11 clean dependency:list verify -Dsort -B'
7777
}
@@ -89,12 +89,12 @@ pipeline {
8989
options { timeout(time: 20, unit: 'MINUTES') }
9090

9191
environment {
92-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
92+
ARTIFACTORY = credentials(String.valueOf(p['artifactory.credentials']))
9393
}
9494

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

ci/pipeline.properties

+6
Original file line numberDiff line numberDiff line change
@@ -22,3 +22,9 @@ 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+
27+
# Credentials
28+
artifactory.credentials=02bd1690-b54f-4c9f-819d-a77cb7a9822c
29+
docker.registry=
30+
docker.credentials=hub.docker.com-springbuildmaster

0 commit comments

Comments
 (0)