15370: Temporarily disable some docker tests.

See https://dev.arvados.org/issues/15370#note-31

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>

diff --git a/cmd/arvados-package/build_test.go b/cmd/arvados-package/build_test.go
index 75e8f7604e8cbbdb22b4ebbd575c0ff5bf9ca537..0f6288360c02b8934a8dddb491d17aa9edf69c80 100644 (file)
--- a/cmd/arvados-package/build_test.go
+++ b/cmd/arvados-package/build_test.go
@@ -29,7 +29,10 @@ 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()
        defer os.RemoveAll(tmpdir)
        err := os.Chmod(tmpdir, 0755)

diff --git a/lib/crunchrun/integration_test.go b/lib/crunchrun/integration_test.go
index ec0893728328153074aba4aecebc6d940663f363..2ba7556cb5cd7c0fcc8633c537eac0929615189c 100644 (file)
--- a/lib/crunchrun/integration_test.go
+++ b/lib/crunchrun/integration_test.go
@@ -249,6 +249,9 @@ 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)