From 8688368753dbcaf9df71d730bf3250da66d5695e Mon Sep 17 00:00:00 2001 From: Francesco Romani Date: Tue, 4 Jun 2024 14:21:57 +0200 Subject: [PATCH] [KNI] improve klogr compatibility the klogr wrapper should not serialize args but rather pass them through, because this is the behavior most similar to what we will have in 4.16 and onwards Signed-off-by: Francesco Romani (cherry picked from commit ad71d2056ae67db27a217d5f2bbfca450eb427a7) --- pkg/noderesourcetopology/filter.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/noderesourcetopology/filter.go b/pkg/noderesourcetopology/filter.go index 225b311bc..fd09db929 100644 --- a/pkg/noderesourcetopology/filter.go +++ b/pkg/noderesourcetopology/filter.go @@ -78,7 +78,7 @@ func singleNUMAContainerLevelHandler(pod *v1.Pod, zones topologyv1alpha2.ZoneLis // subtract the resources requested by the container from the given NUMA. // this is necessary, so we won't allocate the same resources for the upcoming containers - clh := klogr.New().WithValues("logID", logID, "node", nodeInfo.Node().Name) + clh := klogr.NewWithOptions(klogr.WithFormat(klogr.FormatKlog)).WithValues("logID", logID, "node", nodeInfo.Node().Name) err := subtractResourcesFromNUMANodeList(clh, nodes, numaID, qos, container.Resources.Requests, logID) if err != nil { // this is an internal error which should never happen