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

[release/6.0] Update 10.15 to 12.00 queues #80030

Merged
merged 1 commit into from
Jan 4, 2023

Conversation

hoyosjs
Copy link
Member

@hoyosjs hoyosjs commented Dec 29, 2022

Fixes Issue #80029

The Helix queues we use for osx x64 are being decommissioned on 1/1/23. Normalize testing to 12.00.

Customer Impact

Testing only fix - needed to keep PRs green and testing working.

@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost ghost assigned hoyosjs Dec 29, 2022
@hoyosjs hoyosjs changed the title Update 10.15 to 12.00 queues [release/6.0] Update 10.15 to 12.00 queues Dec 29, 2022
@ViktorHofer
Copy link
Member

Note that same as in #79665, we can't merge this one in yet.

@ViktorHofer
Copy link
Member

@hoyosjs we also use that queue in main and presumably also in release/7.0. Will you send PRs for those branches as well?

@hoyosjs
Copy link
Member Author

hoyosjs commented Jan 4, 2023

7.0 uses 12.00 except for emulators. That's the only reason I didn't update it - I didn't know how to test it well and it's not automated (PRs into servicing didn't hit the deprecation warning).

@hoyosjs hoyosjs deleted the juhoyosa/6.0-queues branch January 4, 2023 07:58
@ViktorHofer
Copy link
Member

Yes because 7.0 and main set TreatWarningsAsErrors=false in the helix step. The warning is still present and the legs will start failing when the queue is removed.

@hoyosjs
Copy link
Member Author

hoyosjs commented Jan 5, 2023

Still, are emulators tested on PR? I accept I know very little of our device testing. I can definitely open PRs as needed. cc: @steveisok

@akoeplinger
Copy link
Member

akoeplinger commented Jan 11, 2023

@hoyosjs yes please update main and 7.0 as well, you can test the simulators in main via /azp run runtime-ioslikesimulator

@ghost ghost locked as resolved and limited conversation to collaborators Feb 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants