Skip to content

Commit

Permalink
fix PYTHONPATH on Linux (#57701)
Browse files Browse the repository at this point in the history
* fix PYTHONPATH on Linux for issue#1928

* fix PYTHONPATH on Linux for issue#1928

* roll back setting PYTHONPATH, set EnableAzurePipelinesReporter to false instead

* set EnableAzurePipelinesReporter to false since perf test doesn't use the reporter. This avoids a helix crash
  • Loading branch information
Lxiamail authored Aug 20, 2021
1 parent 4209a95 commit ec25238
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
5 changes: 4 additions & 1 deletion eng/testing/performance/crossgen_perf.proj
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@
</HelixCorrelationPayload>
</ItemGroup>

<PropertyGroup>
<EnableAzurePipelinesReporter>false</EnableAzurePipelinesReporter>
</PropertyGroup>
<!--
Crossgen and Crossgen2 Scenario WorkItems
-->
Expand Down Expand Up @@ -84,4 +87,4 @@
<Timeout>4:00</Timeout>
</HelixWorkItem>
</ItemGroup>
</Project>
</Project>
2 changes: 1 addition & 1 deletion eng/testing/performance/microbenchmarks.proj
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
<Python>py -3</Python>
<CoreRun>%HELIX_CORRELATION_PAYLOAD%\Core_Root\CoreRun.exe</CoreRun>
<BaselineCoreRun>%HELIX_CORRELATION_PAYLOAD%\Baseline_Core_Root\CoreRun.exe</BaselineCoreRun>

<HelixPreCommands>$(HelixPreCommands);call %HELIX_CORRELATION_PAYLOAD%\performance\tools\machine-setup.cmd;set PYTHONPATH=%HELIX_WORKITEM_PAYLOAD%\scripts%3B%HELIX_WORKITEM_PAYLOAD%</HelixPreCommands>
<ArtifactsDirectory>%HELIX_CORRELATION_PAYLOAD%\artifacts\BenchmarkDotNet.Artifacts</ArtifactsDirectory>
<BaselineArtifactsDirectory>%HELIX_CORRELATION_PAYLOAD%\artifacts\BenchmarkDotNet.Artifacts_Baseline</BaselineArtifactsDirectory>
Expand Down Expand Up @@ -81,6 +80,7 @@

<PropertyGroup>
<PartitionCount>30</PartitionCount>
<EnableAzurePipelinesReporter>false</EnableAzurePipelinesReporter>
</PropertyGroup>
<ItemGroup>
<Partition Include="$(BuildConfig).Partition0" Index="0" />
Expand Down

0 comments on commit ec25238

Please sign in to comment.