Skip to content

Commit a1482ea

Browse files
authored
Remove hardcoded OCIR in Dockerfile and change dataHome to run tests in OKD (#2999)
* Remove hardcoded OCIR in Dockerfile and change dataHome to run tests in OKD
1 parent 95bd639 commit a1482ea

File tree

6 files changed

+20
-7
lines changed

6 files changed

+20
-7
lines changed

integration-tests/src/test/java/oracle/weblogic/kubernetes/ItMiiAuxiliaryImage.java

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@
3636
import static java.util.concurrent.TimeUnit.SECONDS;
3737
import static oracle.weblogic.kubernetes.TestConstants.ADMIN_PASSWORD_DEFAULT;
3838
import static oracle.weblogic.kubernetes.TestConstants.ADMIN_USERNAME_DEFAULT;
39+
import static oracle.weblogic.kubernetes.TestConstants.BUSYBOX_IMAGE;
40+
import static oracle.weblogic.kubernetes.TestConstants.BUSYBOX_TAG;
3941
import static oracle.weblogic.kubernetes.TestConstants.DOMAIN_IMAGES_REPO;
4042
import static oracle.weblogic.kubernetes.TestConstants.KIND_REPO;
4143
import static oracle.weblogic.kubernetes.TestConstants.MII_AUXILIARY_IMAGE_NAME;
@@ -53,6 +55,7 @@
5355
import static oracle.weblogic.kubernetes.actions.ActionConstants.MODEL_DIR;
5456
import static oracle.weblogic.kubernetes.actions.ActionConstants.RESOURCE_DIR;
5557
import static oracle.weblogic.kubernetes.actions.ActionConstants.WDT_DOWNLOAD_FILENAME_DEFAULT;
58+
import static oracle.weblogic.kubernetes.actions.ActionConstants.WORK_DIR;
5659
import static oracle.weblogic.kubernetes.actions.TestActions.buildAppArchive;
5760
import static oracle.weblogic.kubernetes.actions.TestActions.createDomainCustomResource;
5861
import static oracle.weblogic.kubernetes.actions.TestActions.defaultAppParams;
@@ -1229,8 +1232,18 @@ public void tearDownAll() {
12291232
}
12301233

12311234
private void createAuxiliaryImage(String stageDirPath, String dockerFileLocation, String auxiliaryImage) {
1235+
//replace the BUSYBOX_IMAGE and BUSYBOX_TAG in Dockerfile
1236+
Path dockerDestFile = Paths.get(WORK_DIR, "auximages", "Dockerfile");
1237+
assertDoesNotThrow(() -> {
1238+
Files.createDirectories(dockerDestFile.getParent());
1239+
Files.copy(Paths.get(dockerFileLocation),
1240+
dockerDestFile, StandardCopyOption.REPLACE_EXISTING);
1241+
replaceStringInFile(dockerDestFile.toString(), "BUSYBOX_IMAGE", BUSYBOX_IMAGE);
1242+
replaceStringInFile(dockerDestFile.toString(), "BUSYBOX_TAG", BUSYBOX_TAG);
1243+
});
1244+
12321245
String cmdToExecute = String.format("cd %s && docker build -f %s %s -t %s .",
1233-
stageDirPath, dockerFileLocation,
1246+
stageDirPath, dockerDestFile.toString(),
12341247
"--build-arg AUXILIARY_IMAGE_PATH=/auxiliary", auxiliaryImage);
12351248
assertTrue(new Command()
12361249
.withParams(new CommandParams()

integration-tests/src/test/java/oracle/weblogic/kubernetes/ItMultiDomainModelsWithLoadBalancer.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ class ItMultiDomainModelsWithLoadBalancer {
144144
private static final String WLDF_OPENSESSION_APP = "opensessionapp";
145145
private static final String WLDF_OPENSESSION_APP_CONTEXT_ROOT = "opensession";
146146
private static final String wlSecretName = "weblogic-credentials";
147-
private static final String DATA_HOME_OVERRIDE = "/u01/oracle/mydata";
147+
private static final String DATA_HOME_OVERRIDE = "/u01/mydata";
148148
private static final String miiImageName = "mii-image";
149149
private static final String wdtModelFileForMiiDomain = "model-multiclusterdomain-sampleapp-wls.yaml";
150150
private static final String miiDomainUid = "miidomain";
@@ -502,7 +502,7 @@ void testLivenessProbe() {
502502

503503
/**
504504
* Verify dataHome override in a domain with domain in image type.
505-
* In this domain, set dataHome to /u01/oracle/mydata in domain custom resource
505+
* In this domain, set dataHome to /u01/mydata in domain custom resource
506506
* The domain contains JMS and File Store configuration
507507
* File store directory is set to /u01/oracle/customFileStore in the model file which should be overridden by dataHome
508508
* File store and JMS server are targeted to the WebLogic cluster cluster-1

integration-tests/src/test/java/oracle/weblogic/kubernetes/TestConstants.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@ public interface TestConstants {
241241
public static final String MII_AUXILIARY_IMAGE_NAME = DOMAIN_IMAGES_REPO + "mii-ai-image";
242242
public static final boolean SKIP_BUILD_IMAGES_IF_EXISTS =
243243
Boolean.parseBoolean(getNonEmptySystemProperty("wko.it.skip.build.images.if.exists", "false"));
244-
public static final String BUSYBOX_IMAGE = "phx.ocir.io/weblogick8s/test-images/docker/busybox";
244+
public static final String BUSYBOX_IMAGE = OCIR_REGISTRY + "/weblogick8s/test-images/docker/busybox";
245245
public static final String BUSYBOX_TAG = "1.34.1";
246246

247247
// Skip the mii/wdt basic image build locally if needed

integration-tests/src/test/java/oracle/weblogic/kubernetes/utils/DomainUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -662,7 +662,7 @@ public static Domain createDomainResourceForDomainInImage(String domainUid,
662662
.spec(new DomainSpec()
663663
.domainUid(domainUid)
664664
.domainHome(WDT_IMAGE_DOMAINHOME_BASE_DIR + "/" + domainUid)
665-
.dataHome("/u01/oracle/mydata")
665+
.dataHome("/u01/mydata")
666666
.domainHomeSourceType("Image")
667667
.image(imageName)
668668
.addImagePullSecretsItem(new V1LocalObjectReference()

integration-tests/src/test/resources/auxiliaryimage/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
# Default '/auxiliary'.
1717
#
1818

19-
FROM phx.ocir.io/weblogick8s/test-images/docker/busybox:1.34.1
19+
FROM BUSYBOX_IMAGE:BUSYBOX_TAG
2020
ARG AUXILIARY_IMAGE_PATH=/auxiliary
2121
ARG USER=oracle
2222
ARG USERID=1000

integration-tests/src/test/resources/auxiliaryimage/negative/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
# Default '/auxiliary'.
1717
#
1818

19-
FROM phx.ocir.io/weblogick8s/test-images/docker/busybox:1.34.1
19+
FROM BUSYBOX_IMAGE:BUSYBOX_TAG
2020
ARG AUXILIARY_IMAGE_PATH=/auxiliary
2121
ARG USER=tester
2222
ARG USERID=1001

0 commit comments

Comments
 (0)