use env instead of build directive to skip integration tests

Author: johnstcn

Makefile

@@ -39,4 +39,4 @@ test:
 
 .PHONY: test-integration
 test-integration:
-	go test -v -count=1 -tags=integration ./integration/
+	CODER_TEST_INTEGRATION=1 go test -v -count=1 ./integration/

integration/docker_test.go

@@ -1,6 +1,3 @@
-//go:build integration
-// +build integration
-
 package integration_test
 
 import (
@@ -24,6 +21,9 @@ import (
 
 func TestDocker(t *testing.T) {
 	t.Parallel()
+	if val, ok := os.LookupEnv("CODER_TEST_INTEGRATION"); !ok || val != "1" {
+		t.Skip("integration tests are skipped unless CODER_TEST_INTEGRATION=true")
+	}
 
 	// Dockerd just tests that dockerd can spin up and function correctly.
 	t.Run("Dockerd", func(t *testing.T) {

integration/gpu_test.go

@@ -2,6 +2,7 @@ package integration
 
 import (
 	"context"
+	"os"
 	"os/exec"
 	"slices"
 	"strings"
@@ -14,6 +15,10 @@ import (
 )
 
 func TestDocker_Nvidia(t *testing.T) {
+	t.Parallel()
+	if val, ok := os.LookupEnv("CODER_TEST_INTEGRATION"); !ok || val != "1" {
+		t.Skip("integration tests are skipped unless CODER_TEST_INTEGRATION=true")
+	}
 	// Only run this test if the nvidia container runtime is detected.
 	// Check if the nvidia runtime is available using `docker info`.
 	// The docker client doesn't expose this information so we need to fetch it ourselves.
@@ -22,6 +27,7 @@ func TestDocker_Nvidia(t *testing.T) {
 	}
 
 	t.Run("Ubuntu", func(t *testing.T) {
+		t.Parallel()
 		ctx, cancel := context.WithCancel(context.Background())
 		t.Cleanup(cancel)
 
@@ -39,6 +45,7 @@ func TestDocker_Nvidia(t *testing.T) {
 	})
 
 	t.Run("Redhat", func(t *testing.T) {
+		t.Parallel()
 		ctx, cancel := context.WithCancel(context.Background())
 		t.Cleanup(cancel)