Skip to content

Commit edfa9a5

Browse files
committed
added WithFlaky() to the device plugin test case: supports extended resources together with ResourceClaim
1 parent 790393a commit edfa9a5

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

test/e2e/dra/dra.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -921,9 +921,14 @@ var _ = framework.SIGDescribe("node")(framework.WithLabel("DRA"), func() {
921921
}).WithTimeout(f.Timeouts.PodDelete).Should(gomega.HaveField("Status.Allocation", (*resourceapi.AllocationResult)(nil)))
922922
})
923923

924+
// https://github.com/kubernetes/kubernetes/issues/133488
925+
// It conflicts with "must run pods with extended resource on dra nodes and device plugin nodes" test case,
926+
// because device plugin does not clean up the extended resource "example.com/resource", and kubelet still
927+
// keeps "example.com/resource" : 0 in node.status.Capacity.
928+
// add WithFlaky to filter out the following test until we can clean up the leaked "example.com/resource" in node.status.
924929
if withKubelet {
925930
// Serial because the example device plugin can only be deployed with one instance at a time.
926-
f.It("supports extended resources together with ResourceClaim", f.WithSerial(), func(ctx context.Context) {
931+
f.It("supports extended resources together with ResourceClaim", f.WithSerial(), f.WithFlaky(), func(ctx context.Context) {
927932
extendedResourceName := deployDevicePlugin(ctx, f, nodes.NodeNames[0:1])
928933

929934
pod := b.PodExternal()

0 commit comments

Comments
 (0)