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

Azure DevOps Pipeline Optimizations and Improvements #395

Merged

Conversation

NightOwl888
Copy link
Contributor

  • Changed dotnet publish to be for the whole solution instead of one project at a time using the undocumented IsPublishable MSBuild property.
  • Fixed package dependency conflicts.
  • Switched to dotnet build instead of dotnet msbuild.
  • Switched to dotnet test instead of dotnet mstest.
  • Commented out extra build steps that are meant for debugging purposes.
  • Added GeneratePackages option to skip dotnet pack and subsequent upload of artifacts for faster testing.
  • Added option to disable installing .NET SDK during build (which was double-installing it).
  • Fail the build if the test runner crashes.
  • Added the --blame-hang-timeout option to detect when a test doesn't finish and to provide diagnostic info for debugging. For now, the timeout is hard-coded at 10 minutes (per test) for a regular build and 35 minutes for a nightly build.

…t up only the test projects and tools as publishable
…Cryptography.Xml reference (netcoreapp3.1) so there is no conflict
…et45) so there is no conflict with Microsoft.NET.Test.Sdk
…test projects to Microsoft.Extensions.Configuration.Abstractions (net48) to 2.0.0
…ions and Microsoft.AspNetCore.TestHost to 2.0.0, and upgraded Lucene.Net.Replicator.AspNetCore to net461.
…tensions.Configuration so they will use 2.0.0 for all test projects
…ncyInjection.Abstractions (net48) to 2.0.0 in tests
…e (net48) to Microsoft.Extensions.Options 2.0.0
…vstest, added logic to copy test binaries to a temporary folder for the run
…epending on whether it is a nightly or weekly build or a regular build
…uild.BinariesDirectory, since the latter is not cleaned up
…test runner crashes or hangs (which also causes a crash)
@NightOwl888 NightOwl888 merged commit e814b28 into apache:master Nov 24, 2020
@NightOwl888 NightOwl888 added this to the 4.8.0-beta00014 milestone Feb 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant