Skip to content

Docker: Add Node config env var SE_NODE_DELETE_SESSION_ON_UI #2871

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 30, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions ENV_VARIABLES.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@
| SE_HTTPS_PRIVATE_KEY | /opt/selenium/secrets/tls.key | | --https-private-key |
| SE_ENABLE_TRACING | true | | --tracing |
| SE_OTEL_EXPORTER_ENDPOINT | | | -Dotel.exporter.otlp.endpoint= |
| SE_OTEL_SERVICE_NAME | selenium-event-bus | | -Dotel.resource.attributes=service.name= |
| SE_OTEL_SERVICE_NAME | selenium-router | | -Dotel.resource.attributes=service.name= |
| SE_OTEL_JVM_ARGS | | | |
| SE_OTEL_TRACES_EXPORTER | otlp | | -Dotel.traces.exporter |
| SE_OTEL_JAVA_GLOBAL_AUTOCONFIGURE_ENABLED | true | | |
Expand All @@ -75,7 +75,7 @@
| SE_EVENT_BUS_PUBLISH_PORT | 4442 | | |
| SE_EVENT_BUS_SUBSCRIBE_PORT | 4443 | | |
| SE_NODE_SESSION_TIMEOUT | 300 | | --session-timeout |
| SE_NODE_ENABLE_MANAGED_DOWNLOADS | true | | --enable-managed-downloads |
| SE_NODE_ENABLE_MANAGED_DOWNLOADS | true | This causes the Node to auto manage files downloaded for a given session on the Node | --enable-managed-downloads |
| SE_NODE_ENABLE_CDP | | | --enable-cdp |
| SE_NODE_REGISTER_PERIOD | 120 | | --register-period |
| SE_NODE_REGISTER_CYCLE | 10 | | --register-cycle |
Expand Down Expand Up @@ -152,3 +152,4 @@
| SE_VIDEO_BUFSIZE | | | |
| SE_VIDEO_CRF | | | |
| SE_VIDEO_MAXRATE | | | |
| SE_NODE_DELETE_SESSION_ON_UI | true | Enable capability to support deleting session on Grid UI | --delete-session-on-ui |
1 change: 1 addition & 0 deletions NodeBase/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ ENV LANG_WHICH=${LANG_WHICH} \
SE_OFFLINE="true" \
SE_NODE_BROWSER_VERSION="stable" \
SE_NODE_PLATFORM_NAME="Linux" \
SE_NODE_DELETE_SESSION_ON_UI="true" \
#============================
# Some configuration options
#============================
Expand Down
4 changes: 4 additions & 0 deletions NodeBase/start-selenium-node.sh
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,10 @@ if [ "$SE_NODE_REGISTER_SHUTDOWN_ON_FAILURE" = "true" ]; then
append_se_opts "--register-shutdown-on-failure"
fi

if [ "$SE_NODE_DELETE_SESSION_ON_UI" = "true" ]; then
append_se_opts "--delete-session-on-ui" "true"
fi

if [ ! -z "$SE_NODE_HEARTBEAT_PERIOD" ]; then
append_se_opts "--heartbeat-period" "${SE_NODE_HEARTBEAT_PERIOD}"
fi
Expand Down
4 changes: 4 additions & 0 deletions Standalone/start-selenium-standalone.sh
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,10 @@ if [ ! -z "$SE_NODE_HEARTBEAT_PERIOD" ]; then
append_se_opts "--heartbeat-period" "${SE_NODE_HEARTBEAT_PERIOD}"
fi

if [ "$SE_NODE_DELETE_SESSION_ON_UI" = "true" ]; then
append_se_opts "--delete-session-on-ui" "true"
fi

if [ ! -z "$SE_LOG_LEVEL" ]; then
append_se_opts "--log-level" "${SE_LOG_LEVEL}"
fi
Expand Down
6 changes: 5 additions & 1 deletion scripts/generate_list_env_vars/description.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,8 @@
description: ''
cli: --session-timeout
- name: SE_NODE_ENABLE_MANAGED_DOWNLOADS
description: ''
description: This causes the Node to auto manage files downloaded for a given session
on the Node
cli: --enable-managed-downloads
- name: SE_NODE_ENABLE_CDP
description: ''
Expand Down Expand Up @@ -464,3 +465,6 @@
- name: SE_VIDEO_MAXRATE
description: ''
cli: ''
- name: SE_NODE_DELETE_SESSION_ON_UI
description: Enable capability to support deleting session on Grid UI
cli: --delete-session-on-ui
4 changes: 3 additions & 1 deletion scripts/generate_list_env_vars/value.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,8 @@
default: '10'
- name: SE_NODE_CONTAINER_NAME
default: ''
- name: SE_NODE_DELETE_SESSION_ON_UI
default: 'true'
- name: SE_NODE_DOCKER_CONFIG_FILENAME
default: docker.toml
- name: SE_NODE_ENABLE_CDP
Expand Down Expand Up @@ -161,7 +163,7 @@
- name: SE_OTEL_RESOURCE_ATTRIBUTES
default: ''
- name: SE_OTEL_SERVICE_NAME
default: selenium-event-bus
default: selenium-router
- name: SE_OTEL_TRACES_EXPORTER
default: otlp
- name: SE_PRESET
Expand Down
Loading