Skip to content

Commit 095134e

Browse files
committed
Polishing.
Extract Docker and Artifactory credentials into properties file. See #569
1 parent 281834b commit 095134e

File tree

2 files changed

+13
-8
lines changed

2 files changed

+13
-8
lines changed

Jenkinsfile

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@ pipeline {
3131
}
3232
options { timeout(time: 30, unit: 'MINUTES') }
3333
environment {
34-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
34+
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
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 'rm -Rf `find . -name "BACKUPDEFAULT*"`'
4141
sh 'rm -Rf `find . -name "ConfigDiskDir*"`'
@@ -65,11 +65,11 @@ pipeline {
6565
}
6666
options { timeout(time: 30, unit: 'MINUTES') }
6767
environment {
68-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
68+
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
6969
}
7070
steps {
7171
script {
72-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
72+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
7373
docker.image(p['docker.java.next.image']).inside(p['docker.java.inside.basic']) {
7474
sh 'rm -Rf `find . -name "BACKUPDEFAULT*"`'
7575
sh 'rm -Rf `find . -name "ConfigDiskDir*"`'
@@ -90,11 +90,11 @@ pipeline {
9090
}
9191
options { timeout(time: 30, unit: 'MINUTES') }
9292
environment {
93-
ARTIFACTORY = credentials('02bd1690-b54f-4c9f-819d-a77cb7a9822c')
93+
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
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.lts.image']).inside(p['docker.java.inside.basic']) {
9999
sh 'rm -Rf `find . -name "BACKUPDEFAULT*"`'
100100
sh 'rm -Rf `find . -name "ConfigDiskDir*"`'
@@ -125,12 +125,12 @@ pipeline {
125125
options { timeout(time: 20, unit: 'MINUTES') }
126126

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

131131
steps {
132132
script {
133-
docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
133+
docker.withRegistry(p['docker.registry'], p['docker.credentials']) {
134134
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.basic']) {
135135
sh 'rm -Rf `find . -name "BACKUPDEFAULT*"`'
136136
sh 'rm -Rf `find . -name "ConfigDiskDir*"`'

ci/pipeline.properties

Lines changed: 5 additions & 0 deletions
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)