[ARVADOS] updated: 1.3.0-2038-g2544aab0e

Git user git at public.arvados.org
Tue Dec 31 15:19:55 UTC 2019


Summary of changes:
 lib/dispatchcloud/container/queue_test.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

       via  2544aab0e538dac2c5456307173abf35f1611e6d (commit)
      from  fc0e284debae23e88e3e17cd3d7b58966eec6090 (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 2544aab0e538dac2c5456307173abf35f1611e6d
Author: Tom Clegg <tom at tomclegg.ca>
Date:   Tue Dec 31 10:18:18 2019 -0500

    Fix variable unintentionally shared by multiple goroutines in test.
    
    fixes #15953
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at tomclegg.ca>

diff --git a/lib/dispatchcloud/container/queue_test.go b/lib/dispatchcloud/container/queue_test.go
index 3e1cd5061..bb8226118 100644
--- a/lib/dispatchcloud/container/queue_test.go
+++ b/lib/dispatchcloud/container/queue_test.go
@@ -70,7 +70,7 @@ func (suite *IntegrationSuite) TestGetLockUnlockCancel(c *check.C) {
 		c.Check(ctr.UUID, check.Equals, uuid)
 
 		wg.Add(1)
-		go func() {
+		go func(uuid string) {
 			defer wg.Done()
 			err := cq.Unlock(uuid)
 			c.Check(err, check.NotNil)
@@ -99,7 +99,7 @@ func (suite *IntegrationSuite) TestGetLockUnlockCancel(c *check.C) {
 			c.Check(ctr.State, check.Equals, arvados.ContainerStateCancelled)
 			err = cq.Lock(uuid)
 			c.Check(err, check.NotNil)
-		}()
+		}(uuid)
 	}
 	wg.Wait()
 }

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list