Merge pull request #3432 from artilleryio/fix/fargate-retries-launch-…

…count
artilleryio · Dec 11, 2024 · 7ae41b4 · 7ae41b4
2 parents 4a6a5aa + 83cfac3
commit 7ae41b4
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/packages/artillery/lib/platform/aws-ecs/legacy/run-cluster.js b/packages/artillery/lib/platform/aws-ecs/legacy/run-cluster.js
@@ -1713,6 +1713,19 @@ async function ecsRunTask(context) {
 
     try {
       const runData = await ecs.runTask(params).promise();
+
+      const launchedTasksCount = runData.tasks?.length || 0;
+      tasksRemaining -= launchedTasksCount;
+
+      if (launchedTasksCount > 0) {
+        const newTaskArns = runData.tasks.map((task) => task.taskArn);
+        context.taskArns = context.taskArns.concat(newTaskArns);
+        artillery.globalEvents.emit('metadata', {
+          platformMetadata: { taskArns: newTaskArns }
+        });
+        debug(`Launched ${launchedTasksCount} tasks`);
+      }
+
       if (runData.failures.length > 0) {
         artillery.log('Some workers failed to start');
         const uniqueReasons = [
@@ -1723,19 +1736,6 @@ async function ecsRunTask(context) {
         await sleep(10 * 1000);
         throw new Error('Not enough ECS capacity');
       }
-
-      if (runData.tasks?.length > 0) {
-        const newTaskArns = runData.tasks.map((task) => task.taskArn);
-        context.taskArns = context.taskArns.concat(newTaskArns);
-        artillery.globalEvents.emit('metadata', {
-          platformMetadata: { taskArns: newTaskArns }
-        });
-        debug(`Launched ${launchCount} tasks`);
-        tasksRemaining -= launchCount;
-        await sleep(250);
-      } else {
-        retries++;
-      }
     } catch (runErr) {
       if (runErr.code === 'ThrottlingException') {
         artillery.log('ThrottlingException returned from ECS, retrying');