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 +38 -37
2 changed files with 38 additions and 37 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 9d18ebb7f1 - Show all commits

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

@ -33,6 +33,7 @@
- alerts.rules.yml - alerts.rules.yml
- django.rules.yml - django.rules.yml
- name: Configure Prometheus alerts - name: Configure Prometheus alerts
template: template:
src: "{{ item }}.j2" src: "{{ item }}.j2"

74
roles/prometheus/templates/docker.rules.yml.j2
View file

@ -10,41 +10,41 @@ groups:
- name: docker.rules - name: docker.rules
rules: rules:
- alert: ContainerDown - alert: ContainerDown
expr: docker_container_running_state expr: docker_container_running_state != 1
for: 0m for: 0m
labels: labels:
severity: critical severity: critical
annotations: annotations:
summary: >- summary: >-
Container Docker mort Container Docker mort
pz2891 marked this conversation as resolved Outdated
otthorn commented 2021-04-12 10:37:05 +02:00
Outdated
Review
Copy link

éteind ?

éteind ?
otthorn commented 2021-04-12 10:39:39 +02:00
Outdated
Review
Copy link

Je sais pas trop, je trouve que mort en français ça fait bizzare. Mais en même temps "dead" semble être le terme correct en Anglais.

Je sais pas trop, je trouve que mort en français ça fait bizzare. Mais en même temps "dead" semble être le terme correct en Anglais.
otthorn commented 2021-04-12 10:41:27 +02:00
Outdated
Review
Copy link

Pour postgres en dessous on utilise "down", ce qui est un Anglissisme mais c'est pas mal.
Sinon on peut dire "tombé" qui est un traduction un peu abusive, mais ça je trouve pas ça choquant.

Pour postgres en dessous on utilise "down", ce qui est un Anglissisme mais c'est pas mal. Sinon on peut dire "tombé" qui est un traduction un peu abusive, mais ça je trouve pas ça choquant.
(instance {{ raw('$labels.instance') }}, container {{ raw('$labels.name') }}) (instance {{ raw('$labels.instance') }}, container {{ raw('$labels.name') }})
pz2891 marked this conversation as resolved Outdated
jeltz commented 2021-04-12 15:16:28 +02:00
Outdated
Review
Copy link

Pas la peine de mettre l'instance, on l'affiche déjà avec Alertbot automatiquemenet.

Pas la peine de mettre l'instance, on l'affiche déjà avec Alertbot automatiquemenet.
- alert: ContainerFailed - alert: ContainerFailed
expr: sum(increase(docker_container_restart_count[5m])) > 2 expr: sum(increase(docker_container_restart_count[5m])) > 2
for: 0m for: 0m
labels: labels:
severity: critical severity: critical
annotations: annotations:
summary: >- summary: >-
Container Docker redémarre souvent Container Docker redémarre souvent
(instance raw('{{ $labels.instance') }}, container {{ raw('$labels.name') }}) (instance {{ raw('$labels.instance') }}, container {{ raw('$labels.name') }})
pz2891 marked this conversation as resolved Outdated
jeltz commented 2021-04-12 15:16:43 +02:00
Outdated
Review
Copy link

Idem (instance superflue).

Idem (instance superflue).
- alert: ContainerFailed - alert: ContainerFailed
expr: expr:
( (
docker_container_cpu_used_total docker_container_cpu_used_total
/ /
docker_container_cpu_capacity_total docker_container_cpu_capacity_total
) * 100 ) * 100
> 30 > 30
for: 0m for: 0m
labels: labels:
severity: critical severity: critical
annotations: annotations:
summary: >- summary: >-
Container Docker utilise beaucoup de CPU Container Docker utilise beaucoup de CPU
pz2891 marked this conversation as resolved Outdated
otthorn commented 2021-04-12 10:40:38 +02:00
Outdated
Review
Copy link

Faudrait peut être faire des phrases.

C'est comme si on disait "Server utilise beaucoup de CPU", c'est pas syntaxiquement correct.
"Le container Docker utilise beaucoup de CPU" peut etre ?

Faudrait peut être faire des phrases. C'est comme si on disait "Server utilise beaucoup de CPU", c'est pas syntaxiquement correct. "Le container Docker utilise beaucoup de CPU" peut etre ?
(instance {{ raw('$labels.instance') }}, container {{ raw('$labels.name') }}, (instance {{ raw('$labels.instance') }}, container {{ raw('$labels.name') }},
pz2891 marked this conversation as resolved Outdated
jeltz commented 2021-04-12 15:16:52 +02:00
Outdated
Review
Copy link

Idem (instance superflue).

Idem (instance superflue).
value {{ raw('$value | printf "%.1f"'') }}) value {{ raw('$value | printf "%.1f"') }})
... ...