Setting queue.spec.capacity.cpu field empty in child queue will result in an error: Failed to check queue's hierarchical structure

[yccharles]

Description

1. build hierarchical queue like this: root's child queue is root-sub1. root-sub1's child queue is root-sub1-sub1

root 
 |
root-sub1 ( capacity.cpu = 100,  mem=10Gi)
 |
root-sub1-sub1 ( capacity.mem = 5Gi)

2. create vcjob with queue = root-sub1-sub1 .

we can see error log in vc-scheduler : Failed to check queue's hierarchical structure, error.....

Steps to reproduce the issue

1. create an queeu with queue.spec.capacity.cpu field empty .
2. create a vcjob .

Describe the results you received and expected

Because of there is a default values when cpu/mem is null . Cpu would be setting to MaxFloat64.

volcano/pkg/scheduler/plugins/capacity/capacity.go

Line 304 in 0966fd5

attr.capability.MilliCPU = math.MaxFloat64

So, should we reject creation a queue in admission-webhook when queue.spec.capacity.cpu/mem is null ?

What version of Volcano are you using?

master

Any other relevant information

No response

[JesseStutler]

Thank you very much for discovering these problems before our version is released. The hierarchical queue has just been developed and there may indeed be some bugs. We still need to continue iterating. Do you have any requirements for using the hierarchical queue?

[JesseStutler]

Actually the capability setting is this line if capability is nil:

volcano/pkg/scheduler/plugins/capacity/capacity.go

Line 599 in 0966fd5

attr.capability.MilliCPU = math.MaxFloat64

[JesseStutler]

@Monokaix We need to also carry this issue into v2.0 milestone.