Bladeren bron

Fix test requirement for ARM

Correctly passes the DOCKER_ENGINE_GOARCH env var
to the testing environment

Also fixes logic for skipping a test if on ARM.

Signed-off-by: Christopher Jones <tophj@linux.vnet.ibm.com>
Christopher Jones 9 jaren geleden
bovenliggende
commit
d12f4babdd
2 gewijzigde bestanden met toevoegingen van 2 en 1 verwijderingen
  1. 1 0
      hack/make.sh
  2. 1 1
      integration-cli/requirements.go

+ 1 - 0
hack/make.sh

@@ -211,6 +211,7 @@ test_env() {
 	# use "env -i" to tightly control the environment variables that bleed into the tests
 	env -i \
 		DEST="$DEST" \
+		DOCKER_ENGINE_GOARCH="$DOCKER_ENGINE_GOARCH" \
 		DOCKER_GRAPHDRIVER="$DOCKER_GRAPHDRIVER" \
 		DOCKER_USERLANDPROXY="$DOCKER_USERLANDPROXY" \
 		DOCKER_HOST="$DOCKER_HOST" \

+ 1 - 1
integration-cli/requirements.go

@@ -30,7 +30,7 @@ var (
 		"Test requires a Linux daemon",
 	}
 	NotArm = testRequirement{
-		func() bool { return os.Getenv("DOCKER_ENGINE_GOARCH") == "arm" },
+		func() bool { return os.Getenv("DOCKER_ENGINE_GOARCH") != "arm" },
 		"Test requires a daemon not running on ARM",
 	}
 	SameHostDaemon = testRequirement{