diff options
author | Harald Musum <musum@yahoo-inc.com> | 2017-02-06 13:48:24 +0100 |
---|---|---|
committer | Harald Musum <musum@yahoo-inc.com> | 2017-02-06 13:48:24 +0100 |
commit | 856eb42280a2bcde415bf4cbce7b68c3439f8992 (patch) | |
tree | 7aa4dd412306000472e05e9597ab4bf487bebda6 /node-repository | |
parent | fd5127cf95e4a49851166f3f73b9c00ecb8d9e39 (diff) |
Update comments
Diffstat (limited to 'node-repository')
-rw-r--r-- | node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/AclProvisioningTest.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/AclProvisioningTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/AclProvisioningTest.java index 5cea884504e..8f9bef1241b 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/AclProvisioningTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/AclProvisioningTest.java @@ -134,7 +134,7 @@ public class AclProvisioningTest { @Test public void trusted_nodes_for_docker_hosts_and_proxy_nodes_in_zone_application() { - ApplicationId applicationId = tester.makeApplicationId(); // use same id for bot allocate calls below + ApplicationId applicationId = tester.makeApplicationId(); // use same id for both allocate calls below List<Node> configServers = setConfigServers("cfg1:1234,cfg2:1234,cfg3:1234"); // Populate repo @@ -144,7 +144,7 @@ public class AclProvisioningTest { // Allocate 3 proxy nodes List<Node> activeProxyNodes = allocateNodes(NodeType.proxy, applicationId); assertEquals(3, activeProxyNodes.size()); - // Allocate 2 Docker hosts, total of 5 hosts + // Allocate 2 Docker hosts, a total of 5 hosts List<Node> activeDockerHosts = allocateNodes(NodeType.host, applicationId); assertEquals(5, activeDockerHosts.size()); @@ -152,14 +152,14 @@ public class AclProvisioningTest { Node proxyNode = activeProxyNodes.get(0); List<NodeAcl> proxyNodeAcls = tester.nodeRepository().getNodeAcls(proxyNode, false); - // Trusted nodes is all config servers and all proxy nodes + // Trusted nodes are all Docker hosts, all proxy nodes and all config servers assertAcls(Arrays.asList(activeDockerHosts, activeProxyNodes, configServers), proxyNodeAcls); // Get trusted nodes for first Docker host Node dockerHost = activeDockerHosts.get(0); List<NodeAcl> dockerHostNodeAcls = tester.nodeRepository().getNodeAcls(dockerHost, false); - // Trusted nodes is all config servers and all proxy nodes + // Trusted nodes are all Docker hosts, all proxy nodes and all config servers assertAcls(Arrays.asList(activeDockerHosts, activeProxyNodes, configServers), dockerHostNodeAcls); } |