diff options
| author | Valerij Fredriksen <freva@users.noreply.github.com> | 2018-01-18 17:04:26 +0100 |
|---|---|---|
| committer | Valerij Fredriksen <valerijf@oath.com> | 2018-01-18 17:09:04 +0100 |
| commit | 5ce572d580765b133003be98649135166b708c01 (patch) | |
| tree | 6a75b943937fab6e6c12d38b014669f49a5e955e | |
| parent | b2abdfe88851ab4a8127bb171dce6505243f44a8 (diff) | |
Revert "don't allow nodes with spec divergence to go to dirty"
| -rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/NodeRepository.java | 5 | ||||
| -rw-r--r-- | node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/RestApiTest.java | 2 |
2 files changed, 3 insertions, 4 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/NodeRepository.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/NodeRepository.java index 19894050ff4..c26e59a0b1a 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/NodeRepository.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/NodeRepository.java @@ -391,9 +391,8 @@ public class NodeRepository extends AbstractComponent { Node nodeToDirty = getNode(hostname, Node.State.provisioned, Node.State.failed, Node.State.parked).orElseThrow(() -> new IllegalArgumentException("Could not deallocate " + hostname + ": No such node in the provisioned, failed or parked state")); - if (nodeToDirty.status().hardwareFailureDescription().isPresent() || nodeToDirty.status().hardwareDivergence().isPresent()) - throw new IllegalArgumentException("Could not deallocate " + hostname + ": It has a hardware failure/spec divergence"); - + if (nodeToDirty.status().hardwareFailureDescription().isPresent()) + throw new IllegalArgumentException("Could not deallocate " + hostname + ": It has a hardware failure"); return setDirty(nodeToDirty); } 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 c23a7f9990a..bdea767eb0d 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 @@ -282,7 +282,7 @@ public class RestApiTest { "{\"message\":\"Moved host12.yahoo.com to failed\"}"); assertResponse(new Request("http://localhost:8080/nodes/v2/state/dirty/host12.yahoo.com", new byte[0], Request.Method.PUT), 400, - "{\"error-code\":\"BAD_REQUEST\",\"message\":\"Could not deallocate host12.yahoo.com: It has a hardware failure/spec divergence\"}"); + "{\"error-code\":\"BAD_REQUEST\",\"message\":\"Could not deallocate host12.yahoo.com: It has a hardware failure\"}"); } @Test |