Skip to content

Commit 64ac168

Browse files
committed
only add ADMIN_PORT_SECURE env var if its value is true
1 parent 5ed03f0 commit 64ac168

File tree

4 files changed

+5
-4
lines changed

4 files changed

+5
-4
lines changed

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -803,7 +803,9 @@ void overrideContainerWeblogicEnvVars(List<V1EnvVar> vars) {
803803
addEnvVar(vars, "DOMAIN_HOME", getDomainHome());
804804
addEnvVar(vars, "ADMIN_NAME", getAsName());
805805
addEnvVar(vars, "ADMIN_PORT", getAsPort().toString());
806-
addEnvVar(vars, "ADMIN_PORT_SECURE", Boolean.toString(isLocalAdminProtocolChannelSecure()));
806+
if (isLocalAdminProtocolChannelSecure()) {
807+
addEnvVar(vars, "ADMIN_PORT_SECURE", "true");
808+
}
807809
addEnvVar(vars, "SERVER_NAME", getServerName());
808810
addEnvVar(vars, "DOMAIN_UID", getDomainUID());
809811
addEnvVar(vars, "NODEMGR_HOME", NODEMGR_HOME);

operator/src/main/resources/scripts/startNodeManager.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
# ${DOMAIN_UID}/${SERVER_NAME}_nodemanager.out
2727
# Default:
2828
# Use LOG_HOME. If LOG_HOME not set, use NODEMGR_HOME.
29+
# ADMIN_PORT_SECURE = "true" if the admin protocol is secure. Default is false
2930
#
3031
# If SERVER_NAME is set, then this NM is for a WL Server and these must also be set:
3132
#
@@ -51,6 +52,7 @@ export WL_HOME="${WL_HOME:-/u01/oracle/wlserver}"
5152
stm_script=${WL_HOME}/server/bin/startNodeManager.sh
5253

5354
SERVER_NAME=${SERVER_NAME:-introspector}
55+
ADMIN_PORT_SECURE=${ADMIN_PORT_SECURE:-false}
5456

5557
trace "Starting node manager for domain-uid='$DOMAIN_UID' and server='$SERVER_NAME'."
5658

operator/src/main/resources/scripts/startServer.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,6 @@ checkEnv \
172172
SERVICE_NAME \
173173
ADMIN_NAME \
174174
ADMIN_PORT \
175-
ADMIN_PORT_SECURE \
176175
SERVER_OUT_IN_POD_LOG \
177176
AS_SERVICE_NAME || exitOrLoop
178177

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -443,7 +443,6 @@ public void whenPodCreated_hasPredefinedEnvVariables() {
443443
hasEnvVar("DOMAIN_HOME", "/u01/oracle/user_projects/domains"),
444444
hasEnvVar("ADMIN_NAME", ADMIN_SERVER),
445445
hasEnvVar("ADMIN_PORT", Integer.toString(ADMIN_PORT)),
446-
hasEnvVar("ADMIN_PORT_SECURE", "false"),
447446
hasEnvVar("SERVER_NAME", getServerName()),
448447
hasEnvVar("ADMIN_USERNAME", null),
449448
hasEnvVar("ADMIN_PASSWORD", null),
@@ -933,7 +932,6 @@ V1Container createPodSpecContainer() {
933932
.addEnvItem(envItem("DOMAIN_HOME", "/u01/oracle/user_projects/domains"))
934933
.addEnvItem(envItem("ADMIN_NAME", ADMIN_SERVER))
935934
.addEnvItem(envItem("ADMIN_PORT", Integer.toString(ADMIN_PORT)))
936-
.addEnvItem(envItem("ADMIN_PORT_SECURE", "false"))
937935
.addEnvItem(envItem("SERVER_NAME", getServerName()))
938936
.addEnvItem(envItem("ADMIN_USERNAME", null))
939937
.addEnvItem(envItem("ADMIN_PASSWORD", null))

0 commit comments

Comments
 (0)