Skip to content

Commit 727d5fd

Browse files
committed
Switch to Broadcom docker proxy.
Closes #1822
1 parent 3bb4b60 commit 727d5fd

File tree

4 files changed

+33
-41
lines changed

4 files changed

+33
-41
lines changed

Jenkinsfile

+22-16
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,11 @@ pipeline {
3838

3939
steps {
4040
script {
41-
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.docker']) {
42-
sh "PROFILE=ci,all-dbs JENKINS_USER_NAME=${p['jenkins.user.name']} ci/test.sh"
43-
sh "JENKINS_USER_NAME=${p['jenkins.user.name']} ci/clean.sh"
41+
docker.withRegistry(p['docker.proxy.registry'], p['docker.proxy.credentials']) {
42+
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.docker']) {
43+
sh "PROFILE=ci,all-dbs JENKINS_USER_NAME=${p['jenkins.user.name']} ci/test.sh"
44+
sh "JENKINS_USER_NAME=${p['jenkins.user.name']} ci/clean.sh"
45+
}
4446
}
4547
}
4648
}
@@ -66,9 +68,11 @@ pipeline {
6668
}
6769
steps {
6870
script {
69-
docker.image(p['docker.java.next.image']).inside(p['docker.java.inside.docker']) {
70-
sh "PROFILE=ci,all-dbs JENKINS_USER_NAME=${p['jenkins.user.name']} ci/test.sh"
71-
sh "JENKINS_USER_NAME=${p['jenkins.user.name']} ci/clean.sh"
71+
docker.withRegistry(p['docker.proxy.registry'], p['docker.proxy.credentials']) {
72+
docker.image(p['docker.java.next.image']).inside(p['docker.java.inside.docker']) {
73+
sh "PROFILE=ci,all-dbs JENKINS_USER_NAME=${p['jenkins.user.name']} ci/test.sh"
74+
sh "JENKINS_USER_NAME=${p['jenkins.user.name']} ci/clean.sh"
75+
}
7276
}
7377
}
7478
}
@@ -93,16 +97,18 @@ pipeline {
9397
}
9498
steps {
9599
script {
96-
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.basic']) {
97-
sh 'MAVEN_OPTS="-Duser.name=' + "${p['jenkins.user.name']}" + ' -Duser.home=/tmp/jenkins-home" ' +
98-
"./mvnw -s settings.xml -Pci,artifactory " +
99-
"-Dartifactory.server=${p['artifactory.url']} " +
100-
"-Dartifactory.username=${ARTIFACTORY_USR} " +
101-
"-Dartifactory.password=${ARTIFACTORY_PSW} " +
102-
"-Dartifactory.staging-repository=${p['artifactory.repository.snapshot']} " +
103-
"-Dartifactory.build-name=spring-data-relational " +
104-
"-Dartifactory.build-number=spring-data-relational-${BRANCH_NAME}-build-${BUILD_NUMBER} " +
105-
"-Dmaven.test.skip=true clean deploy -U -B"
100+
docker.withRegistry(p['docker.proxy.registry'], p['docker.proxy.credentials']) {
101+
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.basic']) {
102+
sh 'MAVEN_OPTS="-Duser.name=' + "${p['jenkins.user.name']}" + ' -Duser.home=/tmp/jenkins-home" ' +
103+
"./mvnw -s settings.xml -Pci,artifactory " +
104+
"-Dartifactory.server=${p['artifactory.url']} " +
105+
"-Dartifactory.username=${ARTIFACTORY_USR} " +
106+
"-Dartifactory.password=${ARTIFACTORY_PSW} " +
107+
"-Dartifactory.staging-repository=${p['artifactory.repository.snapshot']} " +
108+
"-Dartifactory.build-name=spring-data-relational " +
109+
"-Dartifactory.build-number=spring-data-relational-${BRANCH_NAME}-build-${BUILD_NUMBER} " +
110+
"-Dmaven.test.skip=true clean deploy -U -B"
111+
}
106112
}
107113
}
108114
}

ci/accept-third-party-license.sh

+4-4
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,13 @@
33
{
44
echo "mcr.microsoft.com/mssql/server:2019-CU16-ubuntu-20.04"
55
echo "ibmcom/db2:11.5.7.0a"
6-
echo "harbor-repo.vmware.com/mcr-proxy-cache/mssql/server:2019-CU16-ubuntu-20.04"
7-
echo "harbor-repo.vmware.com/dockerhub-proxy-cache/ibmcom/db2:11.5.7.0a"
6+
echo "docker-hub.usw1.packages.broadcom.com/mssql/server:2019-CU16-ubuntu-20.04"
7+
echo "docker-hub.usw1.packages.broadcom.com/ibmcom/db2:11.5.7.0a"
88
} > spring-data-jdbc/src/test/resources/container-license-acceptance.txt
99

