Add "[DRY RUN]" to process execution output logging (#1695)

lbussell · web-flow · commit 41c28c390600 · 2025-05-06T13:07:53.000-07:00
diff --git a/src/Microsoft.DotNet.ImageBuilder/src/ExecuteHelper.cs b/src/Microsoft.DotNet.ImageBuilder/src/ExecuteHelper.cs
@@ -75,7 +75,8 @@ private static string Execute(
         {
             info.RedirectStandardError = true;
             executeMessageOverride ??= $"{info.FileName} {info.Arguments}";
-            s_loggerService.WriteSubheading($"EXECUTING: {executeMessageOverride}");
+            string prefix = isDryRun ? "EXECUTING [DRY RUN]" : "EXECUTING";
+            s_loggerService.WriteSubheading($"{prefix}: {executeMessageOverride}");
 
             if (isDryRun)
             {
@@ -91,7 +92,7 @@ private static string Execute(
             if (processResult.Process.ExitCode != 0)
             {
                 string exceptionMsg = errorMessage ?? $@"Failed to execute {info.FileName} {info.Arguments}
-                
+
                 {processResult.StandardError}";
 
                 throw new InvalidOperationException(exceptionMsg);

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,8 @@ private static string Execute(`
`75`	`75`	`{`
`76`	`76`	`info.RedirectStandardError = true;`
`77`	`77`	`executeMessageOverride ??= $"{info.FileName} {info.Arguments}";`
`78`		`- s_loggerService.WriteSubheading($"EXECUTING: {executeMessageOverride}");`
	`78`	`+ string prefix = isDryRun ? "EXECUTING [DRY RUN]" : "EXECUTING";`
	`79`	`+ s_loggerService.WriteSubheading($"{prefix}: {executeMessageOverride}");`
`79`	`80`
`80`	`81`	`if (isDryRun)`
`81`	`82`	`{`
`@@ -91,7 +92,7 @@ private static string Execute(`
`91`	`92`	`if (processResult.Process.ExitCode != 0)`
`92`	`93`	`{`
`93`	`94`	`string exceptionMsg = errorMessage ?? $@"Failed to execute {info.FileName} {info.Arguments}`
`94`		`-`
	`95`	`+`
`95`	`96`	`{processResult.StandardError}";`
`96`	`97`
`97`	`98`	`throw new InvalidOperationException(exceptionMsg);`