From 324fcc36ba4945623aa8e6b6f8466e45b6be93af Mon Sep 17 00:00:00 2001
From: Rabi Mishra <ramishra@redhat.com>
Date: Thu, 23 Apr 2020 14:27:09 +0530
Subject: [PATCH] Check for InternalApi in role for HostnameResolveNetwork

It's possible that some custom roles don't have InternalApi
network. Fallback to ctlplane/provisioning in those cases.

Change-Id: I385cd695bba0e3ec159f10d5ef87ee186cf29734
Closes-Bug: #1874330
---
 network/service_net_map.j2.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/network/service_net_map.j2.yaml b/network/service_net_map.j2.yaml
index 6b8e79d929..2ed5c5c574 100644
--- a/network/service_net_map.j2.yaml
+++ b/network/service_net_map.j2.yaml
@@ -96,7 +96,11 @@ parameters:
       {{role.name}}HostnameResolveNetwork: {{ _service_nets.get('storage', 'ctlplane') }}
 {%- endfor %}
 {%- for role in roles if not role.name.startswith('Ceph') %}
+    {%- if 'InternalApi' in role.networks %}
       {{role.name}}HostnameResolveNetwork: {{ _service_nets.get('internal_api', 'ctlplane') }}
+    {%- else %}
+      {{role.name}}HostnameResolveNetwork: ctlplane
+    {%- endif %}
 {%- endfor %}
       # special-case for MetricsQdrNetwork, which will vary depending on role
 {%- for role in roles if role.name == 'CephStorage' or role.name == 'BlockStorage' or role.name == 'ObjectStorage' %}