ESQL: Retry test on 403 (elastic#114450)

Retry the async test when you get a 403 - that could be because security has not yet booted. We should have permission to fetch everything.
FaatimaAbidi · Oct 11, 2024 · 4c48aa3 · 4c48aa3
1 parent a62228a
commit 4c48aa3
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/...p-attack/src/javaRestTest/java/org/elasticsearch/xpack/esql/heap_attack/HeapAttackIT.java b/...p-attack/src/javaRestTest/java/org/elasticsearch/xpack/esql/heap_attack/HeapAttackIT.java
@@ -129,6 +129,15 @@ public void testSortByManyLongsTooMuchMemoryAsync() throws IOException {
                 try {
                     resp = client().performRequest(fetch);
                 } catch (ResponseException e) {
+                    if (e.getResponse().getStatusLine().getStatusCode() == 403) {
+                        /*
+                         * There's a bug when loading from the translog with security
+                         * enabled. If we retry a few times we'll load from the index
+                         * itself and should succeed.
+                         */
+                        logger.error("polled for results got 403");
+                        continue;
+                    }
                     if (e.getResponse().getStatusLine().getStatusCode() == 404) {
                         logger.error("polled for results got 404");
                         continue;