volcano-sh / volcano

A Cloud Native Batch System (Project under CNCF)
https://volcano.sh
Apache License 2.0
4.24k stars 971 forks source link

AllocateWithDynamicPVC ut fails probabilistically #2812

Open wangyang0616 opened 1 year ago

wangyang0616 commented 1 year ago

What happened: Allocate ut has the problem of probabilistic failure. Error probability is about 20%.

Details are as follows:

E0424 01:27:51.763183   21833 allocate.go:213] Failed to bind Task c1-p1 on n1 in Session f898e6de-93ba-41fa-836e-79df5c5741c7, err: error getting PVC "c1/pvc": could not find v1.PersistentVolumeClaim "c1/pvc"
--- FAIL: TestAllocateWithDynamicPVC (0.22s)
    --- FAIL: TestAllocateWithDynamicPVC/resource_not_match (0.11s)
        allocate_test.go:482: expected: map[c1/p1:[GetPodVolumes AllocateVolumes RevertVolumes]], got map[c1/p1:[GetPodVolumes] c1/p2:[GetPodVolumes AllocateVolumes RevertVolumes]] 
FAIL
FAIL    volcano.sh/volcano/pkg/scheduler/actions/allocate   0.557s

What you expected to happen: UT runs successfully.

How to reproduce it (as minimally and precisely as possible): make unit-test

Anything else we need to know?: The ut verification of the master is blocked by #2810. After fixing #2810, the problem will be exposed.

Environment:

stale[bot] commented 1 year ago

Hello 👋 Looks like there was no activity on this issue for last 90 days. Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗 If there will be no activity for 60 days, this issue will be closed (we can always reopen an issue if we need!).