summaryrefslogtreecommitdiffstats
path: root/node-repository
diff options
context:
space:
mode:
authorMartin Polden <mpolden@mpolden.no>2020-05-29 15:33:55 +0200
committerMartin Polden <mpolden@mpolden.no>2020-06-03 15:10:21 +0200
commit8d2a0e5a3903d447fcce76558725558ea61d6bb0 (patch)
treed046fb1efad3f5a491cf087f889cb9ffe38e9cfa /node-repository
parentdbed0aaf832b9dc5b1376aa2724ddcf1125fe31f (diff)
Update InactiveExpirer javadoc
Diffstat (limited to 'node-repository')
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/InactiveExpirer.java21
1 files changed, 13 insertions, 8 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/InactiveExpirer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/InactiveExpirer.java
index b2c57e984eb..3cb7cc218a7 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/InactiveExpirer.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/InactiveExpirer.java
@@ -5,6 +5,7 @@ import com.yahoo.vespa.hosted.provision.Node;
import com.yahoo.vespa.hosted.provision.NodeRepository;
import com.yahoo.vespa.hosted.provision.node.Agent;
import com.yahoo.vespa.hosted.provision.node.History;
+import com.yahoo.vespa.hosted.provision.node.Status;
import java.time.Clock;
import java.time.Duration;
@@ -14,15 +15,19 @@ import java.util.List;
* Maintenance job which moves inactive nodes to dirty or parked after timeout.
*
* The timeout is in place for two reasons:
- * <ul>
- * <li>To ensure that the new application configuration has time to
- * propagate before the node is used for something else
- * <li>To provide a grace period in which nodes can be brought back to active
- * if they were deactivated in error. As inactive nodes retain their state
- * they can be brought back to active and correct state faster than a new node.
- * </ul>
*
- * Nodes with the retired flag should not be reused and will be moved to parked instead of dirty.
+ * - To ensure that the new application configuration has time to
+ * propagate before the node is used for something else.
+ *
+ * - To provide a grace period in which nodes can be brought back to active
+ * if they were deactivated in error. As inactive nodes retain their state
+ * they can be brought back to active and correct state faster than a new node.
+ *
+ * Nodes with following flags set are not reusable and will be moved to parked
+ * instead of dirty:
+ *
+ * - {@link Status#wantToRetire()} (when set by an operator)
+ * - {@link Status#wantToDeprovision()}
*
* @author bratseth
* @author mpolden