feat(container): Refactor handling of Docker push command (#228)

prom3theu5 · Jun 20, 2024 · 19f5dfb · 19f5dfb
1 parent 66b0e5c
commit 19f5dfb
Showing 1 changed file with 20 additions and 11 deletions.
diff --git a/src/Aspirate.Services/Implementations/ContainerCompositionService.cs b/src/Aspirate.Services/Implementations/ContainerCompositionService.cs
@@ -65,23 +65,32 @@ private async Task<ShellCommandResult> PushContainer(string builder, string? reg
     {
         if (!string.IsNullOrEmpty(registry))
         {
-            var pushArgumentBuilder = ArgumentsBuilder
-                .Create()
-                .AppendArgument(DockerLiterals.PushCommand, string.Empty, quoteValue: false);
+            ShellCommandResult? result = null;
 
             foreach (var fullImage in fullImages)
             {
+                var pushArgumentBuilder = ArgumentsBuilder
+                    .Create()
+                    .AppendArgument(DockerLiterals.PushCommand, string.Empty, quoteValue: false);
+
                 pushArgumentBuilder.AppendArgument(fullImage.ToLower(), string.Empty, quoteValue: false, allowDuplicates: true);
-            }
 
-            return await shellExecutionService.ExecuteCommand(
-                new()
+                result = await shellExecutionService.ExecuteCommand(
+                    new()
+                    {
+                        Command = builder,
+                        ArgumentsBuilder = pushArgumentBuilder,
+                        NonInteractive = nonInteractive.GetValueOrDefault(),
+                        ShowOutput = true,
+                    });
+
+                if (!result.Success)
                 {
-                    Command = builder,
-                    ArgumentsBuilder = pushArgumentBuilder,
-                    NonInteractive = nonInteractive.GetValueOrDefault(),
-                    ShowOutput = true,
-                });
+                    break;
+                }
+            }
+
+            return result;
         }
 
         return new ShellCommandResult(true, string.Empty, string.Empty, 0);