From b41b7bf869982a82d9ddddcef714a672c530b6f7 Mon Sep 17 00:00:00 2001 From: Teoman ONAY Date: Wed, 2 Oct 2024 16:36:31 +0200 Subject: [PATCH] cephadm-adopt: custom prometheus port lost after adoption If a custom Prometheus port was used before adoption, it was not taken into account and default 9095 was set instead. Now custom port is re-applied. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2242346 Signed-off-by: Teoman ONAY --- infrastructure-playbooks/cephadm-adopt.yml | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/infrastructure-playbooks/cephadm-adopt.yml b/infrastructure-playbooks/cephadm-adopt.yml index 8fb2beba7..2ecf68dbd 100644 --- a/infrastructure-playbooks/cephadm-adopt.yml +++ b/infrastructure-playbooks/cephadm-adopt.yml @@ -1500,7 +1500,18 @@ CEPHADM_IMAGE: '{{ ceph_docker_registry }}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}' - name: Update the placement of prometheus hosts - ansible.builtin.command: "{{ cephadm_cmd }} shell -k /etc/ceph/{{ cluster }}.client.admin.keyring --fsid {{ fsid }} -- ceph orch apply prometheus --placement='{{ groups.get(monitoring_group_name, []) | length }} label:{{ monitoring_group_name }}'" + ceph_orch_apply: + fsid: "{{ fsid }}" + spec: | + service_name: prometheus + service_id: "{{ ansible_facts['hostname'] }}" + placement: + label: {{ monitoring_group_name }} + count: {{ groups.get(monitoring_group_name, []) | length }} + {% if prometheus_port is defined and prometheus_port != 9095 %} + spec: + port: {{ prometheus_port }} + {% endif %} changed_when: false environment: CEPHADM_IMAGE: '{{ ceph_docker_registry }}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}'