fix review findings

Author: sbueringer

controllers/remote/keyedmutex.go

@@ -27,13 +27,13 @@ import (
 // A second Lock call if the lock is already held for a key returns false.
 type keyedMutex struct {
 	locksMtx sync.Mutex
-	locks    map[client.ObjectKey]*sync.Mutex
+	locks    map[client.ObjectKey]bool
 }
 
 // newKeyedMutex creates a new keyed mutex ready for use.
 func newKeyedMutex() *keyedMutex {
 	return &keyedMutex{
-		locks: make(map[client.ObjectKey]*sync.Mutex),
+		locks: make(map[client.ObjectKey]bool),
 	}
 }
 
@@ -50,10 +50,8 @@ func (k *keyedMutex) TryLock(key client.ObjectKey) bool {
 		return false
 	}
 
-	// Lock doesn't exist yet, create and lock the lock.
-	l := &sync.Mutex{}
-	k.locks[key] = l
-	l.Lock()
+	// Lock doesn't exist yet, create the lock.
+	k.locks[key] = true
 
 	return true
 }
@@ -63,8 +61,7 @@ func (k *keyedMutex) Unlock(key client.ObjectKey) {
 	k.locksMtx.Lock()
 	defer k.locksMtx.Unlock()
 
-	if l, ok := k.locks[key]; ok {
-		l.Unlock()
+	if k.locks[key] {
 		delete(k.locks, key)
 	}
 }