diff options
author | Harald Musum <musum@yahoo-inc.com> | 2017-03-08 15:24:19 +0100 |
---|---|---|
committer | Harald Musum <musum@yahoo-inc.com> | 2017-03-08 15:24:19 +0100 |
commit | 4a140b28876c418af2345b2f35639dbe3669dd73 (patch) | |
tree | d4b150ee74b4e6d3d08003b830e1346ca5144366 /node-repository | |
parent | 7d99c6f7dd5760a941fbd7941249447605d3ab1c (diff) |
Stop supporting creation of node with ipAddress
* Only support 'ipAddresses' array in request body from now on
Diffstat (limited to 'node-repository')
2 files changed, 5 insertions, 21 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/v2/NodesApiHandler.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/v2/NodesApiHandler.java index f45512c3c6f..c9c91d59641 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/v2/NodesApiHandler.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/v2/NodesApiHandler.java @@ -194,9 +194,7 @@ public class NodesApiHandler extends LoggingRequestHandler { private Node createNode(Inspector inspector) { Optional<String> parentHostname = optionalString(inspector.field("parentHostname")); - Optional<String> ipAddress = optionalString(inspector.field("ipAddress")); Set<String> ipAddresses = new HashSet<>(); - ipAddress.ifPresent(ipAddresses::add); inspector.field("ipAddresses").traverse((ArrayTraverser) (i, item) -> ipAddresses.add(item.asString())); return nodeRepository.createNode( diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/RestApiTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/RestApiTest.java index afe07adb645..1ef453eaecf 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/RestApiTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/RestApiTest.java @@ -442,28 +442,14 @@ public class RestApiTest { ", \"type\":\"host\"}"; } - // TODO: Simplify when ipAddress is not used anymore (see NodesApiHandler.createNode()) private String createIpAddresses(String... ipAddress) { - final String ipAddressJsonPart; - switch (ipAddress.length) { - case 0: - ipAddressJsonPart = ""; - break; - case 1: - // Old format - ipAddressJsonPart = "\"ipAddress\":\"" + ipAddress[0] + "\","; - break; - default: - ipAddressJsonPart = "\"ipAddresses\":[" + - Arrays.stream(ipAddress) - .map(ip -> "\"" + ip + "\"") - .collect(Collectors.joining(",")) + - "],"; - } - return ipAddressJsonPart; + return "\"ipAddresses\":[" + + Arrays.stream(ipAddress) + .map(ip -> "\"" + ip + "\"") + .collect(Collectors.joining(",")) + + "],"; } - /** Asserts a particular response and 200 as response status */ private void assertResponse(Request request, String responseMessage) throws IOException { assertResponse(request, 200, responseMessage); |