]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
config: only add related rgw section
authorGuillaume Abrioux <gabrioux@redhat.com>
Thu, 23 Jul 2020 19:12:46 +0000 (21:12 +0200)
committerDimitri Savineau <savineau.dimitri@gmail.com>
Wed, 5 Aug 2020 13:50:09 +0000 (09:50 -0400)
there's no need to add each rgw section on all rgw nodes.
With this commit, only related rgw section are rendered.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 0a581a6e6007812cdad935e4f65909b4306046b2)

roles/ceph-config/templates/ceph.conf.j2

index c9634c1176a96621e7e7904cb758069f3dfb9b10..b15068832c023d2fc3cffc507838158e73d87206 100644 (file)
@@ -105,15 +105,14 @@ osd memory target = {{ _osd_memory_target | default(osd_memory_target) }}
 {% endif %}
 
 {% if inventory_hostname in groups.get(rgw_group_name, []) %}
-{% for host in groups[rgw_group_name] %}
-{% set _rgw_hostname = hostvars[host]['rgw_hostname'] | default(hostvars[host]['ansible_hostname']) %}
+{% set _rgw_hostname = hostvars[inventory_hostname]['rgw_hostname'] | default(hostvars[inventory_hostname]['ansible_hostname']) %}
 {# {{ hostvars[host]['rgw_hostname'] }} for backward compatibility, fqdn issues. See bz1580408 #}
-{% if hostvars[host]['rgw_instances'] is defined %}
-{% for instance in hostvars[host]['rgw_instances'] %}
+{% if hostvars[inventory_hostname]['rgw_instances'] is defined %}
+{% for instance in hostvars[inventory_hostname]['rgw_instances'] %}
 [client.rgw.{{ _rgw_hostname + '.' + instance['instance_name'] }}]
 host = {{ _rgw_hostname }}
 keyring = /var/lib/ceph/radosgw/{{ cluster }}-rgw.{{ _rgw_hostname + '.' + instance['instance_name'] }}/keyring
-log file = /var/log/ceph/{{ cluster }}-rgw-{{ hostvars[host]['ansible_hostname'] + '.' + instance['instance_name'] }}.log
+log file = /var/log/ceph/{{ cluster }}-rgw-{{ hostvars[inventory_hostname]['ansible_hostname'] + '.' + instance['instance_name'] }}.log
 {% set _rgw_binding_socket = instance['radosgw_address'] | default(_radosgw_address) | string + ':' + instance['radosgw_frontend_port'] | default(radosgw_frontend_port) | string %}
 {%- macro frontend_line(frontend_type) -%}
 {%- if frontend_type == 'civetweb' -%}
@@ -133,7 +132,6 @@ rgw_zone = {{ instance['rgw_zone'] }}
 {% endif %}
 {% endfor %}
 {% endif %}
-{% endfor %}
 {% endif %}
 
 {% if inventory_hostname in groups.get(nfs_group_name, []) and inventory_hostname not in groups.get(rgw_group_name, []) %}