Skip to content

Mixin: Add and update alerts #2644

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
SuperQ merged 19 commits into from
Jul 4, 2023
Merged

Mixin: Add and update alerts #2644

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
fc967aa
Add mountpoint to NodeFilesystem alerts
v-zhuravlev Mar 27, 2023
0e0399d
Decrease NodeFilesystem pending time to 15m
v-zhuravlev Mar 27, 2023
fd2d62a
Add CPU and memory alerts
v-zhuravlev Mar 27, 2023
7479418
Add failed systemd service alert
v-zhuravlev Mar 27, 2023
3d8075d
Decrease NodeNetwork*Errs pending period
v-zhuravlev Mar 27, 2023
614030b
Set 'at' everywhere as preposition for instance
v-zhuravlev Mar 27, 2023
94fc82e
Add NodeDiskIOSaturation alert
v-zhuravlev Mar 27, 2023
962de6c
Add %(nodeExporterSelector)s to Network and conntrack alerts
v-zhuravlev Mar 27, 2023
c3ec6e8
Add diskDevice selector
v-zhuravlev Mar 27, 2023
e15e7d6
Fix NodeMemoryHighUtilization alert
v-zhuravlev Mar 27, 2023
580c497
Add NodeSystemSaturation and NodeMemoryMajorPagesFaults
v-zhuravlev Mar 27, 2023
da32f8d
Decrease NodeSystemdServiceFailed severity to warning
v-zhuravlev Mar 29, 2023
e48e790
Extend alert description
v-zhuravlev Apr 5, 2023
2111e70
Add comma after 'mounted on'
v-zhuravlev Apr 5, 2023
77ae769
Add thresholds for memory alerts
v-zhuravlev Apr 5, 2023
6bdc1d9
Add thresholds for memory, disk and system alerts
v-zhuravlev Apr 5, 2023
b7dfb32
Set severity to NodeCPUHighUsage to info
v-zhuravlev Apr 5, 2023
3e250a9
Update NodeSystemSaturation severity
v-zhuravlev Apr 26, 2023
e8d7f4e
Revert alerts pending durtions
v-zhuravlev Jun 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add CPU and memory alerts 
Signed-off-by: Vitaly Zhuravlev <[email protected]>
  • Loading branch information
@v-zhuravlev
v-zhuravlev committed Jun 29, 2023
commit fd2d62af63fa493001b858ab8aa5860591b33fe7
28 changes: 28 additions & 0 deletions docs/node-mixin/alerts/alerts.libsonnet
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -309,6 +309,34 @@
description: 'File descriptors limit at {{ $labels.instance }} is currently at {{ printf "%.2f" $value }}%.',
},
},
{
alert: 'NodeCPUHighUsage',
Copy link
Member

@discordianfish discordianfish Mar 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

High CPU usage is not a problem and can just be an indicator or properly utilizing your machine, so I'd remove these

Copy link
Contributor Author

@v-zhuravlev v-zhuravlev Mar 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps, as long as we can alert on high system load(saturation).

Copy link
Member

@beorn7 beorn7 Mar 29, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, CPU usage is good. :) I mean, this would be a case for the "info" level alerts that I like to promote, but I don't think we have them here in the mixin.

(Info level alerts notify nobody, but you could look at the alerts page while troubleshooting. They point to things that are not problems per se and might be OK, but which you might be interested while there is an actual incident happening.)

Copy link
Member

@discordianfish discordianfish Apr 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I'd be fine with a 'info' level severity. No reason to now just introduce that now that we're on this.

Copy link
Contributor Author

@v-zhuravlev v-zhuravlev Apr 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll make it an info according to this guideline:

info for alerts that do not require any action by itself but mark something as “out of the ordinary”. Those alerts aren’t usually routed anywhere, but can be inspected during troubleshooting.

Copy link

@jcpunk jcpunk May 5, 2023 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps a warning if the usage stays above 98% for 1h would be viable? That would be a case where the host is at capacity and scheduling more tasks there would result in performance degradation. It is a risk folks can accept but something that should be considered as part of the capacity plan.

expr: |||
sum without(mode) (avg without (cpu) (rate(node_cpu_seconds_total{%(nodeExporterSelector)s, mode!="idle"}[2m]))) > 0.8
||| % $._config,
'for': '15m',
labels: {
severity: 'warning',
},
annotations: {
summary: 'High CPU usage.',
description: 'CPU usage on {{ $labels.instance }} has been above 80% for the last 15 minutes, is currently at {{ printf "%.2f" $value }}%.',
},
},
{
alert: 'NodeMemoryHighUtilization',
expr: |||
100 - node_memory_MemAvailable_bytes{%(nodeExporterSelector)s} / node_memory_MemTotal_bytes{%(nodeExporterSelector)s} * 100 < 10
||| % $._config,
'for': '15m',
labels: {
severity: 'warning',
},
annotations: {
summary: 'Host is running out of memory',
description: 'Memory is filling up on {{ $labels.instance }}, has been above 90% for the last 15 minutes, is currently at {{ printf "%.2f" $value }}%.',
},
},
],
},
],
Expand Down