Merge pull request #16783 from justinsb/cleanup_nodeup_kubeapierver

cleanup: better error messages for kube-apiserver healthcheck manifest
kubernetes · Aug 27, 2024 · 4273384 · 4273384
2 parents 722e4b1 + 733cce1
commit 4273384
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/nodeup/pkg/model/kube_apiserver_healthcheck.go b/nodeup/pkg/model/kube_apiserver_healthcheck.go
@@ -42,12 +42,12 @@ func (b *KubeAPIServerBuilder) addHealthcheckSidecar(ctx context.Context, pod *c
 
 	data, err := p.ReadFile(ctx)
 	if err != nil {
-		return fmt.Errorf("error reading kube-apiserver-healthcheck manifest %s: %v", manifest.Path, err)
+		return fmt.Errorf("error reading kube-apiserver-healthcheck manifest %s: %w", p, err)
 	}
 
 	sidecar := &corev1.Pod{}
 	if err := yaml.Unmarshal(data, sidecar); err != nil {
-		return fmt.Errorf("error parsing kube-apiserver-healthcheck manifest %s: %v", manifest.Path, err)
+		return fmt.Errorf("error parsing kube-apiserver-healthcheck manifest %s: %w", p, err)
 	}
 
 	// Quick-and-dirty merge of the fields we care about