[ARVADOS] updated: 1.3.0-2299-g162db4b03

Git user git at public.arvados.org
Thu Feb 27 15:23:36 UTC 2020


Summary of changes:
 build/run-tests.sh                  |  2 +-
 sdk/python/tests/run_test_server.py | 10 +++-------
 2 files changed, 4 insertions(+), 8 deletions(-)

       via  162db4b03e17b449144bc259e3296f56681cfe7c (commit)
      from  37ab0eedec5eaf99c27b6b64fd04cc9248081713 (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 162db4b03e17b449144bc259e3296f56681cfe7c
Author: Tom Clegg <tom at tomclegg.ca>
Date:   Thu Feb 27 10:22:54 2020 -0500

    15954: Fix railsapi startup in run_test_server.py.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at tomclegg.ca>

diff --git a/build/run-tests.sh b/build/run-tests.sh
index fff095e6e..3bb3f0014 100755
--- a/build/run-tests.sh
+++ b/build/run-tests.sh
@@ -405,7 +405,7 @@ checkdiscoverydoc() {
     dd="https://${1}/discovery/v1/apis/arvados/v1/rest"
     if ! (set -o pipefail; curl -fsk "$dd" | grep -q ^{ ); then
         echo >&2 "ERROR: could not retrieve discovery doc from RailsAPI at $dd"
-        tail -v $WORKSPACE/services/api/log/test.log
+        tail -v $WORKSPACE/tmp/railsapi.log
         return 1
     fi
     echo "${dd} ok"
diff --git a/sdk/python/tests/run_test_server.py b/sdk/python/tests/run_test_server.py
index 5b75de08f..bca372ccd 100644
--- a/sdk/python/tests/run_test_server.py
+++ b/sdk/python/tests/run_test_server.py
@@ -327,9 +327,9 @@ def run(leave_running_atexit=False):
     env.pop('ARVADOS_API_HOST_INSECURE', None)
     env.pop('ARVADOS_API_TOKEN', None)
     logf = open(_logfilename('railsapi'), 'a')
-    start_msg = subprocess.check_output(
+    railsapi = subprocess.Popen(
         ['bundle', 'exec',
-         'passenger', 'start', '-d', '-p{}'.format(port),
+         'passenger', 'start', '-p{}'.format(port),
          '--pid-file', pid_file,
          '--log-file', '/dev/stdout',
          '--ssl',
@@ -340,11 +340,7 @@ def run(leave_running_atexit=False):
     if not leave_running_atexit:
         atexit.register(kill_server_pid, pid_file, passenger_root=api_src_dir)
 
-    match = re.search(r'Accessible via: https://(.*?)/', start_msg)
-    if not match:
-        raise Exception(
-            "Passenger did not report endpoint: {}".format(start_msg))
-    my_api_host = match.group(1)
+    my_api_host = "127.0.0.1:"+str(port)
     os.environ['ARVADOS_API_HOST'] = my_api_host
 
     # Make sure the server has written its pid file and started

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list