From 7036627d4717dadc479a2917b6f79859681fd606 Mon Sep 17 00:00:00 2001 From: Wes Walker Date: Fri, 9 Dec 2022 10:37:44 +0100 Subject: [PATCH] Ensure collected issues are processed only by a non-embedded ExecutionContext. This was already implicit. Now, just making it explicit. --- src/Runner.Worker/ExecutionContext.cs | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/Runner.Worker/ExecutionContext.cs b/src/Runner.Worker/ExecutionContext.cs index 465e2ed14..9034a9b87 100644 --- a/src/Runner.Worker/ExecutionContext.cs +++ b/src/Runner.Worker/ExecutionContext.cs @@ -440,11 +440,14 @@ namespace GitHub.Runner.Worker _record.Result = _record.Result ?? TaskResult.Succeeded; _record.State = TimelineRecordState.Completed; - foreach (var issue in _embeddedIssueCollector) + // Before our main timeline's final QueueTimelineRecordUpdate, + // inject any issues collected by embedded ExecutionContexts. + if (!this.IsEmbedded) { - // Before our main timeline's final QueueTimelineRecordUpdate, - // inject any issues collected by embedded ExecutionContexts. - AddIssue(issue); + foreach (var issue in _embeddedIssueCollector) + { + AddIssue(issue); + } } _jobServerQueue.QueueTimelineRecordUpdate(_mainTimelineId, _record);