Skip to content

Commit 2ccb976

Browse files
committed
Use builder to prevent modifying original
1 parent 2d20af9 commit 2ccb976

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

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

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141
import io.kubernetes.client.openapi.models.V1PodSpec;
4242
import io.kubernetes.client.openapi.models.V1PodSpecBuilder;
4343
import io.kubernetes.client.openapi.models.V1Probe;
44+
import io.kubernetes.client.openapi.models.V1ProbeBuilder;
4445
import io.kubernetes.client.openapi.models.V1ResourceRequirements;
4546
import io.kubernetes.client.openapi.models.V1SecretVolumeSource;
4647
import io.kubernetes.client.openapi.models.V1Volume;
@@ -953,7 +954,8 @@ private V1Probe createReadinessProbe(PodTuning tuning) {
953954

954955
@SuppressWarnings("SameParameterValue")
955956
private V1Probe getReadinessProbe() {
956-
return Optional.ofNullable(getServerSpec().getReadinessProbe()).orElse(new V1Probe());
957+
return Optional.ofNullable(getServerSpec().getReadinessProbe())
958+
.map(V1ProbeBuilder::new).map(V1ProbeBuilder::build).orElse(new V1Probe());
957959
}
958960

959961
private V1Probe createLivenessProbe(PodTuning tuning) {
@@ -997,7 +999,8 @@ private V1Probe createLivenessProbe(PodTuning tuning) {
997999
}
9981000

9991001
private V1Probe getLivenessProbe() {
1000-
return Optional.ofNullable(getServerSpec().getLivenessProbe()).orElse(new V1Probe());
1002+
return Optional.ofNullable(getServerSpec().getLivenessProbe())
1003+
.map(V1ProbeBuilder::new).map(V1ProbeBuilder::build).orElse(new V1Probe());
10011004
}
10021005

10031006
private boolean mockWls() {

0 commit comments

Comments
 (0)