fix for issue #2009 - composite summary file (#2077)

This commit is contained in:
Stefan Ruvceski
2022-09-12 20:51:36 +02:00
committed by GitHub
parent ed191b78ae
commit 6e6410d300

View File

@@ -68,7 +68,7 @@ namespace GitHub.Runner.Worker
{ {
try try
{ {
fileCommand.ProcessCommand(context, Path.Combine(_fileCommandDirectory, fileCommand.FilePrefix + _fileSuffix),container); fileCommand.ProcessCommand(context, Path.Combine(_fileCommandDirectory, fileCommand.FilePrefix + _fileSuffix), container);
} }
catch (Exception ex) catch (Exception ex)
{ {
@@ -118,7 +118,7 @@ namespace GitHub.Runner.Worker
if (File.Exists(filePath)) if (File.Exists(filePath))
{ {
var lines = File.ReadAllLines(filePath, Encoding.UTF8); var lines = File.ReadAllLines(filePath, Encoding.UTF8);
foreach(var line in lines) foreach (var line in lines)
{ {
if (line == string.Empty) if (line == string.Empty)
{ {
@@ -312,7 +312,9 @@ namespace GitHub.Runner.Worker
} }
} }
var attachmentName = context.Id.ToString(); var attachmentName = !context.IsEmbedded
? context.Id.ToString()
: context.EmbeddedId.ToString();
Trace.Info($"Queueing file ({filePath}) for attachment upload ({attachmentName})"); Trace.Info($"Queueing file ({filePath}) for attachment upload ({attachmentName})");
// Attachments must be added to the parent context (job), not the current context (step) // Attachments must be added to the parent context (job), not the current context (step)