]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
Adopt with grafana_network not grafana_server_addr
authorJohn Fulton <fulton@redhat.com>
Fri, 7 Feb 2025 18:08:03 +0000 (13:08 -0500)
committerTeoman ONAY <tonay@ibm.com>
Tue, 11 Feb 2025 12:14:13 +0000 (13:14 +0100)
The networks list in a spec is usually a list of ranges,
not a single IP. The grafana_server_addr is a fact created
from the grafana_network range so it is a more appropriate
parameter to pass to the spec.

Follow up to 770a527a9ee6b34091798c6120235026ad1ecfa9

Signed-off-by: John Fulton <fulton@redhat.com>
infrastructure-playbooks/cephadm-adopt.yml

index 25cc2000e89b344cab066c8132f6d3f12699b654..c12bfe64019c17ab2b62e9f7856309b80988cc49 100644 (file)
               placement:
                 label: "{{ monitoring_group_name }}"
                 count: "{{ groups.get(monitoring_group_name, []) | length }}"
-              {% if grafana_server_addr is defined %}
-              networks: {{ grafana_server_addr.split(',') | list if ',' in grafana_server_addr else grafana_server_addr | string }}
+              {% 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:
               placement:
                 label: {{ monitoring_group_name }}
                 count: {{ groups.get(monitoring_group_name, []) | length }}
-              {% if grafana_server_addr is defined %}
-              networks: {{ grafana_server_addr.split(',') | list if ',' in grafana_server_addr else grafana_server_addr | string }}
+              {% 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: