Skip to content

Change alert evaluation time of PostgresqlHighRollbackRate from 0m to 5m#1810

Open
ggouzi wants to merge 1 commit into
canonical:mainfrom
ggouzi:fix-alert-evauliation-period-PostgresqlHighRollbackRate
Open

Change alert evaluation time of PostgresqlHighRollbackRate from 0m to 5m#1810
ggouzi wants to merge 1 commit into
canonical:mainfrom
ggouzi:fix-alert-evauliation-period-PostgresqlHighRollbackRate

Conversation

@ggouzi

@ggouzi ggouzi commented Jun 24, 2026

Copy link
Copy Markdown

Attempt to solve #1809

@ggouzi ggouzi requested a review from a team as a code owner June 24, 2026 09:12
@ggouzi ggouzi requested review from carlcsaposs-canonical, dragomirp, juju-charm-bot, marceloneppel and taurus-forever and removed request for a team June 24, 2026 09:12
@ggouzi ggouzi force-pushed the fix-alert-evauliation-period-PostgresqlHighRollbackRate branch from 0eccc28 to 4ebd45b Compare June 24, 2026 09:33

@taurus-forever taurus-forever left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM in general. Thank you for the bugreport and PR!
PostgresqlHighRollbackRate 0m=5m... smells like miss in testing.
BTW, it should be also backported to PostgreSQL VM 16/edge and PostgreSQL K8s 14+16.

@taurus-forever taurus-forever requested a review from 7annaba3l June 30, 2026 14:26

@7annaba3l 7annaba3l left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM
Note on the side: An improvement that might be added later is to ensure a minimum amount of activity so that a transaction being rolled-back when you have 2 tps is not flagged as problematic.

@ggouzi ggouzi force-pushed the fix-alert-evauliation-period-PostgresqlHighRollbackRate branch from ba9da1e to 5fce0e8 Compare July 3, 2026 04:55
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.

4 participants