bug(SeleniumHQ#2038): Rollback io.opentelemetry 1.31.0 - add test tra…

…cing enabled

Signed-off-by: Viet Nguyen Duc <nguyenducviet4496@gmail.com>

.github/workflows/helm-chart-test.yml

@@ -2,11 +2,7 @@ name: Lint and Test Helm Charts
 
 on:
   push:
-    paths:
-      - 'charts/selenium-grid/**'
   pull_request:
-    paths:
-      - 'charts/selenium-grid/**'
   workflow_dispatch:
 
 permissions:

Base/Dockerfile

@@ -89,7 +89,10 @@ RUN curl -fLo /tmp/cs https://github.com/coursier/launchers/raw/master/coursier
   && mkdir -p /external_jars \
   && chmod -R 775 /external_jars
 
-RUN /tmp/cs fetch --classpath --cache /external_jars io.opentelemetry:opentelemetry-exporter-otlp:1.32.0 io.opentelemetry:opentelemetry-exporter-jaeger:1.32.0 io.grpc:grpc-netty:1.59.0 > /external_jars/.classpath.txt
+RUN /tmp/cs fetch --classpath --cache /external_jars \
+    io.opentelemetry:opentelemetry-exporter-otlp:1.31.0 \
+    io.opentelemetry:opentelemetry-exporter-jaeger:1.31.0 \
+    io.grpc:grpc-netty:1.59.0 > /external_jars/.classpath.txt
 
 RUN chmod 664 /external_jars/.classpath.txt
 RUN rm -fr /root/.cache/*

Distributor/start-selenium-grid-distributor.sh

@@ -58,8 +58,8 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else

EventBus/start-selenium-grid-eventbus.sh

@@ -23,8 +23,8 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else

Hub/start-selenium-grid-hub.sh

@@ -26,15 +26,14 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else
   echo "Tracing is disabled"
 fi
 
-
 java ${JAVA_OPTS:-$SE_JAVA_OPTS} \
   -jar /opt/selenium/selenium-server.jar \
   ${EXTRA_LIBS} hub \

NodeBase/start-selenium-node.sh

@@ -40,8 +40,8 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else

NodeChrome/Dockerfile

@@ -15,7 +15,7 @@ USER root
 #       google-chrome-unstable
 #============================================
 ARG CHROME_VERSION="google-chrome-stable"
-RUN wget -q -O - https://dl-ssl.google.com/linux/linux_signing_key.pub | apt-key add - \
+RUN wget -q -O - https://dl-ssl.google.com/linux/linux_signing_key.pub | gpg --dearmor | tee /etc/apt/trusted.gpg.d/google.gpg >/dev/null \
   && echo "deb http://dl.google.com/linux/chrome/deb/ stable main" >> /etc/apt/sources.list.d/google-chrome.list \
   && apt-get update -qqy \
   && apt-get -qqy install \

NodeDocker/start-selenium-grid-docker.sh

@@ -33,8 +33,8 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else

Router/start-selenium-grid-router.sh

@@ -58,8 +58,8 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else

SessionQueue/start-selenium-grid-session-queue.sh

@@ -23,8 +23,8 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else

Sessions/start-selenium-grid-sessions.sh

@@ -38,8 +38,8 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else

Standalone/start-selenium-standalone.sh

@@ -21,8 +21,8 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else

StandaloneDocker/start-selenium-grid-docker.sh

@@ -18,8 +18,8 @@ EXTRA_LIBS=""
 
 if [ ! -z "$SE_ENABLE_TRACING" ]; then
   EXTERNAL_JARS=$(</external_jars/.classpath.txt)
-  EXTRA_LIBS="--ext "
-  EXTRA_LIBS=${EXTRA_LIBS}:${EXTERNAL_JARS}
+  [ -n "$EXTRA_LIBS" ] && [ -n "${EXTERNAL_JARS}" ] && EXTRA_LIBS=${EXTRA_LIBS}:
+  EXTRA_LIBS="--ext "${EXTRA_LIBS}${EXTERNAL_JARS}
   echo "Tracing is enabled"
   echo "Classpath will be enriched with these external jars : " ${EXTRA_LIBS}
 else

charts/selenium-grid/TESTING.md

@@ -13,13 +13,17 @@ All related testing to this helm chart will be documented in this file.
 |                        | Auto scaling with `scalingType` is `deployment`                      | ✗  |
 |                        | Auto scaling with `autoscaling.scaledOptions.minReplicaCount` is `0` | ✓  |
 | Ingress                | Ingress is enabled without `hostname`                                | ✓  |
+|                        | Ingress is enabled with `hostname` is set                            | ✗  |
 |                        | Hub `sub-path` is set with Ingress `ImplementationSpecific` paths    | ✓  |
 | Distributed components | `isolateComponents` is enabled                                       | ✓  |
+|                        | `isolateComponents` is disabled                                      | ✗  |
 | Browser Nodes          | Node `nameOverride` is set                                           | ✓  |
 |                        | Sanity tests in node                                                 | ✓  |
 |                        | Video recorder is enabled in node                                    | ✗  |
 |                        | Node `extraEnvironmentVariables` is set value                        | ✓  |
 | General                | Set new image registry via `global.seleniumGrid.imageRegistry`       | ✓  |
+| Tracing                | Enable tracing via `SE_ENABLE_TRACING`                               | ✓  |
+|                        | Disable tracing via `SE_ENABLE_TRACING`                              | ✗  |
 
 ## Build & test Docker images with Helm charts
 Noted: These `make` commands are composed and tested on Linux x86_64.

charts/selenium-grid/ci/tracing-values.yaml

@@ -0,0 +1,13 @@
+hub:
+  extraEnvironmentVariables:
+    - name: SE_ENABLE_TRACING
+      value: "true"
+    - name: SE_JAVA_OPTS
+      value: "-Dotel.traces.exporter=jaeger -Dotel.exporter.jaeger.endpoint=http://{{ .Release.Name }}-jaeger-all-in-one-headless:14250 -Dotel.resource.attributes=service.name=selenium -Dotel.java.global-autoconfigure.enabled=true"
+
+components:
+  extraEnvironmentVariables:
+    - name: SE_ENABLE_TRACING
+      value: "true"
+    - name: SE_JAVA_OPTS
+      value: "-Dotel.traces.exporter=jaeger -Dotel.exporter.jaeger.endpoint=http://{{ .Release.Name }}-jaeger-all-in-one-headless:14250 -Dotel.resource.attributes=service.name=selenium -Dotel.java.global-autoconfigure.enabled=true"

tests/K8s/chart_install.sh

@@ -43,8 +43,9 @@ trap 'on_failure' ERR
 echo "Deploy Selenium Grid Chart"
 helm upgrade --install ${RELEASE_NAME} \
 -f ${TEST_VALUES_PATH}/auth-ingress-values.yaml \
--f ${TEST_VALUES_PATH}/autoscaling-values.yaml \
+-f ${TEST_VALUES_PATH}/tracing-values.yaml \
 -f ${TEST_VALUES_PATH}/${MATRIX_BROWSER}-values.yaml \
+-f ${TEST_VALUES_PATH}/autoscaling-values.yaml \
 --set autoscaling.enableWithExistingKEDA=${SELENIUM_GRID_AUTOSCALING} \
 --set autoscaling.scaledOptions.minReplicaCount=${SELENIUM_GRID_AUTOSCALING_MIN_REPLICA} \
 --set global.seleniumGrid.imageTag=${VERSION} --set global.seleniumGrid.imageRegistry=${NAMESPACE} \