aurore/ansible
aurore/ansible
11
2
Fork 0
Code Issues 1 Pull requests 13 Releases Wiki Activity

Installation Prometheus-postgres-exporter #53

Merged
pz2891 merged 29 commits from prometheus_postgres_exporter into master 2021-04-14 20:19:59 +02:00
Conversation 0 Commits 29 Files changed 12 +4 -4
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 6c64bb214c - Show all commits

4
monitoring.yml
View file

@ -130,7 +130,7 @@
# Prometheus targets.json # Prometheus targets.json
prometheus_targets: prometheus_targets:
- targets: | - targets: |
{{ groups['ovh_pve'] + groups['ovh_vm'] | list | sort }} {{ groups['ovh_pve'] + groups['ovh_vm'] | list | sort }}
prometheus_postgres_targets: prometheus_postgres_targets:
- targets: - targets:
- bdd-ovh.adm.auro.re - bdd-ovh.adm.auro.re
@ -145,7 +145,7 @@
prometheus_alertmanager: docker-ovh.adm.auro.re:9093 prometheus_alertmanager: docker-ovh.adm.auro.re:9093
snmp_unifi_password: "{{ vault_snmp_unifi_password }}" snmp_unifi_password: "{{ vault_snmp_unifi_password }}"
# Prometheus targets.json #Prometheus targets.json
prometheus_targets: prometheus_targets:
- prometheus-edc.adm.auro.re - prometheus-edc.adm.auro.re
- prometheus-gs.adm.auro.re - prometheus-gs.adm.auro.re

4
roles/prometheus/tasks/main.yml
View file

@ -20,12 +20,12 @@
- prometheus.yml - prometheus.yml
notify: Restart Prometheus notify: Restart Prometheus
- name: Creates directory for alerts - name: Creates directory for alerts
file: file:
pz2891 marked this conversation as resolved
jeltz commented 2021-04-12 15:13:47 +02:00
Review
Copy link

Ça pourrait être bien de préciser le propriétaire et le groupe du dossier.

Ça pourrait être bien de préciser le propriétaire et le groupe du dossier.
path: /etc/prometheus/alerts path: /etc/prometheus/alerts
state: directory state: directory
- name: Remove old files - name: Remove old files
file: file:
pz2891 marked this conversation as resolved Outdated
jeltz commented 2021-04-12 15:15:00 +02:00
Outdated
Review
Copy link

Est-ce que c'est la peine de garder ces règles une fois que ça a été exécuté une fois sur le parc ?

Peut-être que le fichier alert.rules.yml est créé par défaut dans l'installation Debian ?

Est-ce que c'est la peine de garder ces règles une fois que ça a été exécuté une fois sur le parc ? Peut-être que le fichier `alert.rules.yml` est créé par défaut dans l'installation Debian ?
path: "/etc/prometheus/{{ item }}" path: "/etc/prometheus/{{ item }}"
state: absent state: absent