[ARVADOS] updated: 2.1.0-1835-g8323dcbfe

Git user git at public.arvados.org
Mon Feb 7 16:07:51 UTC 2022


Summary of changes:
 tools/arvbox/lib/arvbox/docker/common.sh               | 7 ++++---
 tools/arvbox/lib/arvbox/docker/service/sdk/run-service | 4 ++--
 2 files changed, 6 insertions(+), 5 deletions(-)

       via  8323dcbfe4833a0303286991dc64cdb0b0c0fa60 (commit)
      from  40aff12f447f1d4c314972e85531b4a9bb835d95 (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 8323dcbfe4833a0303286991dc64cdb0b0c0fa60
Author: Peter Amstutz <peter.amstutz at curii.com>
Date:   Mon Feb 7 11:07:22 2022 -0500

    18657: Only get lock on gem install / bundle install
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>

diff --git a/tools/arvbox/lib/arvbox/docker/common.sh b/tools/arvbox/lib/arvbox/docker/common.sh
index 54379a5ed..3ad82b3f8 100644
--- a/tools/arvbox/lib/arvbox/docker/common.sh
+++ b/tools/arvbox/lib/arvbox/docker/common.sh
@@ -62,7 +62,8 @@ else
 fi
 
 run_bundler() {
-    /var/lib/arvados/bin/gem install --no-document bundler:$BUNDLER_VERSION
+    GEMLOCK=/var/lib/arvados/lib/ruby/gems/gems.lock
+    flock $GEMLOCK /var/lib/arvados/bin/gem install --no-document bundler:$BUNDLER_VERSION
     if test -f Gemfile.lock ; then
         frozen=--frozen
     else
@@ -73,8 +74,8 @@ run_bundler() {
 	# If present, use the one associated with rails workbench or API
 	BUNDLER=$PWD/bin/bundle
     fi
-    if ! $BUNDLER install --verbose --local --no-deployment $frozen "$@" ; then
-        $BUNDLER install --verbose --no-deployment $frozen "$@"
+    if ! flock $GEMLOCK $BUNDLER install --verbose --local --no-deployment $frozen "$@" ; then
+        flock $GEMLOCK $BUNDLER install --verbose --no-deployment $frozen "$@"
     fi
 }
 
diff --git a/tools/arvbox/lib/arvbox/docker/service/sdk/run-service b/tools/arvbox/lib/arvbox/docker/service/sdk/run-service
index d98afe823..c5bf852ab 100755
--- a/tools/arvbox/lib/arvbox/docker/service/sdk/run-service
+++ b/tools/arvbox/lib/arvbox/docker/service/sdk/run-service
@@ -15,8 +15,8 @@ download_cache = /var/lib/pip
 EOF
 
 cd /usr/src/arvados/sdk/cli
-run_bundler --binstubs
-ln -sf /usr/src/arvados/sdk/cli/bin/arv /usr/local/bin/arv
+run_bundler --binstubs=binstubs
+ln -sf /usr/src/arvados/sdk/cli/binstubs/arv /usr/local/bin/arv
 
 export PYCMD=python3
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list