Skip to content

Commit 8cd0b89

Browse files
authored
Fix for intermittent failure for HPACustomMetrics test (#4126)
* Fix for intermittent failure for HPACustomMetrics test
1 parent bf493ba commit 8cd0b89

File tree

1 file changed

+19
-2
lines changed

1 file changed

+19
-2
lines changed

integration-tests/src/test/java/oracle/weblogic/kubernetes/ItHorizontalPodAutoscalerCustomMetrics.java

Lines changed: 19 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
import oracle.weblogic.kubernetes.actions.impl.primitive.HelmParams;
2929
import oracle.weblogic.kubernetes.annotations.IntegrationTest;
3030
import oracle.weblogic.kubernetes.annotations.Namespaces;
31+
import oracle.weblogic.kubernetes.assertions.impl.Kubernetes;
3132
import oracle.weblogic.kubernetes.logging.LoggingFacade;
3233
import oracle.weblogic.kubernetes.utils.ExecCommand;
3334
import oracle.weblogic.kubernetes.utils.ExecResult;
@@ -305,11 +306,27 @@ void testHPAWithCustomMetrics() {
305306
() -> verifyHPA(domainNamespace, "0/5"),
306307
logger,
307308
"Checking if total_open_session metric is 0");
309+
testUntil(
310+
withLongRetryPolicy,
311+
() -> verifyHPA(domainNamespace, "2 3 2"),
312+
logger,
313+
"Checking if replica switched to 2");
314+
308315
try {
309316
checkPodDeleted(managedServerPrefix + 3, domainUid, domainNamespace);
310317
} catch (Exception ex) {
311-
//retry again
312-
checkPodDeleted(managedServerPrefix + 3, domainUid, domainNamespace);
318+
//check if different server was scaled down
319+
try {
320+
if (!Kubernetes.doesPodExist(domainNamespace, domainUid, managedServerPrefix + 1)) {
321+
logger.info("HPA scaled down managed server 1");
322+
} else if (!Kubernetes.doesPodExist(domainNamespace, domainUid, managedServerPrefix + 2)) {
323+
logger.info("HPA scaled down managed server 2");
324+
} else {
325+
checkPodDeleted(managedServerPrefix + 3, domainUid, domainNamespace);
326+
}
327+
} catch (Exception ex1) {
328+
throw ex;
329+
}
313330
}
314331
}
315332

0 commit comments

Comments
 (0)