Fix vet

aws · Aug 17, 2023 · c94d236 · c94d236
1 parent 54dd25a
commit c94d236
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/.github/workflows/pr-tests.yaml b/.github/workflows/pr-tests.yaml
@@ -33,6 +33,6 @@ jobs:
       - name: Build
         run: make build-linux
       - name: Unit test
-        run: make unit-test
+        run: sudo make unit-test
       - name: Upload code coverage
         uses: codecov/codecov-action@v3
diff --git a/pkg/tc/tc_test.go b/pkg/tc/tc_test.go
@@ -74,7 +74,10 @@ func unmount_bpf_fs() error {
 func setupTest(interfaceNames []string, t *testing.T) {
 	mount_bpf_fs()
 	for _, interfaceName := range interfaceNames {
-		if err := netlink.LinkAdd(&netlink.Ifb{netlink.LinkAttrs{Name: interfaceName}}); err != nil {
+		linkAttr := netlink.LinkAttrs{Name: interfaceName}
+		linkIFB := netlink.Ifb{}
+		linkIFB.LinkAttrs = linkAttr
+		if err := netlink.LinkAdd(&linkIFB); err != nil {
 			assert.NoError(t, err)
 		}
 	}
@@ -84,7 +87,10 @@ func teardownTest(interfaceNames []string, t *testing.T) {
 	unmount_bpf_fs()
 	//Cleanup link
 	for _, interfaceName := range interfaceNames {
-		if err := netlink.LinkDel(&netlink.Ifb{netlink.LinkAttrs{Name: interfaceName}}); err != nil {
+		linkAttr := netlink.LinkAttrs{Name: interfaceName}
+		linkIFB := netlink.Ifb{}
+		linkIFB.LinkAttrs = linkAttr
+		if err := netlink.LinkDel(&linkIFB); err != nil {
 			assert.NoError(t, err)
 		}
 	}