[ARVADOS] updated: 1.3.0-63-ge6bc5eeac

Git user git at public.curoverse.com
Tue Jan 8 16:59:17 EST 2019


Summary of changes:
 tools/arvbox/bin/arvbox                                    |  2 +-
 tools/arvbox/lib/arvbox/docker/service/composer/run        |  2 +-
 tools/arvbox/lib/arvbox/docker/service/ready/run-service   |  1 +
 tools/arvbox/lib/arvbox/docker/service/workbench2/run      |  2 +-
 .../lib/arvbox/docker/service/workbench2/run-service       | 14 ++++++++++++--
 5 files changed, 16 insertions(+), 5 deletions(-)

       via  e6bc5eeacc812ac41aadeec6fd7a907b5aab0237 (commit)
      from  28aa8adcc90108922b1274f5568ce3c5745cca78 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit e6bc5eeacc812ac41aadeec6fd7a907b5aab0237
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Tue Jan 8 16:57:56 2019 -0500

    14660: make "sv restart" work for workbench2, prevent fail loop
    
    Print out https url for workbench2 on start.
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/tools/arvbox/bin/arvbox b/tools/arvbox/bin/arvbox
index e2f58cf13..ada53a200 100755
--- a/tools/arvbox/bin/arvbox
+++ b/tools/arvbox/bin/arvbox
@@ -104,7 +104,7 @@ wait_for_arvbox() {
     LOGPID=$!
     while read line ; do
         echo $line
-        if echo $line | grep "Workbench is running at" >/dev/null ; then
+        if echo $line | grep "Workbench2 is running at" >/dev/null ; then
             kill $LOGPID
         fi
     done < $FF
diff --git a/tools/arvbox/lib/arvbox/docker/service/composer/run b/tools/arvbox/lib/arvbox/docker/service/composer/run
index cd2f86a27..50a8ce120 100755
--- a/tools/arvbox/lib/arvbox/docker/service/composer/run
+++ b/tools/arvbox/lib/arvbox/docker/service/composer/run
@@ -5,4 +5,4 @@
 
 set -e
 
-/usr/local/lib/arvbox/runsu.sh $0-service $1
+exec /usr/local/lib/arvbox/runsu.sh $0-service $1
diff --git a/tools/arvbox/lib/arvbox/docker/service/ready/run-service b/tools/arvbox/lib/arvbox/docker/service/ready/run-service
index 4e1371eb6..470d10537 100755
--- a/tools/arvbox/lib/arvbox/docker/service/ready/run-service
+++ b/tools/arvbox/lib/arvbox/docker/service/ready/run-service
@@ -91,5 +91,6 @@ fi
 echo
 echo "Your Arvados-in-a-box is ready!"
 echo "Workbench is running at https://$localip"
+echo "Workbench2 is running at https://$localip:${services[workbench2-ssl]}"
 
 rm -r /tmp/arvbox-ready
diff --git a/tools/arvbox/lib/arvbox/docker/service/workbench2/run b/tools/arvbox/lib/arvbox/docker/service/workbench2/run
index cd2f86a27..50a8ce120 100755
--- a/tools/arvbox/lib/arvbox/docker/service/workbench2/run
+++ b/tools/arvbox/lib/arvbox/docker/service/workbench2/run
@@ -5,4 +5,4 @@
 
 set -e
 
-/usr/local/lib/arvbox/runsu.sh $0-service $1
+exec /usr/local/lib/arvbox/runsu.sh $0-service $1
diff --git a/tools/arvbox/lib/arvbox/docker/service/workbench2/run-service b/tools/arvbox/lib/arvbox/docker/service/workbench2/run-service
index 1aef13279..2dbef4ab8 100755
--- a/tools/arvbox/lib/arvbox/docker/service/workbench2/run-service
+++ b/tools/arvbox/lib/arvbox/docker/service/workbench2/run-service
@@ -18,6 +18,16 @@ if test "$1" = "--only-deps" ; then
     exit
 fi
 
-echo "{\"API_HOST\": \"${localip}:${services[controller-ssl]}\"}" > /usr/src/workbench2/public/config.json
+cat <<EOF > /usr/src/workbench2/public/config.json
+{
+  "API_HOST": "${localip}:${services[controller-ssl]}",
+  "VOCABULARY_URL": "vocabulary-example.json",
+  "FILE_VIEWERS_CONFIG_URL": "file-viewers-example.json"
+}
+EOF
+
 export HTTPS=false
-exec yarn start
+# Can't use "yarn start", need to run the dev server script
+# directly so that the TERM signal from "sv restart" gets to the
+# right process.
+exec node node_modules/react-scripts-ts/scripts/start.js

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list