1010
{
1111
echo "mcr.microsoft.com/mssql/server:2022-latest"
1212
echo "ibmcom/db2:11.5.7.0a"
13-
echo "harbor-repo.vmware.com/mcr-proxy-cache/mssql/server:2022-latest"
14-
echo "harbor-repo.vmware.com/dockerhub-proxy-cache/ibmcom/db2:11.5.7.0a"
13+
echo "docker-hub.usw1.packages.broadcom.com/mssql/server:2022-latest"
14+
echo "docker-hub.usw1.packages.broadcom.com/ibmcom/db2:11.5.7.0a"
1515
} > spring-data-r2dbc/src/test/resources/container-license-acceptance.txt

ci/pipeline.properties

+5-2
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ java.main.tag=17.0.9_9-jdk-focal
33
java.next.tag=20-jdk-jammy
44

55
# Docker container images - standard
6-
docker.java.main.image=harbor-repo.vmware.com/dockerhub-proxy-cache/library/eclipse-temurin:${java.main.tag}
7-
docker.java.next.image=harbor-repo.vmware.com/dockerhub-proxy-cache/library/eclipse-temurin:${java.next.tag}
6+
docker.java.main.image=library/eclipse-temurin:${java.main.tag}
7+
docker.java.next.image=library/eclipse-temurin:${java.next.tag}
88

99
# Supported versions of MongoDB
1010
docker.mongodb.4.4.version=4.4.25
@@ -13,6 +13,7 @@ docker.mongodb.6.0.version=6.0.10
1313

1414
# Supported versions of Redis
1515
docker.redis.6.version=6.2.13
16+
docker.redis.7.version=7.2.4
1617

1718
# Supported versions of Cassandra
1819
docker.cassandra.3.version=3.11.16
@@ -24,6 +25,8 @@ docker.java.inside.docker=-u root -v /var/run/docker.sock:/var/run/docker.sock -
2425
# Credentials
2526
docker.registry=
2627
docker.credentials=hub.docker.com-springbuildmaster
28+
docker.proxy.registry=https://docker-hub.usw1.packages.broadcom.com
29+
docker.proxy.credentials=usw1_packages_broadcom_com-jenkins-token
2730
artifactory.credentials=02bd1690-b54f-4c9f-819d-a77cb7a9822c
2831
artifactory.url=https://repo.spring.io
2932
artifactory.repository.snapshot=libs-snapshot-local

spring-data-relational/src/test/java/org/springframework/data/ProxyImageNameSubstitutor.java

+2-19
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
/**
2626
* An {@link ImageNameSubstitutor} only used on CI servers to leverage internal proxy solution, that needs to vary the
2727
* prefix based on which container image is needed.
28-
*
28+
*
2929
* @author Greg Turnquist
3030
*/
3131
public class ProxyImageNameSubstitutor extends ImageNameSubstitutor {
@@ -37,14 +37,10 @@ public class ProxyImageNameSubstitutor extends ImageNameSubstitutor {
3737

3838
private static final List<String> NAMES_TO_LIBRARY_PROXY_PREFIX = List.of("mariadb", "mysql", "postgres");
3939

40-
private static final List<String> NAMES_TO_MCR_PROXY_PREFIX = List.of("mcr.microsoft.com/mssql");
41-
42-
private static final String PROXY_PREFIX = "harbor-repo.vmware.com/dockerhub-proxy-cache/";
40+
private static final String PROXY_PREFIX = "docker-hub.usw1.packages.broadcom.com/";
4341

4442
private static final String LIBRARY_PROXY_PREFIX = PROXY_PREFIX + "library/";
4543

46-
private static final String MCR_PROXY_PREFIX = "harbor-repo.vmware.com/mcr-proxy-cache/";
47-
4844
@Override
4945
public DockerImageName apply(DockerImageName dockerImageName) {
5046

@@ -62,13 +58,6 @@ public DockerImageName apply(DockerImageName dockerImageName) {
6258
return DockerImageName.parse(transformedName);
6359
}
6460

65-
if (NAMES_TO_MCR_PROXY_PREFIX.stream().anyMatch(s -> dockerImageName.asCanonicalNameString().contains(s))) {
66-
67-
String transformedName = applyMcrProxyPrefix(dockerImageName.asCanonicalNameString());
68-
LOG.info("Converting " + dockerImageName.asCanonicalNameString() + " to " + transformedName);
69-
return DockerImageName.parse(transformedName);
70-
}
71-
7261
LOG.info("Not changing " + dockerImageName.asCanonicalNameString() + "...");
7362
return dockerImageName;
7463
}
@@ -92,10 +81,4 @@ private static String applyProxyAndLibraryPrefix(String imageName) {
9281
return LIBRARY_PROXY_PREFIX + imageName;
9382
}
9483

95-
/**
96-
* Apply an MCR-based prefix.
97-
*/
98-
private static String applyMcrProxyPrefix(String imageName) {
99-
return MCR_PROXY_PREFIX + imageName.substring("mcr.microsoft.com/".length());
100-
}
10184
}

0 commit comments

Comments
 (0)