fix(core): A better case insensitive comparison when bulk loading obj…

…ects (#551)

Previously was using `Timestamped::getId` which may not always return
_expected_ lower case `id`.

front50-core/src/main/groovy/com/netflix/spinnaker/front50/model/StorageServiceSupport.java

@@ -362,7 +362,7 @@ private Set<T> fetchAllItems(Set<T> existingItems) {
 
       Map<String, T> objectsById =
           objects.stream()
-              .collect(Collectors.toMap(Timestamped::getId, Function.identity(), (o1, o2) -> o1));
+              .collect(Collectors.toMap(this::buildObjectKey, Function.identity(), (o1, o2) -> o1));
 
       for (String objectKey : objectKeys) {
         if (objectsById.containsKey(objectKey)) {
@@ -371,6 +371,8 @@ private Set<T> fetchAllItems(Set<T> existingItems) {
           // equivalent to the NotFoundException handling in the exceptional case below
           resultMap.remove(keyToId.get(objectKey));
           numRemoved.getAndIncrement();
+
+          log.warn("Unable to find result for {}:{} (filtering!)", objectType, objectKey);
         }
       }
     } catch (UnsupportedOperationException e) {