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.

Sign up for GitHub

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

Jump to bottom

Allow .NET PR build pipeline to run Analyze in-parallel with tests #14970

Closed
MiYanni opened this issue Sep 8, 2020 · 3 comments
Closed

Allow .NET PR build pipeline to run Analyze in-parallel with tests #14970

MiYanni opened this issue Sep 8, 2020 · 3 comments
Assignees
@pakrym
Labels
Client This issue points to a problem in the data-plane of the library. EngSys This issue is impacting the engineering system.

Comments

@MiYanni
Copy link
Contributor

MiYanni commented Sep 8, 2020

Currently, the Analyze step runs after all tests steps have passed. I'm told that this was for a reason where it would download the package and use it. But now, it does build and run the generation steps so there is potential to parallelize it. So, the ask is to have Analyze run in-parallel with the Test steps. Basically, it would go: Build -> [Everything else in parallel].
@weshaggard weshaggard transferred this issue from Azure/azure-sdk-tools Sep 8, 2020
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Sep 8, 2020
@weshaggard weshaggard added the EngSys This issue is impacting the engineering system. label Sep 8, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Sep 8, 2020
@weshaggard
Copy link
Member

@pakrym does the code generation steps now always build all the packages? If so perhaps we can start doing this step in parallel and not wait to download the packages.

@pakrym
Copy link
Contributor

pakrym commented Sep 8, 2020 •
edited
Loading

The code generation step doesn't build packages but Export-API does build dlls.

@kurtzeborn kurtzeborn added the Client This issue points to a problem in the data-plane of the library. label Sep 15, 2020
@kurtzeborn
Copy link
Member

@pakrym, @weshaggard mentioned you have a PR for this already.

@pakrym pakrym closed this as completed Sep 15, 2020
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-net that referenced this issue Jun 30, 2021
CodeGen from PR 14970 in Azure/azure-rest-api-specs
8521b34 
[Hub Generated] Review request for Microsoft.Media to add version stable/2021-06-01  Part-1 (Azure#14970)

* Adds base for updating Microsoft.Media from version stable/2021-05-01 to version 2021-06-01

* Updates readme

* Updates API version in new specs and examples

* Update the branch with copy of all other services 2020-05-01 version files except encoding.

* update AMS to 2021-06-01 Part1 (All swaggers except encoding)

* fix some lint errors.

* Fix an unreferenced file.

* fix more linting errors

* remove some unreferenced files.

* Fix systemData in streamingservice

* fix more linting errors

* fix 404s

* fix more lint errors

* fix 204 issue for delete

* fix definitions to use v2 common-types

* remove type for the live event list list defenition

* Revert "remove type for the live event list list defenition"

This reverts commit 0f66253d284284037bf2c0d7a04c947d4dca7715.

* removed unused definition
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pakrym pakrym

Labels
Client This issue points to a problem in the data-plane of the library. EngSys This issue is impacting the engineering system.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pakrym @weshaggard @MiYanni @kurtzeborn