diff --git a/pkg/action/hooks.go b/pkg/action/hooks.go index 4bffb6ae0..bbf59a220 100644 --- a/pkg/action/hooks.go +++ b/pkg/action/hooks.go @@ -22,7 +22,6 @@ import ( "github.com/pkg/errors" - "helm.sh/helm/v3/pkg/kube" "helm.sh/helm/v3/pkg/release" helmtime "helm.sh/helm/v3/pkg/time" ) @@ -139,10 +138,8 @@ func (cfg *Configuration) deleteHookByPolicy(h *release.Hook, policy release.Hoo } //wait for resources until they are deleted to avoid conflicts - if kubeClient, ok := cfg.KubeClient.(kube.InterfaceExt); ok { - if err := kubeClient.WaitForDelete(resources, timeout); err != nil { - return err - } + if err := cfg.KubeClient.WaitForDelete(resources, timeout); err != nil { + return err } } return nil diff --git a/pkg/action/uninstall.go b/pkg/action/uninstall.go index ac0c4fee8..f5b26ba75 100644 --- a/pkg/action/uninstall.go +++ b/pkg/action/uninstall.go @@ -131,10 +131,8 @@ func (u *Uninstall) Run(name string) (*release.UninstallReleaseResponse, error) res.Info = kept if u.Wait { - if kubeClient, ok := u.cfg.KubeClient.(kube.InterfaceExt); ok { - if err := kubeClient.WaitForDelete(deletedResources, u.Timeout); err != nil { - errs = append(errs, err) - } + if err := u.cfg.KubeClient.WaitForDelete(deletedResources, u.Timeout); err != nil { + errs = append(errs, err) } } diff --git a/pkg/kube/interface.go b/pkg/kube/interface.go index ce42ed950..b89434d10 100644 --- a/pkg/kube/interface.go +++ b/pkg/kube/interface.go @@ -70,12 +70,7 @@ type Interface interface { // IsReachable checks whether the client is able to connect to the cluster. IsReachable() error -} -// InterfaceExt is introduced to avoid breaking backwards compatibility for Interface implementers. -// -// TODO Helm 4: Remove InterfaceExt and integrate its method(s) into the Interface. -type InterfaceExt interface { // WaitForDelete wait up to the given timeout for the specified resources to be deleted. WaitForDelete(resources ResourceList, timeout time.Duration) error } @@ -111,6 +106,5 @@ type InterfaceResources interface { } var _ Interface = (*Client)(nil) -var _ InterfaceExt = (*Client)(nil) var _ InterfaceDeletionPropagation = (*Client)(nil) var _ InterfaceResources = (*Client)(nil)