brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From andreaturli <>
Subject [GitHub] brooklyn-server pull request #849: Retrieve the right AWS metadata for priva...
Date Mon, 02 Oct 2017 14:24:16 GMT
Github user andreaturli commented on a diff in the pull request:
    --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/
    @@ -2957,7 +2957,7 @@ String getHostnameAws(HostAndPort hostAndPort, LoginCredentials
                         "get public AWS hostname",
    -                            "echo `curl --silent --retry 20`;
    +                            "echo `curl --silent --retry 20"+(privateHost
? "local-hostname" : "public-hostname")+"`; exit"));
    --- End diff --
    sorry @tbouron, `String getHostnameAws(...)` is already very confusing, not mentioning
the fact that is AWS specific which is another smell, IMHO.
    Which hostname is going to return that method? 
    I think we have the chance to improve this area of the code and I think another (boolean)
parameter is not going to help in that respect. wdyt?


View raw message