Skip to content

Commit 8e45ecb

Browse files
committed
Merge branch 'owls-130071' into 'release/4.2'
fix introspector initContainer not added for legacy domain on pv. See merge request weblogic-cloud/weblogic-kubernetes-operator!4988 (cherry picked from commit db82107) affffc8 fix introspector initContainer not added for legacy domain on pv. Co-authored-by: Ryan Eberhard <[email protected]>
1 parent 9f2f0b3 commit 8e45ecb

File tree

1 file changed

+17
-11
lines changed

1 file changed

+17
-11
lines changed

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

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -474,22 +474,28 @@ protected void addInitContainers(V1PodSpec podSpec) {
474474
getInitializeDomainOnPV().ifPresent(initializeDomainOnPV -> addInitDomainOnPVInitContainer(initContainers));
475475
Optional.ofNullable(getAuxiliaryImages()).ifPresent(auxImages -> addInitContainers(initContainers, auxImages));
476476
Optional.ofNullable(getDomainCreationImages()).ifPresent(dcrImages -> addInitContainers(initContainers, dcrImages));
477-
initContainers.addAll(getAdditionalInitContainers().stream()
478-
.filter(container -> isAllowedInIntrospector(container.getName()))
479-
.map(c -> c.env(createEnv(c)).envFrom(c.getEnvFrom()).resources(createResources()))
480-
.toList());
477+
List<V1Container> introspectorInitContainers = getIntrospectorInitContainers();
478+
if (introspectorInitContainers.isEmpty()) {
479+
initContainers.addAll(getAdditionalInitContainers().stream()
480+
.filter(container -> isAllowedInIntrospector(container.getName()))
481+
.map(c -> c.env(createEnv(c)).envFrom(c.getEnvFrom()).resources(createResources()))
482+
.toList());
483+
} else {
484+
for (V1Container initContainer : introspectorInitContainers) {
485+
if (initContainers.stream().noneMatch(
486+
container -> container.getName().equals(initContainer.getName()))) {
487+
initContainer.securityContext(getInitContainerSecurityContext());
488+
initContainers.add(initContainer);
489+
}
490+
}
491+
}
481492
podSpec.initContainers(initContainers);
482493
}
483494

484495
private void addInitContainers(List<V1Container> initContainers, List<? extends DeploymentImage> auxiliaryImages) {
485496
IntStream.range(0, auxiliaryImages.size()).forEach(idx ->
486-
initContainers.add(createInitContainerForAuxiliaryImage(auxiliaryImages.get(idx), idx,
487-
isInitializeDomainOnPV())));
488-
List<V1Container> introspectorInitContainers = getIntrospectorInitContainers();
489-
for (V1Container initContainer : introspectorInitContainers) {
490-
initContainer.securityContext(getInitContainerSecurityContext());
491-
initContainers.add(initContainer);
492-
}
497+
initContainers.add(createInitContainerForAuxiliaryImage(auxiliaryImages.get(idx), idx,
498+
isInitializeDomainOnPV())));
493499
}
494500

495501
private Optional<InitializeDomainOnPV> getInitializeDomainOnPV() {

0 commit comments

Comments
 (0)