Add check for multiple descriptors matching the same key.

Signed-off-by: Milan Lenco <[email protected]>
ligato · May 17, 2019 · 36a4186 · 36a4186
1 parent 9875d2b
commit 36a4186
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/plugins/kvscheduler/internal/registry/registry_impl.go b/plugins/kvscheduler/internal/registry/registry_impl.go
@@ -19,6 +19,7 @@ import (
 
 	. "github.com/ligato/vpp-agent/plugins/kvscheduler/api"
 	"github.com/ligato/vpp-agent/plugins/kvscheduler/internal/utils"
+	"fmt"
 )
 
 const (
@@ -111,8 +112,11 @@ func (reg *registry) GetDescriptorForKey(key string) *KVDescriptor {
 	var keyDescriptor *KVDescriptor
 	for _, descriptor := range reg.descriptors {
 		if descriptor.KeySelector(key) {
+			if keyDescriptor != nil {
+				panic(fmt.Sprintf("key %s is selected by both %s and %s descriptors",
+					key, keyDescriptor.Name, descriptor.Name))
+			}
 			keyDescriptor = descriptor
-			break
 		}
 	}
 	// add entry to cache