ceph-prometheus: fix error in templates

- remove trailing double quotes in jinja templates
- add jinja filename without .j2 suffix

Resolves: #4011

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
(cherry picked from commit 29b0d47c8cc3943ee89aaa660455616f87f90caa)

diff --git a/roles/ceph-prometheus/tasks/main.yml b/roles/ceph-prometheus/tasks/main.yml
index 71affd66b720c9e05d14fa9038791450697db0d2..55ce5e2e23b6f74eaf5b25df190b9db1694e26b8 100644 (file)
--- a/roles/ceph-prometheus/tasks/main.yml
+++ b/roles/ceph-prometheus/tasks/main.yml
@@ -11,7 +11,7 @@
 - name: write prometheus config file
   template:
     src: prometheus.yml.j2
-    dest: "{{ prometheus_conf_dir }}/"
+    dest: "{{ prometheus_conf_dir }}/prometheus.yml"
     owner: "{{ prometheus_user_id }}"
   notify: service handler
 
@@ -41,7 +41,7 @@
 - name: write alertmanager config file
   template:
     src: alertmanager.yml.j2
-    dest: "{{ alertmanager_conf_dir }}/"
+    dest: "{{ alertmanager_conf_dir }}/alertmanager.yml"
     owner: "root"
   notify: service handler
 

diff --git a/roles/ceph-prometheus/templates/alertmanager.service.j2 b/roles/ceph-prometheus/templates/alertmanager.service.j2
index 6a9c9c5242f79d5175f78383ad326a62b7cf399e..207a53e6e934663079c59d1931df78641c9e8e3d 100644 (file)
--- a/roles/ceph-prometheus/templates/alertmanager.service.j2
+++ b/roles/ceph-prometheus/templates/alertmanager.service.j2
@@ -19,7 +19,7 @@ ExecStart=/usr/bin/{{ container_binary }} run --name=alertmanager \
   --memory-swap={{ alertmanager_container_memory * 2 }}GB \
   {{ alertmanager_container_image }} \
   --config.file=/etc/alertmanager/alertmanager.yml \
-  --storage.path=/alertmanager"
+  --storage.path=/alertmanager
 ExecStop=/usr/bin/{{ container_binary }} stop alertmanager
 Restart=always
 RestartSec=10s

diff --git a/roles/ceph-prometheus/templates/prometheus.service.j2 b/roles/ceph-prometheus/templates/prometheus.service.j2
index 1d99c7e05cdfe32b1d8081589a3a8b81fa7bd722..0ae01940bb00c9c9d622466d194fabebda0badcb 100644 (file)
--- a/roles/ceph-prometheus/templates/prometheus.service.j2
+++ b/roles/ceph-prometheus/templates/prometheus.service.j2
@@ -21,7 +21,7 @@ ExecStart=/usr/bin/{{ container_binary }} run --name=prometheus \
   {{ prometheus_container_image }} \
   --config.file=/etc/prometheus/prometheus.yml \
   --storage.tsdb.path=/prometheus \
-  --web.external-url=http://{{ inventory_hostname }}:9090/"
+  --web.external-url=http://{{ inventory_hostname }}:9090/
 ExecStop=/usr/bin/{{ container_binary }} stop prometheus
 Restart=always
 RestartSec=10s