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
1 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 013743f910 - Show all commits

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

@ -18,7 +18,7 @@ groups:
annotations: annotations:
summary: >- summary: >-
Le container Docker est éteint / tombé Le container Docker est éteint / tombé
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') }}) (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
@ -28,7 +28,7 @@ groups:
annotations: annotations:
summary: >- summary: >-
Le container Docker redémarre souvent Le container Docker redémarre souvent
(instance {{ raw('$labels.instance') }}, container {{ raw('$labels.name') }}) (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:
@ -44,7 +44,7 @@ groups:
annotations: annotations:
summary: >- summary: >-
Le container Docker utilise beaucoup de CPU Le 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') }}, (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"') }}) valeur {{ raw('$value | printf "%.1f"') }})
... ...