[Helix] Try out the new dotnet test blame functionality #23862
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ghost
added
the
feature-platform
HaoK
added
area-infrastructure
|
cc @davidfowl |
|
@HaoK we need to configure the blame collector. I'll look into that |
|
Right, I just kicked this off as the initial PR starting point |
|
So looks like at as a starting point we already get ubuntu and windows dumps https://dev.azure.com/dnceng/public/_build/results?buildId=724969&view=ms.vss-test-web.build-test-results-tab&runId=22437750&paneView=dotnet-dnceng.dnceng-build-release-tasks.helix-test-information-tab&resultId=121647 MacOS is the only one that's missing the dmp |
HaoK
changed the title
…729.3 (#24456) Microsoft.Net.Compilers.Toolset From Version 3.8.0-1.20367.11 -> To Version 3.8.0-2.20379.3 Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
* Wait for component to mount before checking for output * Use Exists to check for element with timeout * Update ComponentRenderingTest.cs
* Add display environment * Print out env variables as part of helix runs * Update TestRunner.cs * Remove Debian 8 from helix matrix * Remove Debian 8 skip * Remove debian 8 skip * Remove debian 8 * Remove debian 8 skip * Remove Debian 8 skip * Remove debian 8 * Update Program.cs * Update TestRunner.cs * Update TestRunner.cs
…iles. (#24445) The output of the declaration file for Razor components are unaffected by all inputs other than the input .razor file. Consequently we can avoid regenerating these files if the output is newer than the input. This is the same heuristic we apply to Blazor WebAsssembly's compression artifacts. This PR combines these two improvements for a ~90ms (10%) improvement in the inner loop. ``` 17 ms GenerateBlazorWebAssemblyBootJson 1 calls 22 ms Copy 8 calls 39 ms ProcessFrameworkReferences 1 calls 40 ms RazorTagHelper 1 calls 51 ms ResolveAssemblyReference 1 calls 70 ms GetFileHash 1 calls 80 ms RazorGenerate 2 calls 111 ms Csc 2 calls Time Elapsed 00:00:00.95 ``` ``` 17 ms GenerateBlazorWebAssemblyBootJson 1 calls 21 ms Copy 8 calls 37 ms ProcessFrameworkReferences 1 calls 51 ms ResolveAssemblyReference 1 calls 70 ms Csc 1 calls 72 ms GetFileHash 1 calls 79 ms RazorGenerate 2 calls Time Elapsed 00:00:00.86 ``` In after: Csc calls reduced to one, RazorTagHelper call removed.
…724.1 (#24482) Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.Arcade.Sdk From Version 5.0.0-beta.20364.3 -> To Version 5.0.0-beta.20374.1 - downgrade of the Arcade SDK should be fine Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
* Put back InternalCalls * Removing .NET profiling calls * Remove JS side profiling
* Update documentation on darc * Apply suggestions from code review Co-authored-by: Doug Bunting <[email protected]> * Update code blocks in referenced assemblies Co-authored-by: Doug Bunting <[email protected]>
- now matches timeout in quarantined-pr.yml which uses the same queues - internal PRs may otherwise time out
- Annotated UseMiddleware and UseStartup to preserve constructors and public methods. - Annotated UseHub and MapHub preserve constructors and public methods. - Added a test to verify UseStartup and UseMiddleware works - The linker.xml preserves constructors all of the types that are registered in DI. This should be removed once we get the linker friendly DI changes.
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
Replaced by #25010 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add a hanging test to see if dmps work now