fix(DeployJob): trigger pelias update even if server is already deployed

realCity · Sep 29, 2021 · 429f419 · 429f419
1 parent a8cf6bf
commit 429f419
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/src/main/java/com/conveyal/datatools/manager/jobs/DeployJob.java b/src/main/java/com/conveyal/datatools/manager/jobs/DeployJob.java
@@ -319,6 +319,16 @@ public void jobLogic () {
             }
         }
 
+        // Now that the build was successful, update Pelias
+        if (deployment.peliasUpdate) {
+            // Get log upload URI from deploy job
+            AmazonS3URI logUploadS3URI = getS3FolderURI();
+
+            // Execute the pelias update job and keep track of it
+            PeliasUpdateJob peliasUpdateJob = new PeliasUpdateJob(owner, "Updating Custom Geocoder Database", deployment, logUploadS3URI);
+            addNextJob(peliasUpdateJob);
+        }
+
         // Handle spinning up new EC2 servers for the load balancer's target group.
         if (otpServer.ec2Info != null) {
             if ("true".equals(DataManager.getConfigPropertyAsText("modules.deployment.ec2.enabled"))) {
@@ -341,15 +351,6 @@ public void jobLogic () {
             status.baseUrl = otpServer.publicUrl;
         }
 
-        // Now that the main instance is updated successfully, update Pelias
-        if (deployment.peliasUpdate) {
-            // Get log upload URI from deploy job
-            AmazonS3URI logUploadS3URI = getS3FolderURI();
-
-            // Execute the pelias update job and keep track of it
-            PeliasUpdateJob peliasUpdateJob = new PeliasUpdateJob(owner, "Updating Custom Geocoder Database", deployment, logUploadS3URI);
-            this.addNextJob(peliasUpdateJob);
-        }
 
         status.completed = true;
     }