Skip to content

Commit

Permalink
fix: close client provider instead of Talos client in the upgrade module
Browse files Browse the repository at this point in the history
Otherwise it breaks Theila, which never closes Talos clients during
operation.

Signed-off-by: Artem Chernyshev <[email protected]>
  • Loading branch information
Unix4ever committed Jan 27, 2022
1 parent 0bf161d commit 831f65a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/cluster/kubernetes/talos_managed.go
Original file line number Diff line number Diff line change
Expand Up @@ -333,7 +333,7 @@ func getManifests(ctx context.Context, cluster UpgradeProvider) ([]*unstructured
return nil, err
}

defer talosclient.Close() //nolint:errcheck
defer cluster.Close() //nolint:errcheck

listClient, err := talosclient.Resources.List(ctx, k8s.ControlPlaneNamespaceName, k8s.ManifestType)
if err != nil {
Expand Down

0 comments on commit 831f65a

Please sign in to comment.