Merge pull request #152 from kzys/gofmt

Run gofmt on BuildKite

Author: kzys

.buildkite/pipeline.yml

@@ -33,6 +33,9 @@ steps:
   # multiple go builds are downloading to the same cache.
   - wait
 
+  - label: gofmt -s
+    command: test -z $(gofmt -s -l .)
+
   - label: 'git log validation'
     command: './.buildkite/logcheck.sh'
 

drives.go

@@ -66,27 +66,28 @@ func (b DrivesBuilder) Build() []models.Drive {
 
 // WithDriveID sets the ID of the drive
 func WithDriveID(id string) DriveOpt {
-	return func(d *models.Drive)  {
+	return func(d *models.Drive) {
 		d.DriveID = String(id)
 	}
 }
 
 // WithReadOnly sets the drive read-only
 func WithReadOnly(flag bool) DriveOpt {
-	return func(d *models.Drive)  {
+	return func(d *models.Drive) {
 		d.IsReadOnly = Bool(flag)
 	}
 }
 
 // WithPartuuid sets the unique ID of the boot partition
 func WithPartuuid(uuid string) DriveOpt {
-	return func(d *models.Drive)  {
+	return func(d *models.Drive) {
 		d.Partuuid = uuid
 	}
 }
+
 // WithRateLimiter sets the rate limitter of the drive
 func WithRateLimiter(limiter models.RateLimiter) DriveOpt {
-	return func(d *models.Drive)  {
+	return func(d *models.Drive) {
 		d.RateLimiter = &limiter
 	}
-}
+}

machine_test.go

@@ -167,7 +167,7 @@ func TestJailerMicroVMExecution(t *testing.T) {
 			HtEnabled:   Bool(false),
 		},
 		Drives: []models.Drive{
-			models.Drive{
+			{
 				DriveID:      String("1"),
 				IsRootDevice: Bool(true),
 				IsReadOnly:   Bool(false),
@@ -705,7 +705,7 @@ func TestLogFiles(t *testing.T) {
 		Debug:           true,
 		KernelImagePath: filepath.Join(testDataPath, "vmlinux"), SocketPath: filepath.Join(testDataPath, "socket-path"),
 		Drives: []models.Drive{
-			models.Drive{
+			{
 				DriveID:      String("0"),
 				IsRootDevice: Bool(true),
 				IsReadOnly:   Bool(false),
@@ -898,7 +898,7 @@ func TestPID(t *testing.T) {
 			HtEnabled:   Bool(false),
 		},
 		Drives: []models.Drive{
-			models.Drive{
+			{
 				DriveID:      String("1"),
 				IsRootDevice: Bool(true),
 				IsReadOnly:   Bool(false),

network_test.go

@@ -153,13 +153,13 @@ func TestNetworkInterfacesValidation_CNI(t *testing.T) {
 
 func TestNetworkInterfacesValidation_MultipleStatic(t *testing.T) {
 	err := NetworkInterfaces([]NetworkInterface{
-		NetworkInterface{
+		{
 			StaticConfiguration: &StaticNetworkConfiguration{
 				MacAddress:  mockMacAddrString,
 				HostDevName: tapName,
 			},
 		},
-		NetworkInterface{
+		{
 			StaticConfiguration: &StaticNetworkConfiguration{
 				MacAddress:  "11:22:33:44:55:66",
 				HostDevName: "tap1",
@@ -172,7 +172,7 @@ func TestNetworkInterfacesValidation_MultipleStatic(t *testing.T) {
 func TestNetworkInterfacesValidationFails_MultipleCNI(t *testing.T) {
 	err := NetworkInterfaces([]NetworkInterface{
 		validCNIInterface,
-		NetworkInterface{
+		{
 			CNIConfiguration: &CNIConfiguration{
 				NetworkName: "something-else",
 				netNSPath:   "/a/different/netns",
@@ -185,7 +185,7 @@ func TestNetworkInterfacesValidationFails_MultipleCNI(t *testing.T) {
 func TestNetworkInterfacesValidationFails_IPWithMultiple(t *testing.T) {
 	err := NetworkInterfaces([]NetworkInterface{
 		validStaticNetworkInterface,
-		NetworkInterface{
+		{
 			StaticConfiguration: &StaticNetworkConfiguration{
 				MacAddress:  "11:22:33:44:55:66",
 				HostDevName: "tap1",
@@ -205,7 +205,7 @@ func TestNetworkInterfacesValidationFails_IPWithKernelArg(t *testing.T) {
 func TestNetworkInterfacesValidationFails_CNIWithMultiple(t *testing.T) {
 	err := NetworkInterfaces([]NetworkInterface{
 		validCNIInterface,
-		NetworkInterface{
+		{
 			StaticConfiguration: &StaticNetworkConfiguration{
 				MacAddress:  "11:22:33:44:55:66",
 				HostDevName: "tap1",
@@ -367,7 +367,7 @@ func startCNIMachine(t *testing.T,
 			HtEnabled:  Bool(true),
 		},
 		Drives: []models.Drive{
-			models.Drive{
+			{
 				DriveID:      String("1"),
 				IsRootDevice: Bool(true),
 				IsReadOnly:   Bool(false),