Skip to content

Commit 734d849

Browse files
authored
Logging SLM failures due to master node change at INFO level (#137771)
1 parent d085661 commit 734d849

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/SnapshotLifecycleTask.java

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77

88
package org.elasticsearch.xpack.slm;
99

10+
import org.apache.logging.log4j.Level;
1011
import org.apache.logging.log4j.LogManager;
1112
import org.apache.logging.log4j.Logger;
1213
import org.elasticsearch.ElasticsearchException;
@@ -19,6 +20,7 @@
1920
import org.elasticsearch.client.internal.OriginSettingClient;
2021
import org.elasticsearch.cluster.ClusterState;
2122
import org.elasticsearch.cluster.ClusterStateUpdateTask;
23+
import org.elasticsearch.cluster.NotMasterException;
2224
import org.elasticsearch.cluster.ProjectState;
2325
import org.elasticsearch.cluster.SnapshotsInProgress;
2426
import org.elasticsearch.cluster.metadata.ProjectId;
@@ -589,11 +591,14 @@ public ClusterState execute(ClusterState currentState) throws Exception {
589591

590592
@Override
591593
public void onFailure(Exception e) {
592-
logger.error(
593-
"failed to record snapshot policy execution status [{}] for snapshot [{}] in policy [{}]: {}",
594-
exception.isPresent() ? "failure" : "success",
595-
snapshotId.getName(),
596-
policyName,
594+
logger.log(
595+
e instanceof NotMasterException ? Level.INFO : Level.ERROR,
596+
format(
597+
"failed to record snapshot policy execution status [%s] for snapshot [%s] in policy [%s]",
598+
exception.isPresent() ? "failure" : "success",
599+
snapshotId.getName(),
600+
policyName
601+
),
597602
e
598603
);
599604
}

0 commit comments

Comments
 (0)