bug6038: implemented a reverse DNS lookup in AwsLandscape.findHostnamesForIP

java/com.sap.sse.landscape.aws.test/src/com/sap/sse/landscape/aws/ConnectivityTest.java

@@ -123,6 +123,11 @@ public class ConnectivityTest<ProcessT extends AwsApplicationProcess<String, Sai
         keyPass = "lkayrelakuesyrlasp8caorewyc".getBytes();
     }
     
+    @Test
+    public void testReverseDNSLookup() {
+        assertEquals("mongo0.internal.sapsailing.com", landscape.findHostnamesForIP("172.31.13.233"));
+    }
+    
     @Test
     public void testGetMasterOnApplicationProcess() throws Exception {
         final Iterable<SailingAnalyticsHost<String>> hosts = landscape.getApplicationProcessHostsByTag(region,

java/com.sap.sse.landscape.aws/src/com/sap/sse/landscape/aws/AwsLandscape.java

@@ -762,7 +762,9 @@ public interface AwsLandscape<ShardingKey> extends Landscape<ShardingKey> {
      * to that {@code ipAddress} and returns the record set name as the hostname. This searches
      * the whole of all resource record sets in the AWS account.<p>
      * 
-     * Example: asking for "172.31.13.233" may return something like "mongo0.internal.sapsailing.com"
+     * Example: asking for "172.31.13.233" may return something like "mongo0.internal.sapsailing.com"<p>
+     * 
+     * Note that this strips off any trailing "." (dot) from a resource record set name.
      * 
      * @return the fully-qualified resource record set name if found, otherwise {@code null}
      */

java/com.sap.sse.landscape.aws/src/com/sap/sse/landscape/aws/impl/AwsLandscapeImpl.java

@@ -1761,7 +1761,7 @@ public class AwsLandscapeImpl<ShardingKey> implements AwsLandscape<ShardingKey>
             for (final ResourceRecordSet resourceRecordSet : route53Client.listResourceRecordSets(b->b.hostedZoneId(hostedZone.id())).resourceRecordSets()) {
                 for (final ResourceRecord resourceRecord : resourceRecordSet.resourceRecords()) {
                     if (Util.equalsWithNull(resourceRecord.value(), ipAddress)) {
-                        return resourceRecordSet.name();
+                        return resourceRecordSet.name().replaceFirst("\\.$", "");
                     }
                 }
             }