Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release/6.0-preview2] Turn off LKG calculation #49075

Merged
merged 1 commit into from
Mar 4, 2021

Conversation

DrewScoggins
Copy link
Member

This change moves the performance infrastructure to use
the latest version of dotnet for building and running BDN.
This does not change what bits are tested.

This change moves the performance infrastructure to use
the latest version of dotnet for building and running BDN.
This does not change what bits are tested.
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@DrewScoggins
Copy link
Member Author

Not sure who I need to tag here or what the process for infrastructure changes are in release branches. @danmoseley

@danmoseley
Copy link
Member

Could you share why you're backporting ? I don't have an objection in principle. If you need this in order to have official numbers for preview 2, that seems a necessary part of shipping, and this is approved to merge.

@DrewScoggins
Copy link
Member Author

Yes, this is because we added the PriortityQueue tests, and the version of dotnet that we were using before cannot build the tests because it does not contain the type. I am going to do more validation to make sure that this fully fixes the issue, and then I will merge here.

@danmoseley
Copy link
Member

Actually, it just occurs to me that the last code for Preview 2 has presumably left this repo now... you presumably don't need your change in the product, but we should check that the branch is open. (@mmitche ?)

@mmitche
Copy link
Member

mmitche commented Mar 3, 2021

We still have a couple days if necessary, but under QB approval mode now.

@DrewScoggins
Copy link
Member Author

How do I get QB approval? I would really like to be able to get working runs out of this branch.

@danmoseley
Copy link
Member

My understanding is that the QB in this case is M2 - ie me. So unless @mmitche says otherwise, you can merge when tested/reviewed/green.

@DrewScoggins
Copy link
Member Author

Sorry at @danmoseley, but can you re-approve this? I had to make some changes to do some testing and then took them out, but that reset the approval.

@ghost
Copy link

ghost commented Mar 4, 2021

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

This change moves the performance infrastructure to use
the latest version of dotnet for building and running BDN.
This does not change what bits are tested.

Author: DrewScoggins
Assignees: -
Labels:

area-Infrastructure

Milestone: -

@mmitche mmitche merged commit 3eaf1f3 into dotnet:release/6.0-preview2 Mar 4, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Apr 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants