Merged PR 687317: Enable sourcehashes in fingerprintdata when storefingerprint is enabled as well

Sometimes we enable storeFingerprints without runtimecache miss analyzer feature. In those cases, we need to send the source hashes as well.

Author: semihokur

Public/Src/Engine/Scheduler/PipExecutor.cs

@@ -2274,8 +2274,8 @@ public static async Task<RunnableFromCacheResult> TryCheckProcessRunnableFromCac
         private static void PopulateFingerprintDataWithSourceHashes(BoxRef<ProcessFingerprintComputationEventData> processFingerprintComputationResult, Process pip, IPipExecutionEnvironment environment)
         {
             if (environment.Configuration.EnableDistributedSourceHashing() &&
-                environment.Configuration.Logging.CacheMissAnalysisOption.Mode != CacheMissMode.Disabled &&
-                environment.Configuration.Distribution.BuildRole == DistributedBuildRoles.Worker)
+                environment.Configuration.Distribution.BuildRole == DistributedBuildRoles.Worker &&
+                (environment.Configuration.Logging.StoreFingerprints == true || environment.Configuration.Logging.CacheMissAnalysisOption.Mode != CacheMissMode.Disabled))
             {
                 // If runtime cache miss analyzer is enabled and source file hashing is distributed, workers will report source file hashes back to the orchestrator
                 // because the orchestrator might not have the hashes for all source files.