PR Pipeline - Build and Local Tests#4325
Closed
benrr101 wants to merge 54 commits into
Closed
Conversation
# Conflicts: # eng/pipelines/common/steps/restore-dotnet-tools.yml
Leaving the unused build step for now just in case we need it later
…bles to OneBranch variables file
Move a couple things around WIP for manual test configuration generation
…TestExecuteReaderAsyncWithLargeQuery
…s_SqlAuth as flay
Moving the SA password retrieval deeper into the manual test job
… a bunch of tiny little changes?
benrr101
added
the
Area\Engineering
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
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.
Description
As the goal of this sprint, we want to create a new PR pipeline that simplifies all the complexities of the old one, reduces the test coverage to tests that can run simply and quickly, and completes quickly. This PR introduces the basics of that with a few more changes to come.
This puts the total run time at ~40min in its current state. Compared to the current pipeline which has full runtimes of anywhere from 1.5hrs to 3hrs, this is a massive improvement.
Noteworthy notes have been added inline.
Remaining work
Testing
https://sqlclientdrivers.visualstudio.com/ADO.Net/_build/results?buildId=154247&view=results
https://dev.azure.com/SqlClientDrivers/public/_build/results?buildId=154249&view=results
https://sqlclientdrivers.visualstudio.com/ADO.Net/_build/results?buildId=154095&view=results