Merge pull request #240 from openshift-cherrypick-robot/cherry-pick-2…

…27-to-release-4.17 DFBUGS-81:[release-4.17] controller: update storageclient connection fail log
red-hat-storage · Nov 19, 2024 · 435e72a · 435e72a
2 parents aaf873e + 11915d1
commit 435e72a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/internal/controller/storageclaim_controller.go b/internal/controller/storageclaim_controller.go
@@ -222,7 +222,7 @@ func (r *StorageClaimReconciler) reconcilePhases() (reconcile.Result, error) {
 		10*time.Second,
 	)
 	if err != nil {
-		return reconcile.Result{}, fmt.Errorf("failed to create provider client: %v", err)
+		return reconcile.Result{}, fmt.Errorf("failed to create provider client with endpoint %v: %v", r.storageClient.Spec.StorageProviderEndpoint, err)
 	}
 
 	// Close client-side connections.

diff --git a/internal/controller/storageclient_controller.go b/internal/controller/storageclient_controller.go
@@ -378,7 +378,7 @@ func (r *StorageClientReconciler) newExternalClusterClient() (*providerClient.OC
 	ocsProviderClient, err := providerClient.NewProviderClient(
 		r.ctx, r.storageClient.Spec.StorageProviderEndpoint, time.Second*10)
 	if err != nil {
-		return nil, fmt.Errorf("failed to create a new provider client: %v", err)
+		return nil, fmt.Errorf("failed to create a new provider client with endpoint %v: %v", r.storageClient.Spec.StorageProviderEndpoint, err)
 	}
 
 	return ocsProviderClient, nil

diff --git a/service/status-report/main.go b/service/status-report/main.go
@@ -103,7 +103,7 @@ func main() {
 		10*time.Second,
 	)
 	if err != nil {
-		klog.Exitf("Failed to create grpc client: %v", err)
+		klog.Exitf("Failed to create grpc client with endpoint %v: %v", storageClient.Spec.StorageProviderEndpoint, err)
 	}
 	defer providerClient.Close()