Skip to content

Rethink etcd db alerts - #1036

Merged
daaang merged 2 commits into
productionfrom
add-etcd-db-alert
Aug 3, 2026
Merged

Rethink etcd db alerts#1036
daaang merged 2 commits into
productionfrom
add-etcd-db-alert

Conversation

@daaang

@daaang daaang commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

The previous alerts' 30 minute window was, in this case, way too long to be effective. It takes about 30 minutes in practice to go from "everything is perfectly fine" to "it's full and everything stopped responding to all requests."

Given the short window, it also seems silly to bother with deltas. It should never ever be more than 25% full let alone 50% full, so that should be good enough.

The previous alerts' 30 minute window was, in this case, way too long
to be effective. It takes about 30 minutes in practice to go from
"everything is perfectly fine" to "it's full and everything stopped
responding to all requests."

Given the short window, it also seems silly to bother with deltas. It
should never ever be more than 25% full let alone 50% full, so that
should be good enough.

@botimer botimer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Agreed that, while we prefer the delta style in general, etcd db size is a bit different than general disk trend in terms of expectation and intervention.

expr: >
max by (datacenter) (delta(etcd_debugging_mvcc_db_total_size_in_bytes[5m])) > 10000
for: "30m"
max by (datacenter) (etcd_mvcc_db_total_size_in_bytes) > 2 ^ 30

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I like that the summary includes the more familiar value of 1Gi, though I don't find the 2 ^ 30 form particularly scannable. Should we consider 1 * 1024 * 1024 * 1024 or 1 * 1024^3?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sure

@daaang
daaang merged commit c8880dc into production Aug 3, 2026
8 checks passed
@daaang
daaang deleted the add-etcd-db-alert branch August 3, 2026 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants