diff --git a/internal/controller/csiaddons/csiaddonsnode_controller.go b/internal/controller/csiaddons/csiaddonsnode_controller.go index 13a8b062e..d5f0f763e 100644 --- a/internal/controller/csiaddons/csiaddonsnode_controller.go +++ b/internal/controller/csiaddons/csiaddonsnode_controller.go @@ -95,13 +95,13 @@ func (r *CSIAddonsNodeReconciler) Reconcile(ctx context.Context, req ctrl.Reques nodeID := csiAddonsNode.Spec.Driver.NodeID driverName := csiAddonsNode.Spec.Driver.Name - key := csiAddonsNode.Namespace + "/" + csiAddonsNode.Name + key := csiAddonsNode.Namespace + "/" + util.NormalizeLeaseName(csiAddonsNode.Name) logger = logger.WithValues("NodeID", nodeID, "DriverName", driverName) if !csiAddonsNode.DeletionTimestamp.IsZero() { // if deletion timestamp is set, the CSIAddonsNode is getting deleted, // delete connections and remove finalizer. - logger.Info("Deleting connection") + logger.Info("Deleting connection", "Key", key) r.ConnPool.Delete(key) err = r.removeFinalizer(ctx, &logger, csiAddonsNode) return ctrl.Result{}, err @@ -139,7 +139,7 @@ func (r *CSIAddonsNodeReconciler) Reconcile(ctx context.Context, req ctrl.Reques logger.Info("Successfully connected to sidecar") r.ConnPool.Put(key, newConn) - logger.Info("Added connection to connection pool") + logger.Info("Added connection to connection pool", "Key", key) csiAddonsNode.Status.State = csiaddonsv1alpha1.CSIAddonsNodeStateConnected csiAddonsNode.Status.Message = "Successfully established connection with sidecar" diff --git a/internal/controller/replication.storage/volumereplication_controller.go b/internal/controller/replication.storage/volumereplication_controller.go index 288d0c8ac..c69fd8e40 100644 --- a/internal/controller/replication.storage/volumereplication_controller.go +++ b/internal/controller/replication.storage/volumereplication_controller.go @@ -510,7 +510,7 @@ func getInfoReconcileInterval(parameters map[string]string, logger logr.Logger) func (r *VolumeReplicationReconciler) getReplicationClient(ctx context.Context, driverName, dataSource string) (grpcClient.VolumeReplication, error) { conn, err := r.Connpool.GetLeaderByDriver(ctx, r.Client, driverName) if err != nil { - return nil, fmt.Errorf("no leader for the ControllerService of driver %q", driverName) + return nil, fmt.Errorf("no leader for the ControllerService of driver %q: %w", driverName, err) } for _, cap := range conn.Capabilities {