aboutsummaryrefslogtreecommitdiffstats
path: root/clustercontroller-core
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2021-03-03 14:00:58 +0100
committerHarald Musum <musum@verizonmedia.com>2021-03-03 14:00:58 +0100
commitbe407c5c07cee112832de9de00f597c5c4406587 (patch)
tree88ce19c8d245866a66e2bc5260de640866a84b05 /clustercontroller-core
parent2d5116489de2695acfe7bd7928e65f369ce068f5 (diff)
Print node index wihtout parenthesis as done elsewhere
Diffstat (limited to 'clustercontroller-core')
-rw-r--r--clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java12
-rw-r--r--clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeCheckerTest.java2
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) {