From d317b01244a536cdfb79141c3e7c458a6fc55fd4 Mon Sep 17 00:00:00 2001 From: fazledyn-or Date: Thu, 14 Dec 2023 14:42:18 +0600 Subject: [PATCH] Fixed Inappropriate Logical Expression The length of `finalInterceptors` will always be a non-zero value. In that case, the comparison will always return a True value, making the comparison meaningless. Signed-off-by: fazledyn-or --- .../kubeletplugin/nonblockinggrpcserver.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/staging/src/k8s.io/dynamic-resource-allocation/kubeletplugin/nonblockinggrpcserver.go b/staging/src/k8s.io/dynamic-resource-allocation/kubeletplugin/nonblockinggrpcserver.go index e6a835d9695..b840c91c7df 100644 --- a/staging/src/k8s.io/dynamic-resource-allocation/kubeletplugin/nonblockinggrpcserver.go +++ b/staging/src/k8s.io/dynamic-resource-allocation/kubeletplugin/nonblockinggrpcserver.go @@ -84,9 +84,7 @@ func startGRPCServer(logger klog.Logger, grpcVerbosity int, interceptors []grpc. finalInterceptors = append(finalInterceptors, s.interceptor) } finalInterceptors = append(finalInterceptors, interceptors...) - if len(finalInterceptors) >= 0 { - opts = append(opts, grpc.ChainUnaryInterceptor(finalInterceptors...)) - } + opts = append(opts, grpc.ChainUnaryInterceptor(finalInterceptors...)) s.server = grpc.NewServer(opts...) for _, service := range services { service(s.server)