mirror of
https://github.com/k3s-io/kubernetes.git
synced 2025-08-13 22:05:59 +00:00
node: device-plugin: e2e: Annotate device check with error message
With this change the error message are more helpful and easier to troubleshoot in case of test failures. Signed-off-by: Swati Sehgal <swsehgal@redhat.com>
This commit is contained in:
parent
5c4f397361
commit
5ab4ba6205
@ -166,7 +166,7 @@ func testDevicePlugin(f *framework.Framework, pluginSockDir string) {
|
|||||||
deviceIDRE := "stub devices: (Dev-[0-9]+)"
|
deviceIDRE := "stub devices: (Dev-[0-9]+)"
|
||||||
devID1, err := parseLog(ctx, f, pod1.Name, pod1.Name, deviceIDRE)
|
devID1, err := parseLog(ctx, f, pod1.Name, pod1.Name, deviceIDRE)
|
||||||
framework.ExpectNoError(err, "getting logs for pod %q", pod1.Name)
|
framework.ExpectNoError(err, "getting logs for pod %q", pod1.Name)
|
||||||
gomega.Expect(devID1).To(gomega.Not(gomega.Equal("")))
|
gomega.Expect(devID1).To(gomega.Not(gomega.Equal("")), "pod1 requested a device but started successfully without")
|
||||||
|
|
||||||
v1alphaPodResources, err = getV1alpha1NodeDevices(ctx)
|
v1alphaPodResources, err = getV1alpha1NodeDevices(ctx)
|
||||||
framework.ExpectNoError(err)
|
framework.ExpectNoError(err)
|
||||||
@ -227,7 +227,7 @@ func testDevicePlugin(f *framework.Framework, pluginSockDir string) {
|
|||||||
deviceIDRE := "stub devices: (Dev-[0-9]+)"
|
deviceIDRE := "stub devices: (Dev-[0-9]+)"
|
||||||
devID1, err := parseLog(ctx, f, pod1.Name, pod1.Name, deviceIDRE)
|
devID1, err := parseLog(ctx, f, pod1.Name, pod1.Name, deviceIDRE)
|
||||||
framework.ExpectNoError(err, "getting logs for pod %q", pod1.Name)
|
framework.ExpectNoError(err, "getting logs for pod %q", pod1.Name)
|
||||||
gomega.Expect(devID1).To(gomega.Not(gomega.Equal("")))
|
gomega.Expect(devID1).To(gomega.Not(gomega.Equal("")), "pod1 requested a device but started successfully without")
|
||||||
|
|
||||||
pod1, err = e2epod.NewPodClient(f).Get(ctx, pod1.Name, metav1.GetOptions{})
|
pod1, err = e2epod.NewPodClient(f).Get(ctx, pod1.Name, metav1.GetOptions{})
|
||||||
framework.ExpectNoError(err)
|
framework.ExpectNoError(err)
|
||||||
@ -260,7 +260,7 @@ func testDevicePlugin(f *framework.Framework, pluginSockDir string) {
|
|||||||
devID1, err := parseLog(ctx, f, pod1.Name, pod1.Name, deviceIDRE)
|
devID1, err := parseLog(ctx, f, pod1.Name, pod1.Name, deviceIDRE)
|
||||||
framework.ExpectNoError(err, "getting logs for pod %q", pod1.Name)
|
framework.ExpectNoError(err, "getting logs for pod %q", pod1.Name)
|
||||||
|
|
||||||
gomega.Expect(devID1).To(gomega.Not(gomega.Equal("")))
|
gomega.Expect(devID1).To(gomega.Not(gomega.Equal("")), "pod1 requested a device but started successfully without")
|
||||||
|
|
||||||
pod1, err = e2epod.NewPodClient(f).Get(ctx, pod1.Name, metav1.GetOptions{})
|
pod1, err = e2epod.NewPodClient(f).Get(ctx, pod1.Name, metav1.GetOptions{})
|
||||||
framework.ExpectNoError(err)
|
framework.ExpectNoError(err)
|
||||||
@ -303,7 +303,7 @@ func testDevicePlugin(f *framework.Framework, pluginSockDir string) {
|
|||||||
devID2, err := parseLog(ctx, f, pod2.Name, pod2.Name, deviceIDRE)
|
devID2, err := parseLog(ctx, f, pod2.Name, pod2.Name, deviceIDRE)
|
||||||
framework.ExpectNoError(err, "getting logs for pod %q", pod2.Name)
|
framework.ExpectNoError(err, "getting logs for pod %q", pod2.Name)
|
||||||
|
|
||||||
gomega.Expect(devID1).To(gomega.Not(gomega.Equal(devID2)))
|
gomega.Expect(devID1).To(gomega.Not(gomega.Equal(devID2)), "pod2 requested a device but started successfully without")
|
||||||
})
|
})
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user