Run tests on GitHub Actions

While GitHub Actions' environment doesn't have KVM, we can run
a few tests on there to make sure that the SDK works with different
Go versions.

It doesn't run Firecracker, but running some tests is better than
running no tests.

Signed-off-by: Kazuyoshi Kato <katokazu@amazon.com>

Author: Kazuyoshi Kato

.github/workflows/go.yml

@@ -25,5 +25,5 @@ jobs:
     - name: Check out code into the Go module directory
       uses: actions/checkout@v2
 
-    - name: Build
-      run: make build
+    - name: Test
+      run: make test EXTRAGOARGS=-v

Makefile

@@ -48,16 +48,12 @@ all: build
 
 test: all-tests
 
-unit-tests: check-kvm $(testdata_objects)
+unit-tests: $(testdata_objects)
 	DISABLE_ROOT_TESTS=$(DISABLE_ROOT_TESTS) go test -short ./... $(EXTRAGOARGS)
 
-all-tests: check-kvm $(testdata_objects)
+all-tests: $(testdata_objects)
 	DISABLE_ROOT_TESTS=$(DISABLE_ROOT_TESTS) go test ./... $(EXTRAGOARGS)
 
-check-kvm:
-	@test -w /dev/kvm || \
-		(echo "In order to run firecracker, $(shell whoami) must have write permission to /dev/kvm"; false)
-
 generate build clean::
 	go $@ $(EXTRAGOARGS)
 

fctesting/utils.go

@@ -14,9 +14,13 @@
 package fctesting
 
 import (
+	"fmt"
 	"os"
+	"os/user"
 	"testing"
 
+	"golang.org/x/sys/unix"
+
 	log "github.com/sirupsen/logrus"
 )
 
@@ -31,6 +35,23 @@ func init() {
 	}
 }
 
+func RequiresKVM(t testing.TB) {
+	accessErr := unix.Access("/dev/kvm", unix.W_OK)
+	if accessErr != nil {
+		var name string
+		u, err := user.Current()
+		if err == nil {
+			name = u.Name
+		}
+
+		// On GitHub Actions, user.Current() doesn't return an error, but the name is "".
+		if name == "" {
+			name = fmt.Sprintf("uid=%d", os.Getuid())
+		}
+		t.Skipf("/dev/kvm is not writable from %s: %s", name, accessErr)
+	}
+}
+
 // RequiresRoot will ensure that tests that require root access are actually
 // root. In addition, this will skip root tests if the DISABLE_ROOT_TESTS is
 // set to true

machine_test.go

@@ -122,9 +122,7 @@ func TestNewMachine(t *testing.T) {
 }
 
 func TestJailerMicroVMExecution(t *testing.T) {
-	if testing.Short() {
-		t.Skip()
-	}
+	fctesting.RequiresKVM(t)
 	fctesting.RequiresRoot(t)
 
 	logPath := filepath.Join(testDataLogPath, "TestJailerMicroVMExecution")
@@ -284,9 +282,7 @@ func TestJailerMicroVMExecution(t *testing.T) {
 }
 
 func TestMicroVMExecution(t *testing.T) {
-	if testing.Short() {
-		t.Skip()
-	}
+	fctesting.RequiresKVM(t)
 
 	var nCpus int64 = 2
 	cpuTemplate := models.CPUTemplate(models.CPUTemplateT2)
@@ -398,8 +394,11 @@ func TestMicroVMExecution(t *testing.T) {
 }
 
 func TestStartVMM(t *testing.T) {
+	fctesting.RequiresKVM(t)
+
 	socketPath, cleanup := makeSocketPath(t)
 	defer cleanup()
+
 	cfg := Config{
 		SocketPath: socketPath,
 	}
@@ -437,6 +436,8 @@ func TestStartVMM(t *testing.T) {
 }
 
 func TestLogAndMetrics(t *testing.T) {
+	fctesting.RequiresKVM(t)
+
 	tests := []struct {
 		logLevel string
 		quiet    bool
@@ -523,6 +524,8 @@ func testLogAndMetrics(t *testing.T, logLevel string) string {
 }
 
 func TestStartVMMOnce(t *testing.T) {
+	fctesting.RequiresKVM(t)
+
 	socketPath, cleanup := makeSocketPath(t)
 	defer cleanup()
 
@@ -1252,6 +1255,7 @@ func TestCaptureFifoToFile_leak(t *testing.T) {
 }
 
 func TestWait(t *testing.T) {
+	fctesting.RequiresKVM(t)
 	fctesting.RequiresRoot(t)
 
 	cases := []struct {
@@ -1535,6 +1539,7 @@ func TestSignalForwarding(t *testing.T) {
 }
 
 func TestPauseResume(t *testing.T) {
+	fctesting.RequiresKVM(t)
 	fctesting.RequiresRoot(t)
 
 	dir, err := ioutil.TempDir("", t.Name())
@@ -1646,6 +1651,7 @@ func TestPauseResume(t *testing.T) {
 }
 
 func TestCreateSnapshot(t *testing.T) {
+	fctesting.RequiresKVM(t)
 	fctesting.RequiresRoot(t)
 
 	dir, err := ioutil.TempDir("", t.Name())