Browse Source

libnetwork: fix TestParallel "bad file descriptor"

When running inside a container, testns == origns. Consequently, closing
testns causes the deferred netns.Set(origns) call to fail. Stop closing
the aliased original namespace handle.

Signed-off-by: Cory Snider <csnider@mirantis.com>
Cory Snider 2 years ago
parent
commit
f39b83e232
1 changed files with 3 additions and 1 deletions
  1. 3 1
      libnetwork/libnetwork_linux_test.go

+ 3 - 1
libnetwork/libnetwork_linux_test.go

@@ -1046,7 +1046,9 @@ func runParallelTests(t *testing.T, thrNumber int) {
 			<-thrdone
 		}
 
-		testns.Close()
+		if testns != origins {
+			testns.Close()
+		}
 		if err := net2.Delete(); err != nil {
 			t.Fatal(err)
 		}