diff options
author | Harald Musum <musum@yahooinc.com> | 2023-08-25 12:32:59 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-08-25 12:32:59 +0200 |
commit | 2fa3481c2f1368735af41f4db1f3822a028449b4 (patch) | |
tree | 9ea298148b5fe7e804e701b5459411e8cadd62ed /clustercontroller-core | |
parent | 66b009fc2f83f3bf50f2ed23919b92f8bf154e1a (diff) |
Revert "More logging at higher level, need more info"
Diffstat (limited to 'clustercontroller-core')
-rw-r--r-- | clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java index 212748d6947..8453fb3450c 100644 --- a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java +++ b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java @@ -39,7 +39,6 @@ import static com.yahoo.vespa.clustercontroller.core.NodeStateChangeChecker.Resu import static com.yahoo.vespa.clustercontroller.utils.staterestapi.requests.SetUnitStateRequest.Condition.FORCE; import static com.yahoo.vespa.clustercontroller.utils.staterestapi.requests.SetUnitStateRequest.Condition.SAFE; import static java.util.logging.Level.FINE; -import static java.util.logging.Level.INFO; /** * Checks if a node can be upgraded. @@ -166,7 +165,6 @@ public class NodeStateChangeChecker { if (anotherNodeInGroupAlreadyAllowed(nodeInfo, newDescription)) return allow(); } else { - log.log(INFO, "Checking if we can set " + nodeInfo.getNode() + " to maintenance temporarily"); var optionalResult = checkIfOtherNodesHaveWantedState(nodeInfo, newDescription, clusterState); if (optionalResult.isPresent()) return optionalResult.get(); @@ -245,13 +243,13 @@ public class NodeStateChangeChecker { Set<Integer> groupsWithNodesWantedStateNotUp = groupsWithUserWantedStateNotUp(); if (groupsWithNodesWantedStateNotUp.size() == 0) { - log.log(INFO, "groupsWithNodesWantedStateNotUp=0"); + log.log(FINE, "groupsWithNodesWantedStateNotUp=0"); return Optional.empty(); } Set<Integer> groupsWithSameStateAndDescription = groupsWithSameStateAndDescription(MAINTENANCE, newDescription); if (aGroupContainsNode(groupsWithSameStateAndDescription, node)) { - log.log(INFO, "Node is in group with same state and description, allow"); + log.log(FINE, "Node is in group with same state and description, allow"); return Optional.of(allow()); } // There are groups with nodes not up, but with another description, probably operator set @@ -273,7 +271,7 @@ public class NodeStateChangeChecker { return result; if (numberOfGroupsToConsider < maxNumberOfGroupsAllowedToBeDown) { - log.log(INFO, "Allow, retiredAndNotUpGroups=" + retiredAndNotUpGroups); + log.log(FINE, "Allow, retiredAndNotUpGroups=" + retiredAndNotUpGroups); return Optional.of(allow()); } |