Skip to content

Commit d1ce191

Browse files
ppluleeVietND96
andauthored
Docker: Revert SE_NODE_MAX_SESSIONS writes config to [node] section (#2961)
* fix(config): SE_NODE_MAX_SESSIONS written to wrong section when generating config * Run format --------- Signed-off-by: Viet Nguyen Duc <[email protected]> Co-authored-by: Viet Nguyen Duc <[email protected]>
1 parent a49dcc0 commit d1ce191

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

NodeBase/generate_config

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,10 @@ else
122122
fi
123123
echo "session-timeout = ${SE_NODE_SESSION_TIMEOUT}" >>"$FILENAME"
124124
echo "override-max-sessions = ${SE_NODE_OVERRIDE_MAX_SESSIONS}" >>"$FILENAME"
125+
# Validate SE_NODE_MAX_SESSIONS is a positive integer
126+
if [[ "${SE_NODE_MAX_SESSIONS}" =~ ^[0-9]+$ ]] && [[ "${SE_NODE_MAX_SESSIONS}" -gt 0 ]]; then
127+
echo "max-sessions = ${SE_NODE_MAX_SESSIONS}" >>"$FILENAME"
128+
fi
125129
echo "detect-drivers = false" >>"$FILENAME"
126130
echo "drain-after-session-count = ${DRAIN_AFTER_SESSION_COUNT:-$SE_DRAIN_AFTER_SESSION_COUNT}" >>"$FILENAME"
127131
echo "" >>"$FILENAME"
@@ -174,10 +178,6 @@ if [ -d "/opt/selenium/browsers" ]; then
174178
echo "[[node.driver-configuration]]" >>"$FILENAME"
175179
echo "display-name = \"${SE_NODE_BROWSER_NAME}\"" >>"$FILENAME"
176180
echo "stereotype = '${SE_NODE_STEREOTYPE}'" >>"$FILENAME"
177-
# Validate SE_NODE_MAX_SESSIONS is a positive integer
178-
if [[ "${SE_NODE_MAX_SESSIONS}" =~ ^[0-9]+$ ]] && [[ "${SE_NODE_MAX_SESSIONS}" -gt 0 ]]; then
179-
echo "max-sessions = ${SE_NODE_MAX_SESSIONS}" >>"$FILENAME"
180-
fi
181181
echo "" >>"$FILENAME"
182182
fi
183183

0 commit comments

Comments
 (0)