From b6adbbb227cfe5a097072eef37d4ef13d98000db Mon Sep 17 00:00:00 2001 From: Benoit Tigeot Date: Mon, 7 Apr 2025 16:50:00 +0200 Subject: [PATCH] Enforce error style with others Signed-off-by: Benoit Tigeot --- pkg/action/action.go | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/pkg/action/action.go b/pkg/action/action.go index 1993e6241..1996e0ff8 100644 --- a/pkg/action/action.go +++ b/pkg/action/action.go @@ -244,9 +244,6 @@ type RESTClientGetter interface { ToRESTMapper() (meta.RESTMapper, error) } -// DebugLog sets the logger that writes debug strings -type DebugLog func(format string, v ...interface{}) - // capabilities builds a Capabilities from discovery information. func (cfg *Configuration) getCapabilities() (*chartutil.Capabilities, error) { if cfg.Capabilities != nil { @@ -270,8 +267,8 @@ func (cfg *Configuration) getCapabilities() (*chartutil.Capabilities, error) { apiVersions, err := GetVersionSet(dc) if err != nil { if discovery.IsGroupDiscoveryFailedError(err) { - cfg.Log.Warn("The Kubernetes server has an orphaned API service. Server reports: %s", err) - cfg.Log.Warn("To fix this, kubectl delete apiservice ") + cfg.Log.Warn("the kubernetes server has an orphaned API service", "errors", err) + cfg.Log.Warn("to fix this, kubectl delete apiservice ") } else { return nil, errors.Wrap(err, "could not get apiVersions from Kubernetes") } @@ -370,7 +367,7 @@ func GetVersionSet(client discovery.ServerResourcesInterface) (chartutil.Version // recordRelease with an update operation in case reuse has been set. func (cfg *Configuration) recordRelease(r *release.Release) { if err := cfg.Releases.Update(r); err != nil { - cfg.Log.Warn("Failed to update release %s: %s", r.Name, err) + cfg.Log.Warn("failed to update release", "name", r.Name, "revision", r.Version, "error", err) } }