Skip to content

Commit 61ca156

Browse files
committed
Use Harbor Proxy service on CI.
When run on CI servers, leverage an internal proxy service using Testcontainers ability to plugin a custom ImageNameSubstitor. See #1518 Original Pull Request: #1516
1 parent e9304b4 commit 61ca156

File tree

5 files changed

+97
-7
lines changed

5 files changed

+97
-7
lines changed

Jenkinsfile

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,14 +32,13 @@ pipeline {
3232
options { timeout(time: 30, unit: 'MINUTES') }
3333

3434
environment {
35-
DOCKER_HUB = credentials("${p['docker.credentials']}")
3635
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
36+
TESTCONTAINERS_IMAGE_SUBSTITUTOR = 'org.springframework.data.jdbc.ProxyImageNameSubstitutor'
3737
}
3838

3939
steps {
4040
script {
4141
docker.image(p['docker.java.main.image']).inside(p['docker.java.inside.docker']) {
42-
sh "docker login --username ${DOCKER_HUB_USR} --password ${DOCKER_HUB_PSW}"
4342
sh "PROFILE=ci,all-dbs ci/test.sh"
4443
sh "ci/clean.sh"
4544
}
@@ -63,14 +62,13 @@ pipeline {
6362
options { timeout(time: 30, unit: 'MINUTES') }
6463

6564
environment {
66-
DOCKER_HUB = credentials("${p['docker.credentials']}")
6765
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
66+
TESTCONTAINERS_IMAGE_SUBSTITUTOR = 'org.springframework.data.jdbc.ProxyImageNameSubstitutor'
6867
}
6968

7069
steps {
7170
script {
7271
docker.image(p['docker.java.next.image']).inside(p['docker.java.inside.docker']) {
73-
sh "docker login --username ${DOCKER_HUB_USR} --password ${DOCKER_HUB_PSW}"
7472
sh "PROFILE=ci ci/test.sh"
7573
sh "ci/clean.sh"
7674
}
@@ -85,14 +83,13 @@ pipeline {
8583
options { timeout(time: 30, unit: 'MINUTES') }
8684

8785
environment {
88-
DOCKER_HUB = credentials("${p['docker.credentials']}")
8986
ARTIFACTORY = credentials("${p['artifactory.credentials']}")
87+
TESTCONTAINERS_IMAGE_SUBSTITUTOR = 'org.springframework.data.jdbc.ProxyImageNameSubstitutor'
9088
}
9189

9290
steps {
9391
script {
9492
docker.image(p['docker.java.lts.image']).inside(p['docker.java.inside.docker']) {
95-
sh "docker login --username ${DOCKER_HUB_USR} --password ${DOCKER_HUB_PSW}"
9693
sh "PROFILE=ci ci/test.sh"
9794
sh "ci/clean.sh"
9895
}

ci/accept-third-party-license.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,5 @@
33
{
44
echo "mcr.microsoft.com/mssql/server:2017-CU12"
55
echo "ibmcom/db2:11.5.0.0a"
6-
} > spring-data-jdbc/src/test/resources/container-license-acceptance.txt
6+
echo "harbor-repo.vmware.com/dockerhub-proxy-cache/ibmcom/db2:11.5.0.0a"
7+
} > spring-data-jdbc/src/test/resources/container-license-acceptance.txt

ci/test.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
set -euo pipefail
44

55
ci/accept-third-party-license.sh
6+
67
mkdir -p /tmp/jenkins-home
78
chown -R 1001:1001 .
89
MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" \
Lines changed: 84 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,84 @@
1+
/*
2+
* Copyright 2023 the original author or authors.
3+
*
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
*
8+
* https://www.apache.org/licenses/LICENSE-2.0
9+
*
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
15+
*/
16+
package org.springframework.data.jdbc;
17+
18+
import java.util.Arrays;
19+
import java.util.List;
20+
21+
import org.slf4j.Logger;
22+
import org.slf4j.LoggerFactory;
23+
import org.testcontainers.utility.DockerImageName;
24+
import org.testcontainers.utility.ImageNameSubstitutor;
25+
26+
/**
27+
* An {@link ImageNameSubstitutor} only used on CI servers to leverage internal proxy solution, that needs to vary the
28+
* prefix based on which container image is needed.
29+
*
30+
* @author Greg Turnquist
31+
*/
32+
public class ProxyImageNameSubstitutor extends ImageNameSubstitutor {
33+
34+
private static final Logger LOG = LoggerFactory.getLogger(ProxyImageNameSubstitutor.class);
35+
36+
private static final List<String> NAMES_TO_PROXY_PREFIX = Arrays.asList("ryuk", "arm64v8/mariadb", "ibmcom/db2",
37+
"gvenzl/oracle-xe");
38+
39+
private static final List<String> NAMES_TO_LIBRARY_PROXY_PREFIX = Arrays.asList("mariadb", "mysql", "postgres");
40+
41+
private static final String PROXY_PREFIX = "harbor-repo.vmware.com/dockerhub-proxy-cache/";
42+
43+
private static final String LIBRARY_PROXY_PREFIX = PROXY_PREFIX + "library/";
44+
45+
@Override
46+
public DockerImageName apply(DockerImageName dockerImageName) {
47+
48+
if (NAMES_TO_PROXY_PREFIX.stream().anyMatch(s -> dockerImageName.asCanonicalNameString().contains(s))) {
49+
50+
String transformedName = applyProxyPrefix(dockerImageName.asCanonicalNameString());
51+
LOG.info("Converting " + dockerImageName.asCanonicalNameString() + " to " + transformedName);
52+
return DockerImageName.parse(transformedName);
53+
}
54+
55+
if (NAMES_TO_LIBRARY_PROXY_PREFIX.stream().anyMatch(s -> dockerImageName.asCanonicalNameString().contains(s))) {
56+
57+
String transformedName = applyProxyAndLibraryPrefix(dockerImageName.asCanonicalNameString());
58+
LOG.info("Converting " + dockerImageName.asCanonicalNameString() + " to " + transformedName);
59+
return DockerImageName.parse(transformedName);
60+
}
61+
62+
LOG.info("Not changing " + dockerImageName.asCanonicalNameString() + "...");
63+
return dockerImageName;
64+
}
65+
66+
@Override
67+
protected String getDescription() {
68+
return "Spring Data Proxy Image Name Substitutor";
69+
}
70+
71+
/**
72+
* Apply a non-library-based prefix.
73+
*/
74+
private static String applyProxyPrefix(String imageName) {
75+
return PROXY_PREFIX + imageName;
76+
}
77+
78+
/**
79+
* Apply a library based prefix.
80+
*/
81+
private static String applyProxyAndLibraryPrefix(String imageName) {
82+
return LIBRARY_PROXY_PREFIX + imageName;
83+
}
84+
}

spring-data-relational/pom.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,13 @@
8484
<scope>test</scope>
8585
</dependency>
8686

87+
<dependency>
88+
<groupId>org.testcontainers</groupId>
89+
<artifactId>testcontainers</artifactId>
90+
<version>${testcontainers}</version>
91+
<scope>test</scope>
92+
</dependency>
93+
8794
</dependencies>
8895

8996
</project>

0 commit comments

Comments
 (0)