Github user tbouron commented on a diff in the pull request:

    https://github.com/apache/brooklyn-server/pull/830#discussion_r158289882
  
    --- Diff: 
locations/container/src/main/java/org/apache/brooklyn/container/location/kubernetes/KubernetesLocation.java
 ---
    @@ -347,8 +386,8 @@ public Boolean call() {
                     if (result.isEmpty()) {
                         return false;
                     }
    -                List<HasMetadata> check = 
client.resource(result.get(0)).inNamespace(result.get(0).getMetadata().getNamespace()).get();
    -                if (result.size() > 1 || check.size() != 1 || 
check.get(0).getMetadata() == null) {
    +                HasMetadata check = 
client.resource(result.get(0)).inNamespace(result.get(0).getMetadata().getNamespace()).get();
    --- End diff --
    
    Is that normal the signature changed from `List<HasMetadata>` to 
`HasMetadata` ?


---

Reply via email to