Skip to content

[mariadb] slow-log housekeeping, dedicated slow-log volume, PVC-free alert - #12731

Open
s10 wants to merge 1 commit into
masterfrom
C5384329/mariadb-slowlogs
Open

s10 wants to merge 1 commit into
masterfrom
C5384329/mariadb-slowlogs

Conversation

@s10

@s10 s10 commented Sep 7, 2026

Copy link
Copy Markdown
Contributor
  • delete slow-query logs older than slow_query_log.cleanup.max_age_days (default 14) on pod init
  • optionally write the slow query log to a dedicated PVC (slow_query_log.persistence)
  • opt-in data-PVC free-space warning at alerts.pvc_free_enabled (threshold alerts.pvc_free_threshold, default 0.20)
  • fix context label typo (datbase) on the too-many-connections alert
  • chart version bumped to 0.43.0

@s10
s10 force-pushed the C5384329/mariadb-slowlogs branch 5 times, most recently from 0660ffb to 693cb56 Compare September 11, 2026 14:29
@s10
s10 marked this pull request as ready for review September 11, 2026 14:40
vssldmtrv
vssldmtrv previously approved these changes Sep 11, 2026
@s10
s10 marked this pull request as draft September 11, 2026 17:03
@s10
s10 force-pushed the C5384329/mariadb-slowlogs branch from 693cb56 to 0f5829c Compare September 14, 2026 09:44
galkindmitrii
galkindmitrii previously approved these changes Sep 14, 2026
Comment thread common/mariadb/templates/alerts/_volume.alerts.tpl Outdated
…alert

- prune slow-query logs older than slow_query_log.cleanup.max_age_days (default 14) on pod init
- optionally write the slow query log to a dedicated PVC (slow_query_log.persistence)
- opt-in data-PVC free-space alert at alerts.pvc_free_enabled (threshold alerts.pvc_free_threshold, default 0.20; severity alerts.pvc_free_severity, default warning)
- extract dataDir/dataPvcName/slowLogDir/slowLogPvcName path helpers so PVC names and the volume alert stay in sync
- fix context label typo (datbase) on the too-many-connections alert
- chart version bumped to 0.43.0
@s10
s10 force-pushed the C5384329/mariadb-slowlogs branch from 0f5829c to 73b2df6 Compare September 14, 2026 11:01
@s10
s10 marked this pull request as ready for review September 14, 2026 11:03
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.

3 participants