diff options
author | Harald Musum <musum@verizonmedia.com> | 2021-03-03 14:00:58 +0100 |
---|---|---|
committer | Harald Musum <musum@verizonmedia.com> | 2021-03-03 14:00:58 +0100 |
commit | be407c5c07cee112832de9de00f597c5c4406587 (patch) | |
tree | 88ce19c8d245866a66e2bc5260de640866a84b05 /clustercontroller-core | |
parent | 2d5116489de2695acfe7bd7928e65f369ce068f5 (diff) |
Print node index wihtout parenthesis as done elsewhere
Diffstat (limited to 'clustercontroller-core')
2 files changed, 7 insertions, 7 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 ca3169ede6f..918f01eef16 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 @@ -334,13 +334,13 @@ public class NodeStateChangeChecker { for (DistributorNodeInfo distributorNodeInfo : clusterInfo.getDistributorNodeInfo()) { Integer distributorClusterStateVersion = distributorNodeInfo.getHostInfo().getClusterStateVersionOrNull(); if (distributorClusterStateVersion == null) { - return Result.createDisallowed("Distributor node (" + distributorNodeInfo.getNodeIndex() - + ") has not reported any cluster state version yet."); + return Result.createDisallowed("Distributor node " + distributorNodeInfo.getNodeIndex() + + " has not reported any cluster state version yet."); } else if (distributorClusterStateVersion != clusterStateVersion) { - return Result.createDisallowed("Distributor node (" + distributorNodeInfo.getNodeIndex() - + ") does not report same version (" - + distributorNodeInfo.getHostInfo().getClusterStateVersionOrNull() - + ") as fleetcontroller has (" + clusterStateVersion + ")"); + return Result.createDisallowed("Distributor node " + distributorNodeInfo.getNodeIndex() + + " does not report same version (" + + distributorNodeInfo.getHostInfo().getClusterStateVersionOrNull() + + ") as fleetcontroller (" + clusterStateVersion + ")"); } List<StorageNode> storageNodes = distributorNodeInfo.getHostInfo().getDistributor().getStorageNodes(); diff --git a/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeCheckerTest.java b/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeCheckerTest.java index 3f654d36246..41284cc95d0 100644 --- a/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeCheckerTest.java +++ b/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeCheckerTest.java @@ -273,7 +273,7 @@ public class NodeStateChangeCheckerTest { nodeStorage, defaultAllUpClusterState(), SetUnitStateRequest.Condition.SAFE, UP_NODE_STATE, MAINTENANCE_NODE_STATE); assertFalse(result.settingWantedStateIsAllowed()); assertFalse(result.wantedStateAlreadySet()); - assertThat(result.getReason(), is("Distributor node (0) has not reported any cluster state version yet.")); + assertThat(result.getReason(), is("Distributor node 0 has not reported any cluster state version yet.")); } private NodeStateChangeChecker.Result transitionToSameState(State state, String oldDescription, String newDescription) { |