Merge "Fix HostnameMap lookup - replace str_replace with yaql"

puppet/role.role.j2.yaml

@@ -360,9 +360,11 @@ resources:
       user_data_format: SOFTWARE_CONFIG
       user_data: {get_resource: UserData}
       name:
-        str_replace:
+        yaql:
-            template: {get_param: Hostname}
+          expression: $.data.hostname_map.get($.data.hostname, $.data.hostname)
-            params: {get_param: HostnameMap}
+          data:
+            hostname: {get_param: Hostname}
+            hostname_map: {get_param: HostnameMap}
       software_config_transport: {get_param: SoftwareConfigTransport}
       metadata:
         map_merge:

releasenotes/notes/fix-str-replace-hostname-issue-393fb6954f12f7e6.yaml

@@ -0,0 +1,10 @@
+---
+fixes:
+  - |
+    Fixes an issue where Custom Hostnames is in use. Previously it was possible
+    that the ``HostnameMap`` lookup lookup would return unexpected and
+    incorrect results if the hostname (the key) is not in the map. The string
+    replacement mechanism used previously was replaced by a yaql expression
+    that will do exact matching only. If no matching key is in the
+    HostnameMap, the default hostname is returned.
+    `bug 1781560  <https://review.openstack.org/#/c/582475/>`_.