Addressing comments on ctx for logger

Author: ChristianZaccaria

pkg/controllers/raycluster_controller.go

@@ -103,7 +103,7 @@ func (r *RayClusterReconciler) Reconcile(ctx context.Context, req ctrl.Request)
 
 	isLocalInteractive := annotationBoolVal(logger, &cluster, "sdk.codeflare.dev/local_interactive")
 	ingressDomain := cluster.ObjectMeta.Annotations["sdk.codeflare.dev/ingress_domain"]
-	isOpenShift, ingressHost := getClusterType(logger, r.kubeClient, &cluster, ingressDomain)
+	isOpenShift, ingressHost := getClusterType(ctx, r.kubeClient, &cluster, ingressDomain)
 
 	if cluster.ObjectMeta.DeletionTimestamp.IsZero() {
 		if !controllerutil.ContainsFinalizer(&cluster, oAuthFinalizer) {

pkg/controllers/support.go

@@ -127,8 +127,9 @@ func getDiscoveryClient(config *rest.Config) (*discovery.DiscoveryClient, error)
 
 // Check where we are running. We are trying to distinguish here whether
 // this is vanilla kubernetes cluster or Openshift
-func getClusterType(logger logr.Logger, clientset *kubernetes.Clientset, cluster *rayv1.RayCluster, ingressDomain string) (bool, string) {
+func getClusterType(ctx context.Context, clientset *kubernetes.Clientset, cluster *rayv1.RayCluster, ingressDomain string) (bool, string) {
 	// The discovery package is used to discover APIs supported by a Kubernetes API server.
+	logger := ctrl.LoggerFrom(ctx)
 	config, err := ctrl.GetConfig()
 	if err == nil && config != nil {
 		dclient, err := getDiscoveryClient(config)