diff --git a/Makefile b/Makefile index 3653dd73b..115f613e5 100755 --- a/Makefile +++ b/Makefile @@ -238,7 +238,7 @@ unit-test: go test -v -mod=vendor -timeout 30s "./pkg/..." -cover sanity-test: - cd test && go test -v -timeout 30s "./sanity/" -run TestSanity + cd test && go test -mod=readonly -v -timeout 30s "./sanity/" -run TestSanity build-e2e-test: cd test && go build -o ../bin/e2e-test-ci ./e2e diff --git a/pkg/cloud_provider/clientset/clientset.go b/pkg/cloud_provider/clientset/clientset.go index ac9903093..2862b7148 100644 --- a/pkg/cloud_provider/clientset/clientset.go +++ b/pkg/cloud_provider/clientset/clientset.go @@ -86,6 +86,7 @@ func (c *Clientset) ConfigureNodeLister(nodeName string) { nodeObj.Status = corev1.NodeStatus{} nodeObj.ObjectMeta.Annotations = nil nodeObj.ObjectMeta.Labels = newLabels + return obj, nil } diff --git a/pkg/cloud_provider/clientset/fake.go b/pkg/cloud_provider/clientset/fake.go index dc59aca36..b7ed449d1 100644 --- a/pkg/cloud_provider/clientset/fake.go +++ b/pkg/cloud_provider/clientset/fake.go @@ -36,6 +36,7 @@ func NewFakeClientset() *FakeClientset { // Default setting for most unit tests is pod doesn't use host network & workload identity is enabled on the node fakeClientSet.CreatePod( /*hostNetworkEnabled */ false) fakeClientSet.CreateNode( /* isWorkloadIdentityEnabledOnNode */ true) + return fakeClientSet } diff --git a/pkg/csi_driver/node_test.go b/pkg/csi_driver/node_test.go index b5bb148e5..769a302a3 100644 --- a/pkg/csi_driver/node_test.go +++ b/pkg/csi_driver/node_test.go @@ -193,6 +193,7 @@ func TestNodePublishVolume(t *testing.T) { } func TestNodePublishVolumeWIDisabledOnNode(t *testing.T) { + t.Parallel() defaultPerm := os.FileMode(0o750) + os.ModeDir // Setup mount target path tmpDir := "/tmp/var/lib/kubelet/pods/test-pod-id/volumes/kubernetes.io~csi/"