From: Teoman ONAY <tonay@ibm.com>
Date: Fri, 24 Jan 2025 12:56:41 +0000 (+0100)
Subject: cephadm-adopt: fix ssl condition in rgw placement task
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=0f7b229a8d5843bbe214adf401bdb8a9c59d5d2e;p=ceph-ansible.git

cephadm-adopt: fix ssl condition in rgw placement task

rgw daemons fail to start as they were looking for
ssl certificates while none were available.

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit bf76bded4c3760a6610284cc477f642c8635f06d)
---

diff --git a/infrastructure-playbooks/cephadm-adopt.yml b/infrastructure-playbooks/cephadm-adopt.yml
index ebdd310e2..39830f126 100644
--- a/infrastructure-playbooks/cephadm-adopt.yml
+++ b/infrastructure-playbooks/cephadm-adopt.yml
@@ -954,7 +954,7 @@
           {% endif %}
           spec:
             rgw_frontend_port: {{ radosgw_frontend_port }}
-          {% if radosgw_frontend_ssl_certificate is defined %}
+          {% if radosgw_frontend_ssl_certificate | length > 0 %}
             {{ "ssl: true" }}
           {% endif %}
           extra_container_args:
@@ -1486,9 +1486,8 @@
               placement:
                 label: "{{ monitoring_group_name }}"
                 count: "{{ groups.get(monitoring_group_name, []) | length }}"
-              {% if grafana_server_addr is defined %}
-              networks:
-              - {{ grafana_server_addr }}
+              {% if grafana_network is defined %}
+              networks: {{ grafana_network.split(',') | list if ',' in grafana_network else [grafana_network] }}
               {% endif %}
           delegate_to: "{{ groups[mon_group_name][0] }}"
           environment:
@@ -1509,6 +1508,9 @@
               placement:
                 label: {{ monitoring_group_name }}
                 count: {{ groups.get(monitoring_group_name, []) | length }}
+              {% if grafana_network is defined %}
+              networks: {{ grafana_network.split(',') | list if ',' in grafana_network else [grafana_network] }}
+              {% endif %}
               {% if prometheus_port is defined and prometheus_port != 9095 %}
               spec:
                 port: {{ prometheus_port }}