From 844662e7fabb7e50ebe12af95dac30dda1488c2f Mon Sep 17 00:00:00 2001 From: Patrick Ohly Date: Tue, 31 Aug 2021 08:39:55 +0200 Subject: [PATCH] kubelet: use generic ephemeral volume helper functions The name concatenation and ownership check were originally considered small enough to not warrant dedicated functions, but the intent of the code is more readable with them. --- pkg/kubelet/kubelet_pods.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/kubelet/kubelet_pods.go b/pkg/kubelet/kubelet_pods.go index 115161003cf..0917f2bf873 100644 --- a/pkg/kubelet/kubelet_pods.go +++ b/pkg/kubelet/kubelet_pods.go @@ -39,6 +39,7 @@ import ( "k8s.io/apimachinery/pkg/util/sets" utilvalidation "k8s.io/apimachinery/pkg/util/validation" utilfeature "k8s.io/apiserver/pkg/util/feature" + "k8s.io/component-helpers/storage/ephemeral" runtimeapi "k8s.io/cri-api/pkg/apis/runtime/v1alpha2" "k8s.io/klog/v2" podutil "k8s.io/kubernetes/pkg/api/v1/pod" @@ -2040,7 +2041,7 @@ func (kl *Kubelet) hasHostMountPVC(pod *v1.Pod) bool { if !utilfeature.DefaultFeatureGate.Enabled(features.GenericEphemeralVolume) { continue } - pvcName = pod.Name + "-" + volume.Name + pvcName = ephemeral.VolumeClaimName(pod, &volume) default: continue }