kubeadm: change os.Remove to os.RemoveAll

This would not correctly clean out the tmp dir, since os.Remove doesn't recursively remove dirs, which could exist in the tmp dir.

kubeadm: change os.Remove to os.RemoveAll
011629fb · Derek McQuay · 9e427c88 · 011629fb
Unverified Commit 011629fb authored Feb 06, 2017 by Derek McQuay
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

certs_test.go cmd/kubeadm/app/phases/certs/certs_test.go +1 -1

No files found.
--- a/cmd/kubeadm/app/phases/certs/certs_test.go
+++ b/cmd/kubeadm/app/phases/certs/certs_test.go
@@ -32,7 +32,7 @@ func TestCreatePKIAssets(t *testing.T) {
 	if err != nil {
 		t.Fatalf("Couldn't create tmpdir")
 	}
-	defer os.Remove(tmpdir)
+	defer os.RemoveAll(tmpdir)

 	var tests = []struct {
 		cfg      *kubeadmapi.MasterConfiguration