entrouvert
/
publik-infra
16
0
Fork 0
Code Issues Pull Requests Releases Activity

rework journald counter to respect ignored error (#78679) #2

Closed
gbaffoin wants to merge 0 commits from wip/78679 into main
pull from: wip/78679
merge into: entrouvert:main
Conversation 1 Commits - Files Changed - +2
gbaffoin commented 2023-08-22 10:38:10 +02:00
Owner
Copy Link

la fonction journald ne respecte pas les messages ignorés pour l'alerteur critical (et non plus pour error en fait).
de plus j'ajoute des exceptions supplémentaires pour atténuer le nombre de remonté.

la fonction journald ne respecte pas les messages ignorés pour l'alerteur critical (et non plus pour error en fait). de plus j'ajoute des exceptions supplémentaires pour atténuer le nombre de remonté.
gbaffoin added 1 commit 2023-08-22 10:38:11 +02:00
gitea/publik-infra/pipeline/head This commit looks good Details
9816770b49 rework journald counter to respect ignored error (#78679)
pducroquet requested changes 2023-08-22 10:47:58 +02:00
pducroquet left a comment
Owner
Copy Link

Ne pas faire de copier coller :)

Ne pas faire de copier coller :)
prometheus-entrouvert-exporter/prometheus-system-exporter.py Outdated
@ -133,0 +134,4 @@
j.seek_realtime(fifteen)
j.add_match(PRIORITY=2)
eo_journal.labels(ctn, "error").set(0)
for e in j:
pducroquet commented 2023-08-22 10:47:47 +02:00
Owner
Copy Link

Factoriser ça avec la boucle précédente.

Par exemple:

# This is an implicit OR, cf documentation of add_match
j.add_match(PRIORITY=2)
j.add_match(PRIORITY=3)

voire j.log_level(3)

Puis:

priority_to_tag = {2: "critical", 3: "error"}

... # BLABLA du code ignored machin tout ça

eo_journal.labels(ctn, priority_to_tag[e["PRIORITY"]]).inc()
Factoriser ça avec la boucle précédente. Par exemple: ``` # This is an implicit OR, cf documentation of add_match j.add_match(PRIORITY=2) j.add_match(PRIORITY=3) ``` voire ```j.log_level(3)``` Puis: ``` priority_to_tag = {2: "critical", 3: "error"} ... # BLABLA du code ignored machin tout ça eo_journal.labels(ctn, priority_to_tag[e["PRIORITY"]]).inc() ```
gbaffoin marked this conversation as resolved
prometheus-entrouvert-exporter/prometheus-system-exporter.py Outdated
@ -133,0 +146,4 @@
break
if ignored_message:
continue
else:
pducroquet commented 2023-08-22 10:42:28 +02:00
Owner
Copy Link

En fait la variable ignored_message est inutile.

        for ignored_string in ignored_strings:
            if ignored_string in msg:
                break
        else:
            eo_journal.labels(ctn, "error").inc()

Note que le else est au niveau du for.

En fait la variable ignored_message est inutile. ``` for ignored_string in ignored_strings: if ignored_string in msg: break else: eo_journal.labels(ctn, "error").inc() ``` Note que le else est au niveau du for.
gbaffoin commented 2023-08-22 11:12:11 +02:00
Author
Owner
Copy Link

vu et appliqué

vu et appliqué
gbaffoin marked this conversation as resolved
gbaffoin force-pushed wip/78679 from 9816770b49 to 7ccde7c38a 2023-08-22 11:08:47 +02:00 Compare
pducroquet reviewed 2023-08-22 11:52:01 +02:00
prometheus-entrouvert-exporter/prometheus-system-exporter.py Outdated
@ -115,4 +117,1 @@
j.seek_realtime(fifteen)
j.add_match(PRIORITY=2)
eo_journal.labels(ctn, "critical").set(len(list(j)))
j.seek_realtime(fifteen)
pducroquet commented 2023-08-22 11:52:01 +02:00
Owner
Copy Link

Où est partie cette ligne ?

Où est partie cette ligne ?
gbaffoin commented 2023-08-22 11:55:30 +02:00
Author
Owner
Copy Link

le match priority en bas.
le eo_journal.labels(ctn, "critical").set(len(list(j))) dans /dev/null (on initialise a zero et on incrément ensuite).
le j.seek_realtime(fifteen) etait en ndoublon avec la ligne 115

le match priority en bas. le eo_journal.labels(ctn, "critical").set(len(list(j))) dans /dev/null (on initialise a zero et on incrément ensuite). le j.seek_realtime(fifteen) etait en ndoublon avec la ligne 115
pducroquet marked this conversation as resolved
pducroquet approved these changes 2023-08-22 12:08:57 +02:00
pducroquet force-pushed wip/78679 from 7ccde7c38a to fedaf4684f 2023-08-23 13:36:02 +02:00 Compare
pducroquet commented 2023-08-23 13:36:45 +02:00
Owner
Copy Link

Oops, j'ai écrasé la branche, pardon. Je ferme cette demande de pull et j'en ouvre une autre

Oops, j'ai écrasé la branche, pardon. Je ferme cette demande de pull et j'en ouvre une autre
pducroquet closed this pull request 2023-08-23 13:36:45 +02:00
All checks were successful
gitea/publik-infra/pipeline/head This commit looks good
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
pducroquet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-infra#2
No description provided.
Delete Branch "wip/78679"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?