Fix minor issues after PR #1881 and #1981 (#1983)

Signed-off-by: Viet Nguyen Duc <[email protected]>
SeleniumHQ · Oct 27, 2023 · 0f7367e · 0f7367e
1 parent 020ded4
commit 0f7367e
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/Video/Dockerfile b/Video/Dockerfile
@@ -6,7 +6,7 @@ LABEL authors="Selenium <[email protected]>"
 #========================
 RUN apk update \
   && apk upgrade \
-  && apk add --no-cache --update --virtual .build-deps supervisor bash curl jq xset python3 py3-psutil py3-pip \
+  && apk add --no-cache --update --virtual .build-deps supervisor bash curl jq xset xdpyinfo python3 py3-psutil py3-pip \
   && ln -sf python3 /usr/bin/python \
   && rm -rf /tmp/* /var/cache/apk/*
 

diff --git a/Video/supervisord.conf b/Video/supervisord.conf
@@ -10,6 +10,7 @@ pidfile=/var/run/supervisor/supervisord.pid   ; (supervisord pidfile;default sup
 nodaemon=true                                 ; (start in foreground if true;default false)
 minfds=1024                                   ; (min. avail startup file descriptors;default 1024)
 minprocs=200                                  ; (min. avail process descriptors;default 200)
+user=root                                     ; (default is current user, required if root)
 
 [program:video-recording]
 priority=0

diff --git a/charts/selenium-grid/templates/video-cm.yaml b/charts/selenium-grid/templates/video-cm.yaml
@@ -20,7 +20,10 @@ data:
     return_code=1
     max_attempts=600
     attempts=0
-    mkfifo /videos/uploadpipe
+    if [ ! -p /videos/uploadpipe ];
+    then
+      mkfifo /videos/uploadpipe
+    fi
     if [[ "$UPLOAD_DESTINATION_PREFIX" = "" ]]
     then
         echo Upload destination not known since UPLOAD_DESTINATION_PREFIX is not set. Exiting video recorder.