fix convertQuanToPercent func

[autumn0207]

the result of func convertQuanToPercent needs to be multiplied by 100

Signed-off-by: Tian Qiu 329508111@qq.com

[autumn0207]

Can you provide more details about the datas amount and percent you tested locally?

@Thor-wl OK.
I have placed a task that requires 1 core on a node with a capacity of 60 cores.

apiVersion: v1
kind: Pod
...
spec:
  containers:
  - command:
    - sleep
    - 10m
    image: nginx:latest
    imagePullPolicy: Always
    name: nginx
    resources:
      limits:
        cpu: "1"
      requests:
        cpu: "1"

Now the task is evicted by rescheduling plugin and I add logs to print the node cpu utilization and the return value of function convertQuanToPercent:

I0702 15:22:54.298356       1 node_utilization_util.go:175] 60.95572916666303 0.015625

Obviously the return value of function convertQuanToPercent for this task should be 1.5625 not 0.015625, otherwise the following codes will be meaningless:

utilization.utilization[v1.ResourceCPU] -= convertQuanToPercent(v1.ResourceCPU, &usedCPU, utilization.nodeInfo.Status.Capacity)

[Thor-wl]

Can you provide more details about the datas amount and percent you tested locally?

@Thor-wl OK. I have placed a task that requires 1 core on a node with a capacity of 60 cores.

apiVersion: v1
kind: Pod
...
spec:
  containers:
  - command:
    - sleep
    - 10m
    image: nginx:latest
    imagePullPolicy: Always
    name: nginx
    resources:
      limits:
        cpu: "1"
      requests:
        cpu: "1"

Now the task is evicted by rescheduling plugin and I add logs to print the node cpu utilization and the return value of function convertQuanToPercent:

I0702 15:22:54.298356       1 node_utilization_util.go:175] 60.95572916666303 0.015625

Obviously the return value of function convertQuanToPercent for this task should be 1.5625 not 0.015625, otherwise the following codes will be meaningless:

utilization.utilization[v1.ResourceCPU] -= convertQuanToPercent(v1.ResourceCPU, &usedCPU, utilization.nodeInfo.Status.Capacity)

Good job! I'm OK about that. Please format the code to pass the code verification and push again.

[Thor-wl]

@autumn0207 Can you cherry-pick all the bug fixes to branch release 1.6 as well? I'll launch a bug fix release later.
/lgtm
/approve

[volcano-sh-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Thor-wl

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• pkg/scheduler/OWNERS [Thor-wl]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[autumn0207]

@autumn0207 Can you cherry-pick all the bug fixes to branch release 1.6 as well? I'll launch a bug fix release later. /lgtm /approve

@Thor-wl
sure, already done, refer to #2330