Skip to content

Commit

Permalink
fix(applications): fix dedupe of front50 and clouddriver apps (#632) (#…
Browse files Browse the repository at this point in the history
…633)

We were trying to dedupe on appname but each system presents them in a different case (front50 upper
clouddriver lower)

(cherry picked from commit 18b642f)

Co-authored-by: Cameron Fieber <cfieber@netflix.com>
  • Loading branch information
mergify[bot] and cfieber authored Apr 6, 2020
1 parent aaebd07 commit c3b6b6d
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ protected Set<Application> loadAll() throws ProviderException {
// clouddriver version definitely won't)
List<Application> applications =
Streams.concat(front50Applications.stream(), clouddriverApplications.stream())
.filter(distinctByKey(Application::getName))
.filter(distinctByKey(a -> a.getName().toUpperCase()))
// Collect to a list instead of set since we're about to modify the applications
.collect(toImmutableList());

Expand Down

0 comments on commit c3b6b6d

Please sign in to comment.