Do not switch and remove node in one iteration

SeleniumHQ · Apr 27, 2023 · 6cccb39 · 6cccb39
1 parent ef69f1b
commit 6cccb39
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/java/src/org/openqa/selenium/grid/distributor/GridModel.java b/java/src/org/openqa/selenium/grid/distributor/GridModel.java
@@ -224,8 +224,7 @@ public void purgeDeadNodes() {
         if (node.getAvailability() == UP && lostTime.isBefore(now)) {
           LOG.info(String.format("Switching Node %s from UP to DOWN", node.getExternalUri()));
           replacements.put(node, rewrite(node, DOWN));
-        }
-        if (node.getAvailability() == DOWN && deadTime.isBefore(now)) {
+        } else if (node.getAvailability() == DOWN && deadTime.isBefore(now)) {
           LOG.info(String.format("Removing Node %s, DOWN for too long", node.getExternalUri()));
           toRemove.add(node);
         }