Skip to content

Commit 1e82a5b

Browse files
committed
fix issues with volumes and volume mounts for introspector job
1 parent 5502c30 commit 1e82a5b

File tree

2 files changed

+20
-8
lines changed

2 files changed

+20
-8
lines changed

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

Lines changed: 19 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import java.io.File;
55
import java.util.Arrays;
66
import java.util.List;
7+
import java.util.Optional;
78
import oracle.kubernetes.operator.KubernetesConstants;
89
import oracle.kubernetes.operator.LabelConstants;
910
import oracle.kubernetes.operator.ProcessingConstants;
@@ -70,8 +71,13 @@ private List<V1PersistentVolumeClaim> getClaims() {
7071
return info.getClaims().getItems();
7172
}
7273

74+
private String getDiscoveredClaim() {
75+
return getClaims().isEmpty() ? null : getClaims().iterator().next().getMetadata().getName();
76+
}
77+
7378
private String getClaimName() {
74-
return getClaims().iterator().next().getMetadata().getName();
79+
return Optional.ofNullable(info.getDomain().getPersistentVolumeClaimName())
80+
.orElse(getDiscoveredClaim());
7581
}
7682

7783
// ----------------------- step methods ------------------------------
@@ -208,7 +214,7 @@ private V1PodSpec createPodSpec(TuningParameters tuningParameters) {
208214

209215
podSpec.setImagePullSecrets(info.getDomain().getSpec().getImagePullSecrets());
210216

211-
if (!getClaims().isEmpty()) {
217+
if (getClaimName() != null) {
212218
podSpec.addVolumesItem(
213219
new V1Volume()
214220
.name(STORAGE_VOLUME)
@@ -222,10 +228,12 @@ private V1PodSpec createPodSpec(TuningParameters tuningParameters) {
222228
.name(secretName + "-volume")
223229
.secret(getOverrideSecretVolumeSource(secretName)));
224230
}
225-
podSpec.addVolumesItem(
226-
new V1Volume()
227-
.name(getConfigOverrides() + "-volume")
228-
.configMap(getOverridesVolumeSource(getConfigOverrides())));
231+
if (getConfigOverrides() != null && getConfigOverrides().length() > 0) {
232+
podSpec.addVolumesItem(
233+
new V1Volume()
234+
.name(getConfigOverrides() + "-volume")
235+
.configMap(getOverridesVolumeSource(getConfigOverrides())));
236+
}
229237

230238
return podSpec;
231239
}
@@ -239,11 +247,14 @@ private V1Container createContainer(TuningParameters tuningParameters) {
239247
.command(getContainerCommand())
240248
.env(getEnvironmentVariables(tuningParameters))
241249
.addVolumeMountsItem(volumeMount(STORAGE_VOLUME, STORAGE_MOUNT_PATH))
242-
.addVolumeMountsItem(
243-
readOnlyVolumeMount(getConfigOverrides() + "-volume", OVERRIDES_CM_MOUNT_PATH))
244250
.addVolumeMountsItem(readOnlyVolumeMount(SECRETS_VOLUME, SECRETS_MOUNT_PATH))
245251
.addVolumeMountsItem(readOnlyVolumeMount(SCRIPTS_VOLUME, SCRIPTS_MOUNTS_PATH));
246252

253+
if (getConfigOverrides() != null && getConfigOverrides().length() > 0) {
254+
container.addVolumeMountsItem(
255+
readOnlyVolumeMount(getConfigOverrides() + "-volume", OVERRIDES_CM_MOUNT_PATH));
256+
}
257+
247258
List<String> configOverrideSecrets = getConfigOverrideSecrets();
248259
for (String secretName : configOverrideSecrets) {
249260
container.addVolumeMountsItem(

operator/src/main/resources/scripts/introspectDomain.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -617,6 +617,7 @@ def addSitCfgXml(self):
617617
self.writeln("<?xml version='1.0' encoding='UTF-8'?>")
618618
self.writeln("<d:domain xmlns:d=\"http://xmlns.oracle.com/weblogic/domain\" xmlns:f=\"http://xmlns.oracle.com/weblogic/domain-fragment\" xmlns:s=\"http://xmlns.oracle.com/weblogic/situational-config\">")
619619
self.indent()
620+
self.writeln("<s:expiration> 2020-07-16T19:20+01:00 </s:expiration>")
620621
self.customizeNodeManagerCreds()
621622
self.customizeDomainLogPath()
622623
self.customizeServers()

0 commit comments

Comments
 (0)