Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Conversation

brianrob
Copy link
Member

@brianrob brianrob commented Jun 6, 2017

Porting changes from master to enable Linux performance runs against release/2.0.0.

José Rivero added 2 commits June 6, 2017 13:37
- Removing `[assembly: MeasureInstructionsRetired]` from performance tests.
- Due to this issue: microsoft/xunit-performance#231, we are blocked from running on performance tests on Linux.
  The tag is not used by infrastructure and it is passed during runtime when profiling on Windows.
- Adding the option to execute CoreRun with the stability prefix.
- Moving file to be archived by Jenkins and updating groovy file.
- Setting the maximum number of iterations.
- This update contains a fix for this issue: microsoft/xunit-performance#230
- Consolidate the Api version in two places: `dependencies.prop` and `PerfHarness.csproj`
- Move performance log files to root directory where Jenkins can archive them
- Fixed the output to console, so it is easier to see on the Jenkins job the sccript steps live and get an exact repro step if needed
@brianrob brianrob requested a review from jorive June 6, 2017 20:39
@brianrob
Copy link
Member Author

brianrob commented Jun 6, 2017

@dotnet-bot test this please

@jorive
Copy link

jorive commented Jun 6, 2017

:shipit:

Copy link

@jorive jorive left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@brianrob
Copy link
Member Author

brianrob commented Jun 6, 2017

Fixes #12121.

@brianrob
Copy link
Member Author

brianrob commented Jun 6, 2017

@dotnet-bot test ci please

@brianrob brianrob merged commit fdd4e91 into dotnet:release/2.0.0 Jun 6, 2017
@brianrob brianrob deleted the linux_perf_runs branch June 6, 2017 23:55
@karelz karelz modified the milestone: 2.0.0 Aug 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants