Skip to content

Commit 306224c

Browse files
authored
Fix for the auxiliary image upgrade test to adjust the old volume name prefix to new prefix for hash (#3692)
* Fix for aux image upgrade tests.
1 parent 89087c9 commit 306224c

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

common/src/main/java/oracle/kubernetes/common/AuxiliaryImageConstants.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ private AuxiliaryImageConstants() {
1111

1212
public static final String AUXILIARY_IMAGE_TARGET_PATH = "/tmpAuxiliaryImage";
1313
public static final String AUXILIARY_IMAGE_VOLUME_NAME_PREFIX = "ai-vol-";
14+
public static final String AUXILIARY_IMAGE_VOLUME_NAME_OLD_PREFIX = "aux-image-volume-";
1415
public static final String AUXILIARY_IMAGE_INIT_CONTAINER_WRAPPER_SCRIPT = "/weblogic-operator/scripts/auxImage.sh";
1516
public static final String AUXILIARY_IMAGE_INIT_CONTAINER_NAME_PREFIX = "operator-aux-container";
1617
public static final String AUXILIARY_IMAGE_DEFAULT_INIT_CONTAINER_COMMAND

operator/src/main/java/oracle/kubernetes/operator/helpers/PodStepContext.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,8 @@
8282
import org.apache.commons.lang3.builder.EqualsBuilder;
8383
import org.apache.commons.lang3.builder.HashCodeBuilder;
8484

85+
import static oracle.kubernetes.common.AuxiliaryImageConstants.AUXILIARY_IMAGE_VOLUME_NAME_OLD_PREFIX;
86+
import static oracle.kubernetes.common.AuxiliaryImageConstants.AUXILIARY_IMAGE_VOLUME_NAME_PREFIX;
8587
import static oracle.kubernetes.common.CommonConstants.COMPATIBILITY_MODE;
8688
import static oracle.kubernetes.common.helpers.AuxiliaryImageEnvVars.AUXILIARY_IMAGE_MOUNT_PATH;
8789
import static oracle.kubernetes.common.logging.MessageKeys.CYCLING_POD_EVICTED;
@@ -1177,7 +1179,8 @@ private boolean canAdjustLegacyHashToMatch(V1Pod currentPod, String requiredHash
11771179
}
11781180

11791181
private void adjustVolumeMountName(List<V1VolumeMount> convertedVolumeMounts, V1VolumeMount volumeMount) {
1180-
convertedVolumeMounts.add(volumeMount.name(volumeMount.getName().replaceAll("^" + COMPATIBILITY_MODE, "")));
1182+
convertedVolumeMounts.add(volumeMount.name(volumeMount.getName().replaceAll("^" + COMPATIBILITY_MODE
1183+
+ AUXILIARY_IMAGE_VOLUME_NAME_PREFIX, AUXILIARY_IMAGE_VOLUME_NAME_OLD_PREFIX)));
11811184
}
11821185

11831186
private void adjustContainer(List<V1Container> convertedContainers, V1Container container, V1Pod currentPod) {
@@ -1212,7 +1215,8 @@ private boolean isPodFromOperatorMajorMinorVersion34RevAbove0(V1Pod currentPod)
12121215
}
12131216

12141217
private void adjustVolumeName(List<V1Volume> convertedVolumes, V1Volume volume) {
1215-
convertedVolumes.add(volume.name(volume.getName().replaceAll("^" + COMPATIBILITY_MODE, "")));
1218+
convertedVolumes.add(volume.name(volume.getName().replaceAll("^" + COMPATIBILITY_MODE
1219+
+ AUXILIARY_IMAGE_VOLUME_NAME_PREFIX, AUXILIARY_IMAGE_VOLUME_NAME_OLD_PREFIX)));
12161220
}
12171221

12181222
private void convertAuxImagesInitContainerVolumeAndMounts(V1Pod recipe, V1Pod currentPod) {

operator/src/test/java/oracle/kubernetes/operator/helpers/ReferenceObjects.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1567,7 +1567,7 @@ class ReferenceObjects {
15671567
+ " - mountPath: /weblogic-operator/introspector\n"
15681568
+ " name: weblogic-domain-introspect-cm-volume\n"
15691569
+ " - mountPath: /auxiliary\n"
1570-
+ " name: compat-aux-image-volume-auxiliaryimagevolume1\n"
1570+
+ " name: compat-ai-vol-auxiliaryimagevolume1\n"
15711571
+ " hostname: uid1-admin-server\n"
15721572
+ " imagePullSecrets: []\n"
15731573
+ " initContainers:\n"
@@ -1589,7 +1589,7 @@ class ReferenceObjects {
15891589
+ " name: compat-operator-aux-container1\n"
15901590
+ " volumeMounts:\n"
15911591
+ " - mountPath: /tmpAuxiliaryImage\n"
1592-
+ " name: compat-aux-image-volume-auxiliaryimagevolume1\n"
1592+
+ " name: compat-ai-vol-auxiliaryimagevolume1\n"
15931593
+ " - mountPath: /weblogic-operator/scripts\n"
15941594
+ " name: weblogic-scripts-cm-volume\n"
15951595
+ " nodeSelector: {}\n"

0 commit comments

Comments
 (0)