Skip to content

Commit

Permalink
Add context to remote clients
Browse files Browse the repository at this point in the history
  • Loading branch information
JoelSpeed committed Feb 14, 2020
1 parent 808c359 commit 223f30c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions controllers/machinehealthcheck_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ func (r *MachineHealthCheckReconciler) reconcile(ctx context.Context, cluster *c
logger = logger.WithValues("cluster", cluster.Name)

// Create client for target cluster
clusterClient, err := remote.NewClusterClient(r.Client, cluster, r.scheme)
clusterClient, err := remote.NewClusterClient(ctx, r.Client, cluster, r.scheme)
if err != nil {
logger.Error(err, "Error building target cluster client")
return ctrl.Result{}, err
Expand Down Expand Up @@ -352,7 +352,7 @@ func (r *MachineHealthCheckReconciler) watchClusterNodes(ctx context.Context, c
return nil
}

config, err := remote.RESTConfig(c, cluster)
config, err := remote.RESTConfig(ctx, c, cluster)
if err != nil {
return errors.Wrap(err, "error fetching remote cluster config")
}
Expand Down

0 comments on commit 223f30c

Please sign in to comment.