De-flake Volume GCEPD test.
Force detach of PD prior to deletion.
This commit is contained in:
		| @@ -47,14 +47,14 @@ import ( | |||||||
| 	"strings" | 	"strings" | ||||||
| 	"time" | 	"time" | ||||||
|  |  | ||||||
|  | 	. "github.com/onsi/ginkgo" | ||||||
|  | 	. "github.com/onsi/gomega" | ||||||
| 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" | 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" | ||||||
|  | 	"k8s.io/apimachinery/pkg/types" | ||||||
| 	"k8s.io/kubernetes/pkg/api/v1" | 	"k8s.io/kubernetes/pkg/api/v1" | ||||||
| 	"k8s.io/kubernetes/pkg/client/clientset_generated/clientset" | 	"k8s.io/kubernetes/pkg/client/clientset_generated/clientset" | ||||||
| 	"k8s.io/kubernetes/pkg/cloudprovider/providers/vsphere" | 	"k8s.io/kubernetes/pkg/cloudprovider/providers/vsphere" | ||||||
| 	"k8s.io/kubernetes/test/e2e/framework" | 	"k8s.io/kubernetes/test/e2e/framework" | ||||||
|  |  | ||||||
| 	. "github.com/onsi/ginkgo" |  | ||||||
| 	. "github.com/onsi/gomega" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| func DeleteCinderVolume(name string) error { | func DeleteCinderVolume(name string) error { | ||||||
| @@ -518,7 +518,11 @@ var _ = framework.KubeDescribe("Volumes [Volume]", func() { | |||||||
| 			volumeName, err := framework.CreatePDWithRetry() | 			volumeName, err := framework.CreatePDWithRetry() | ||||||
| 			Expect(err).NotTo(HaveOccurred()) | 			Expect(err).NotTo(HaveOccurred()) | ||||||
| 			defer func() { | 			defer func() { | ||||||
| 				framework.ExpectNoError(framework.DeletePDWithRetry(volumeName)) | 				// - Get NodeName from the pod spec to which the volume is mounted. | ||||||
|  | 				// - Force detach and delete. | ||||||
|  | 				pod, err := f.PodClient().Get(config.Prefix+"-client", metav1.GetOptions{}) | ||||||
|  | 				Expect(err).NotTo(HaveOccurred(), "Failed getting pod %q.", config.Prefix+"-client") | ||||||
|  | 				detachAndDeletePDs(volumeName, []types.NodeName{types.NodeName(pod.Spec.NodeName)}) | ||||||
| 			}() | 			}() | ||||||
|  |  | ||||||
| 			defer func() { | 			defer func() { | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Jon Cope
					Jon Cope