Skip to content

Commit

Permalink
[Backport 2.14][PLAT-13249] Change clock skew alert threshold to 250
Browse files Browse the repository at this point in the history
Summary:
To detect clock skew issues before TServer starts crashing we're changing threshold to 250ms.

Original diff: https://phorge.dev.yugabyte.com/D33582

Test Plan: Testead manually.

Reviewers: vbansal

Reviewed By: vbansal

Subscribers: yugaware, sanketh

Tags: #jenkins-ready

Differential Revision: https://phorge.dev.yugabyte.com/D33588
  • Loading branch information
anmalysh-yb committed Mar 28, 2024
1 parent a3ae4e5 commit 2102766
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 1 deletion.
Expand Up @@ -132,6 +132,11 @@ private SyncResult syncDefinition(UUID definitionUuid) {
|| !configuration.isActive()) {
swamperHelper.removeAlertDefinition(definitionUuid);
requiresReload.set(true);
if (definition != null) {
// Don't want to retry inactive definitions.
definition.setConfigWritten(true);
alertDefinitionService.save(definition);
}
return SyncResult.REMOVED;
}
if (definition.isConfigWritten()) {
Expand Down
@@ -0,0 +1,12 @@
-- Copyright (c) YugaByte, Inc.

--- Update default threshold
update alert_configuration
set thresholds = replace(thresholds,
'"SEVERE":{"condition":"GREATER_THAN","threshold":500.0}',
'"SEVERE":{"condition":"GREATER_THAN","threshold":250.0}')
where template = 'CLOCK_SKEW';

-- Recreate alert definition with new threshold
update alert_definition set config_written = false where configuration_uuid IN
(select uuid from alert_configuration where template = 'CLOCK_SKEW');
2 changes: 1 addition & 1 deletion managed/src/main/resources/reference.conf
Expand Up @@ -66,7 +66,7 @@ yb {
# Alerts thresholds
alert {
# Value of maximum allowed clock skew before an alert is generated (in ms).
max_clock_skew_ms = 500
max_clock_skew_ms = 250
# Value of maximum allowed replication lag before an alert is generated (in ms).
replication_lag_ms = 180000
# Value of maximum allowed percents of used memory on nodes.
Expand Down

0 comments on commit 2102766

Please sign in to comment.