Merge pull request #107 from depot/update-service

depot · Sep 9, 2024 · 55f95be · 55f95be
2 parents d20515c + f3b4581
commit 55f95be
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/handlers/updater.ts b/src/handlers/updater.ts
@@ -37,12 +37,6 @@ async function checkForUpdates() {
 
   const newerThan = parseISO(req.newerThan)
 
-  try {
-    await ecs.send(new UpdateServiceCommand({cluster, service: serviceName, forceNewDeployment: true}))
-  } catch (err) {
-    console.error('Error updating service, ignoring', err)
-  }
-
   const {taskArns} = await ecs.send(new ListTasksCommand({cluster, serviceName}))
   if (!taskArns || taskArns.length === 0) return
   const {tasks} = await ecs.send(new DescribeTasksCommand({cluster, tasks: taskArns}))
@@ -55,6 +49,12 @@ async function checkForUpdates() {
     if (desiredStatus !== 'RUNNING') continue
     console.log('Found old task to upgrade', task.taskArn)
     await ecs.send(new StopTaskCommand({cluster, task: task.taskArn, reason: 'Upgrading'}))
+
+    try {
+      await ecs.send(new UpdateServiceCommand({cluster, service: serviceName, forceNewDeployment: true}))
+    } catch (err) {
+      console.error('Error updating service, ignoring', err)
+    }
   }
 }