Fix incorrect node context creation - we correctly only ever log the cache-based projectstarted messages from the scheduler node. Before we were incorrectly saying they came from the remote node.

Author: baronfel

src/Build/BackEnd/Components/Logging/LoggingServiceLogMethods.cs

@@ -574,7 +574,7 @@ public ProjectStartedEventArgs CreateProjectStartedForCachedProject(
             string targetNames,
             string toolsVersion)
         {
-            Debug.Assert(remoteNodeEvaluationBuildEventContext.NodeId == Scheduler.InProcNodeId, "Cached project build events should only be logged from the in-proc scheduler node.");
+            Debug.Assert(currentNodeBuildEventContext.NodeId == Scheduler.VirtualNode, "Cached project build events should only be logged from the scheduler virtual node.");
 
             BuildEventContext validatedRemoteNodeEvaluationBuildContext = ValidatePreexistingProjectContextId(
                 remoteNodeEvaluationBuildEventContext,

src/Build/BackEnd/Components/Scheduler/Scheduler.cs

@@ -2202,8 +2202,7 @@ private ScheduleResponse GetResponseForResult(int parentRequestNode, BuildReques
         private void LogRequestHandledFromCache(BuildRequest request, BuildResult result)
         {
             BuildRequestConfiguration configuration = _configCache[request.ConfigurationId];
-            int nodeId = _schedulingData.GetAssignedNodeForRequestConfiguration(request.ConfigurationId);
-            NodeLoggingContext nodeContext = new NodeLoggingContext(_componentHost.LoggingService, s_schedulerNodeBuildEventContext, nodeId, true);
+            NodeLoggingContext nodeContext = new NodeLoggingContext(_componentHost.LoggingService, s_schedulerNodeBuildEventContext, VirtualNode, true);
             nodeContext.LogRequestHandledFromCache(request, configuration, result);
 
             TraceScheduler(