github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-08-13 13:55:41 +00:00
Code Issues Projects Releases Wiki Activity

Refactor: avoid code duplication when converting custom resources

Browse Source 
Signed-off-by: Itamar Holder <iholder@redhat.com>
This commit is contained in:
Itamar Holder 2024-04-08 09:42:16 +03:00
parent 47207f9aad
commit b9109ab7e4
1 changed files with 14 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
pkg/kubelet/kubelet_pods.go
View File

@ -2094,6 +2094,18 @@ func (kl *Kubelet) convertToAPIContainerStatuses(pod *v1.Pod, podStatus *kubecon
if oldStatus.Resources == nil {
oldStatus.Resources = &v1.ResourceRequirements{}
}
convertCustomResources := func(inResources, outResources v1.ResourceList) {
for extendedResourceName, extendedResourceQuantity := range inResources {
if extendedResourceName == v1.ResourceCPU || extendedResourceName == v1.ResourceMemory ||
extendedResourceName == v1.ResourceStorage || extendedResourceName == v1.ResourceEphemeralStorage {
continue
}
outResources[extendedResourceName] = extendedResourceQuantity.DeepCopy()
}
}
// Convert Limits
if container.Resources.Limits != nil {
limits = make(v1.ResourceList)
@ -2111,14 +2123,7 @@ func (kl *Kubelet) convertToAPIContainerStatuses(pod *v1.Pod, podStatus *kubecon
limits[v1.ResourceEphemeralStorage] = ephemeralStorage.DeepCopy()
}
for extendedResourceName, extendedResourceQuantity := range container.Resources.Limits {
if extendedResourceName == v1.ResourceCPU || extendedResourceName == v1.ResourceMemory ||
extendedResourceName == v1.ResourceStorage || extendedResourceName == v1.ResourceEphemeralStorage {
continue
}
limits[extendedResourceName] = extendedResourceQuantity.DeepCopy()
}
convertCustomResources(container.Resources.Limits, limits)
}
// Convert Requests
if status.AllocatedResources != nil {
@ -2135,14 +2140,7 @@ func (kl *Kubelet) convertToAPIContainerStatuses(pod *v1.Pod, podStatus *kubecon
requests[v1.ResourceEphemeralStorage] = ephemeralStorage.DeepCopy()
}
for extendedResourceName, extendedResourceQuantity := range status.AllocatedResources {
if extendedResourceName == v1.ResourceCPU || extendedResourceName == v1.ResourceMemory ||
extendedResourceName == v1.ResourceStorage || extendedResourceName == v1.ResourceEphemeralStorage {
continue
}
requests[extendedResourceName] = extendedResourceQuantity.DeepCopy()
}
convertCustomResources(status.AllocatedResources, requests)
}
//TODO(vinaykul,derekwaynecarr,InPlacePodVerticalScaling): Update this to include extended resources in
// addition to CPU, memory, ephemeral storage. Add test case for extended resources.