feature/367 fix: skip container if pre-update command fails

containrrr · Mar 28, 2020 · c1a0da9 · c1a0da9
1 parent fac26df
commit c1a0da9
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/internal/actions/update.go b/internal/actions/update.go
@@ -74,9 +74,13 @@ func stopStaleContainer(container container.Container, client container.Client,
 		return
 	}
 	if params.LifecycleHooks {
-		lifecycle.ExecutePreUpdateCommand(client, container)
+		if err := lifecycle.ExecutePreUpdateCommand(client, container); err != nil {
+			log.Error(err)
+			log.Info("Skipping container as the pre-update command failed")
+			return
+		}
 	}
-
+	
 	if err := client.StopContainer(container, params.Timeout); err != nil {
 		log.Error(err)
 	}
@@ -139,4 +143,4 @@ func checkDependencies(containers []container.Container) {
 			}
 		}
 	}
-}
+}