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

Where applicable, update Windows pools used to azsdk-pool-mms-win-2022-general and rename vmImage to the windows-20xx format #3152

Merged
merged 5 commits into from
Feb 21, 2023

Conversation

konrad-jamrozik
Copy link

This PR is a follow-up to:

Where applicable, update Windows pools used to azsdk-pool-mms-win-2022-general and rename vmImage to the windows-20xx format.

This discussion explains why we chose given vmImage format:

Mike Harder: 1ES Hosted Pool image name changes
posted in Azure SDK / Engineering System 🛠️ at Wednesday, February 15, 2023 12:10 PM

For further context, please see:

@konrad-jamrozik konrad-jamrozik added the Central-EngSys This issue is owned by the Engineering System team. label Feb 17, 2023
@konrad-jamrozik konrad-jamrozik self-assigned this Feb 17, 2023
@konrad-jamrozik
Copy link
Author

konrad-jamrozik commented Feb 17, 2023

@mikeharder what to do about lines like this one?

pool: server

Convert it to Ubuntu or Windows? I mean, eventually. For now I am tempted to just leave it be.

@konrad-jamrozik
Copy link
Author

@jsquire @lirenhe @m-nash can you kindly approve this PR? Thank you!

@mikeharder
Copy link
Member

@mikeharder what to do about lines like this one?

pool: server

Convert it to Ubuntu or Windows? I mean, eventually. For now I am tempted to just leave it be.

Any server jobs should be left alone. They are the best kind of job that doesn't even require an agent:

https://learn.microsoft.com/en-us/azure/devops/pipelines/process/phases?view=azure-devops&tabs=yaml#server-jobs

@konrad-jamrozik konrad-jamrozik merged commit 0e2bf30 into feature/v3 Feb 21, 2023
@konrad-jamrozik konrad-jamrozik deleted the users/kojamroz/win_2022 branch February 21, 2023 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants