From 447a2d0fd2d1a9d647aa0d0723a6e9255372f261 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?David=20Test=C3=A9?= <david.teste@fratris.fr>
Date: Wed, 17 Jul 2024 09:59:44 +0200
Subject: [PATCH] feat(slab): wait slab until async task is over

This is done to handle various timeouts configuration on spawn
request in Slab. Until now the timeout was set to 15 minutes. This
was fine since default spawn timeout in Slab was set to 10 minutes.
However in some cases, slab backend profiles can request a longer
duration before timeout.
Since the asynchronous task created in Slab will eventually
finish, we now simply wait for this moment.
---
 dist/index.js | 10 ++++------
 src/slab.js   | 10 ++++------
 2 files changed, 8 insertions(+), 12 deletions(-)

diff --git a/dist/index.js b/dist/index.js
index 343061f..359cd1b 100644
--- a/dist/index.js
+++ b/dist/index.js
@@ -49766,7 +49766,8 @@ async function startInstanceRequest() {
 }
 
 async function waitForInstance(taskId, taskName) {
-  for (let i = 0; i < 60; i++) {
+  // while (true) equivalent to please ESLint
+  for (;;) {
     await utils.sleep(15)
 
     try {
@@ -49788,19 +49789,16 @@ async function waitForInstance(taskId, taskName) {
           return
         }
       } else {
-        core.error(
+        core.setFailed(
           `Failed to wait for instance (HTTP status code: ${response.status})`
         )
+        return
       }
     } catch (error) {
       core.error('Failed to fetch or remove instance task')
       throw error
     }
   }
-
-  core.setFailed(
-    'Timeout while waiting for instance to be running after 15 mins.'
-  )
 }
 
 async function terminateInstanceRequest(runnerName) {
diff --git a/src/slab.js b/src/slab.js
index 3c550ea..f873fee 100644
--- a/src/slab.js
+++ b/src/slab.js
@@ -58,7 +58,8 @@ async function startInstanceRequest() {
 }
 
 async function waitForInstance(taskId, taskName) {
-  for (let i = 0; i < 60; i++) {
+  // while (true) equivalent to please ESLint
+  for (;;) {
     await utils.sleep(15)
 
     try {
@@ -80,19 +81,16 @@ async function waitForInstance(taskId, taskName) {
           return
         }
       } else {
-        core.error(
+        core.setFailed(
           `Failed to wait for instance (HTTP status code: ${response.status})`
         )
+        return
       }
     } catch (error) {
       core.error('Failed to fetch or remove instance task')
       throw error
     }
   }
-
-  core.setFailed(
-    'Timeout while waiting for instance to be running after 15 mins.'
-  )
 }
 
 async function terminateInstanceRequest(runnerName) {