Fix incorrect destination address when hostname_variable equals with …

…destination_address

CHANGELOG.md

@@ -7,7 +7,8 @@ IMPROVEMENTS:
 
 BUG FIXES:
 
-- Fix grouping dynamic inventory on security group bug ([#165](https://github.com/alibaba/ansible-provider/pull/165))
+- Fix incorrect destination address when hostname_variable equals with destination_address ([#170](https://github.com/alibaba/ansible-provider/pull/170))
+- Fix grouping dynamic inventory on security group bug ([#169](https://github.com/alibaba/ansible-provider/pull/169))
 - Default to describe all instances in all regions ([#158](https://github.com/alibaba/ansible-provider/pull/158))
 - Fix ecs_instance_filters last page describe bug ([#157](https://github.com/alibaba/ansible-provider/pull/157))
 - Fix ecs_instance_filters's page_size bug ([#156](https://github.com/alibaba/ansible-provider/pull/156))

contrib/inventory/alicloud.py

@@ -323,8 +323,8 @@ def add_instance(self, instance, region):
         # If we can't get a nice hostname, use the destination address
         if not hostname:
             hostname = dest
-
-        hostname = self.to_safe(hostname).lower()
+        else:
+            hostname = self.to_safe(hostname).lower()
 
         # if we only want to include hosts that match a pattern, skip those that don't
         if self.pattern_include and not self.pattern_include.match(hostname):