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

…27-to-release-4.16 DFBUGS-532:[release-4.16] controller: update storageclient connection fail log
red-hat-storage · Nov 19, 2024 · 45b3302 · 45b3302
2 parents 4beea0d + 5bdb2a9
commit 45b3302
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/controllers/storageclaim_controller.go b/controllers/storageclaim_controller.go
@@ -215,7 +215,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/controllers/storageclient_controller.go b/controllers/storageclient_controller.go
@@ -242,7 +242,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
@@ -95,7 +95,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()