Moving config generation in docker files

This allow random users to launch the images. The behavior is needed for OpenShift. Cf. issue SeleniumHQ#3 from ddavison/selenium-openshift-templates
Remi-p · Jun 7, 2017 · 73e0c87 · 73e0c87
1 parent 60eeed0
commit 73e0c87
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 2 deletions.
diff --git a/NodeBase/entry_point.sh b/NodeBase/entry_point.sh
@@ -1,7 +1,6 @@
 #!/bin/bash
 
 source /opt/bin/functions.sh
-/opt/bin/generate_config > /opt/selenium/config.json
 
 export GEOMETRY="$SCREEN_WIDTH""x""$SCREEN_HEIGHT""x""$SCREEN_DEPTH"
 

diff --git a/NodeChrome/Dockerfile.txt b/NodeChrome/Dockerfile.txt
@@ -39,3 +39,6 @@ COPY generate_config /opt/bin/generate_config
 # Chrome Launch Script Modification
 #=================================
 COPY chrome_launcher.sh /opt/google/chrome/google-chrome
+
+# Generating config inside the image rather than with entry_point.sh
+RUN /opt/bin/generate_config > /opt/selenium/config.json
diff --git a/NodeDebug/entry_point.sh b/NodeDebug/entry_point.sh
@@ -3,7 +3,6 @@
 # IMPORTANT: Change this file only in directory NodeDebug!
 
 source /opt/bin/functions.sh
-/opt/bin/generate_config > /opt/selenium/config.json
 
 export GEOMETRY="$SCREEN_WIDTH""x""$SCREEN_HEIGHT""x""$SCREEN_DEPTH"
 

diff --git a/NodeFirefox/Dockerfile.txt b/NodeFirefox/Dockerfile.txt
@@ -35,3 +35,6 @@ COPY generate_config /opt/bin/generate_config
 # To run a command as administrator (user "root"), use "sudo <command>". See "man sudo_root" for details.
 # When logging into the container
 RUN sudo echo ""
+
+# Generating config inside the image rather than with entry_point.sh
+RUN /opt/bin/generate_config > /opt/selenium/config.json