From e1e1a331e6c5c418ef7a124ef920d8858f07a9a0 Mon Sep 17 00:00:00 2001 From: Simon Alling Date: Wed, 1 Dec 2021 17:29:59 +0100 Subject: [PATCH] Move said comment to same line as code Signed-off-by: Simon Alling --- pkg/action/hooks.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pkg/action/hooks.go b/pkg/action/hooks.go index 22b69b191..db5ed0949 100644 --- a/pkg/action/hooks.go +++ b/pkg/action/hooks.go @@ -88,8 +88,7 @@ func (cfg *Configuration) execHook(rl *release.Release, hook release.HookEvent, h.LastRun.CompletedAt = helmtime.Now() if isTestHook(h) { - // "Why create a new client for each hook?" Because `cfg.RESTClientGetter` can be `nil` before the `for` loop, as detected by the `TestInstall/basic_install` and `TestInstallRelease` tests. - client, err := cfg.KubernetesClientSet() + client, err := cfg.KubernetesClientSet() // "Why create a new client for each hook?" Because `cfg.RESTClientGetter` can be `nil` before the `for` loop, as detected by the `TestInstall/basic_install` and `TestInstallRelease` tests. if err != nil { return errors.Wrapf(err, "unable to create Kubernetes client set to fetch pod logs") }