fix: vgpu metrics not update when pod deleted

[googs1025]

fix: #3605

When a pod is deleted, we need to re-update the metrics

// SubResource frees the gpu hold by the pod
func (gs *GPUDevices) SubResource(pod *v1.Pod) {
	ids, ok := pod.Annotations[AssignedIDsAnnotations]
	if !ok {
		return
	}
	podDev := decodePodDevices(ids)
	for _, val := range podDev {
		for _, deviceused := range val {
			if gs == nil {
				break
			}
			for index, gsdevice := range gs.Device {
				if gsdevice.UUID == deviceused.UUID {
					klog.V(4).Infoln("VGPU subsctracting pod", pod.Name, "device", deviceused)
					gs.Device[index].UsedMem -= uint(deviceused.Usedmem)
					gs.Device[index].UsedNum--
					gs.Device[index].UsedCore -= uint(deviceused.Usedcores)
				}
			}
		}
	}
	gs.GetStatus()
}

[volcano-sh-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign k82cn
You can assign the PR to them by writing /assign @k82cn in a comment when ready.

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

Needs approval from an approver in each of these files:

• pkg/scheduler/OWNERS

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

[googs1025]

/assign @archlitchi @Monokaix

[googs1025]

I haven't tested it yet, I will do it in the next few days

[googs1025]

/kind bug

[Monokaix]

gs.GetStatus() has a get semantics but actually does an update operation，I think change to updateStatus is better，what do you think？ @archlitchi

[googs1025]

gs.GetStatus() has a get semantics but actually does an update operation，I think change to updateStatus is better，what do you think？ @archlitchi

+1
IMO, This method name is indeed very misleading. Does it need to be modified?

[googs1025]

@archlitchi /PTAL