LRCI-4899 Do not return an offline or unreachable slave

liferay-commerce · Nov 13, 2024 · a8ee7c7 · a8ee7c7
1 parent c883c78
commit a8ee7c7
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/...enkins-results-parser/src/main/java/com/liferay/jenkins/results/parser/JenkinsMaster.java b/...enkins-results-parser/src/main/java/com/liferay/jenkins/results/parser/JenkinsMaster.java
@@ -479,7 +479,17 @@ public List<JSONObject> getQueueItemJSONObjects() {
 	}
 
 	public JenkinsSlave getRandomJenkinsSlave() {
-		return JenkinsResultsParserUtil.getRandomListItem(getJenkinsSlaves());
+		List<JenkinsSlave> jenkinsSlaves = new ArrayList<>();
+
+		for (JenkinsSlave jenkinsSlave : getJenkinsSlaves()) {
+			if (jenkinsSlave.isOffline() || !jenkinsSlave.isReachable()) {
+				continue;
+			}
+
+			jenkinsSlaves.add(jenkinsSlave);
+		}
+
+		return JenkinsResultsParserUtil.getRandomListItem(jenkinsSlaves);
 	}
 
 	public String getRemoteURL() {