[arvados] updated: 2.1.0-2558-g13f6d4570

git repository hosting git at public.arvados.org
Mon May 30 13:56:47 UTC 2022


Summary of changes:
 cmd/arvados-package/build_test.go | 2 --
 lib/crunchrun/integration_test.go | 3 ---
 2 files changed, 5 deletions(-)

       via  13f6d45704efc68ca8419e8917376aa44fdee1be (commit)
      from  3fa6aa4043286ad61e5f29c136d3cc2942e8750d (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 13f6d45704efc68ca8419e8917376aa44fdee1be
Author: Tom Clegg <tom at curii.com>
Date:   Mon May 30 09:56:23 2022 -0400

    15370: Re-enable docker tests.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/cmd/arvados-package/build_test.go b/cmd/arvados-package/build_test.go
index 0f6288360..7130e3394 100644
--- a/cmd/arvados-package/build_test.go
+++ b/cmd/arvados-package/build_test.go
@@ -29,8 +29,6 @@ func (s *BuildSuite) TestBuildAndInstall(c *check.C) {
 		c.Skip("skipping docker tests in short mode")
 	} else if _, err := exec.Command("docker", "info").CombinedOutput(); err != nil {
 		c.Skip("skipping docker tests because docker is not available")
-	} else if os.Getenv("ENABLE_DOCKER_TESTS") == "" {
-		c.Skip("docker tests temporarily disabled if ENABLE_DOCKER_TESTS is not set, see https://dev.arvados.org/issues/15370#note-31")
 	}
 
 	tmpdir := c.MkDir()
diff --git a/lib/crunchrun/integration_test.go b/lib/crunchrun/integration_test.go
index 96ba576a4..ce92a9b80 100644
--- a/lib/crunchrun/integration_test.go
+++ b/lib/crunchrun/integration_test.go
@@ -249,9 +249,6 @@ func (s *integrationSuite) testRunTrivialContainer(c *C) {
 	if err := exec.Command("which", s.engine).Run(); err != nil {
 		c.Skip(fmt.Sprintf("%s: %s", s.engine, err))
 	}
-	if s.engine == "docker" && os.Getenv("ENABLE_DOCKER_TESTS") == "" {
-		c.Skip("docker tests temporarily disabled if ENABLE_DOCKER_TESTS is not set, see https://dev.arvados.org/issues/15370#note-31")
-	}
 	s.cr.Command = []string{"sh", "-c", "cat /mnt/in/inputfile >/mnt/out/inputfile && cat /mnt/json >/mnt/out/json && ! touch /mnt/in/shouldbereadonly && mkdir /mnt/out/emptydir"}
 	s.setup(c)
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list