Skip to content

Commit 710cd17

Browse files
authored
Correct data type for domainHomeInImage webhook conversion (#3890)
1 parent f2c6ddf commit 710cd17

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

common/src/main/java/oracle/kubernetes/common/utils/SchemaConversionUtils.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright (c) 2022, Oracle and/or its affiliates.
1+
// Copyright (c) 2022, 2023, Oracle and/or its affiliates.
22
// Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl.
33

44
package oracle.kubernetes.common.utils;
@@ -430,7 +430,7 @@ private void convertDomainHomeInImageToDomainHomeSourceType(Map<String, Object>
430430
Object existing = domainSpec.remove("domainHomeInImage");
431431
if (existing != null && !domainSpec.containsKey("domainHomeSourceType")) {
432432
domainSpec.put("domainHomeSourceType",
433-
Boolean.parseBoolean((String) existing) ? "Image" : "PersistentVolume");
433+
Boolean.TRUE.equals(existing) ? "Image" : "PersistentVolume");
434434
}
435435
}
436436
}

common/src/test/java/oracle/kubernetes/common/utils/SchemaConversionUtilsTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright (c) 2022, Oracle and/or its affiliates.
1+
// Copyright (c) 2022, 2023, Oracle and/or its affiliates.
22
// Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl.
33

44
package oracle.kubernetes.common.utils;
@@ -297,7 +297,7 @@ private Map<String, Object> getDomainSpec(Map<String, Object> domainMap) {
297297
}
298298

299299
private void setDomainHomeInImage(Map<String, Object> v8Domain, boolean domainHomeInImage) {
300-
getDomainSpec(v8Domain).put("domainHomeInImage", String.valueOf(domainHomeInImage));
300+
getDomainSpec(v8Domain).put("domainHomeInImage", domainHomeInImage);
301301
}
302302

303303
private void setDomainHomeSourceType(Map<String, Object> v8Domain, String domainHomeSourceType) {

0 commit comments

Comments
 (0)