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

Updated WindowsAppSDK-RunTestsInPipeline-Job.yml #3996

Merged
merged 3 commits into from
Nov 29, 2023

Conversation

kythant
Copy link
Contributor

@kythant kythant commented Nov 28, 2023

Pipeline runs seem to be failing consistently due to failure to acquire a co_refresh VM for testing.
OSGWiki says cobalt Home/Pro SKUs hit End of Support on Oct 10, 2023. That appears to explain it.
Skipping testing on co_refresh to unblock pipeline runs. Cobalt is 21h2. We are still testing the Windows.11.Enterprise.MultiSession.21h2 OS image which has another year before end of support, so we don't lose much test coverage with this change.

@kythant
Copy link
Contributor Author

kythant commented Nov 28, 2023

/azp run

@DrusTheAxe
Copy link
Member

"21H2" is both Win10 and Win11. You're referring to the Win11 21H2 (aka SV1)?

https://en.wikipedia.org/wiki/Windows_10_version_history
image

https://en.wikipedia.org/wiki/Windows_11_version_history
image

We are still testing the Windows.11.Enterprise.MultiSession.21h2...so we don't lose much test coverage with this change.

Zero loss if you count supported releases. Win11 21H2 Home/Pro supported ended last month (Oct'2023). Win11 21H2 Enterprise plus Win11 22H2 are still supported and their test pipelines are still humming so all good

@DrusTheAxe DrusTheAxe self-requested a review November 28, 2023 08:32
Copy link
Member

@DrusTheAxe DrusTheAxe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DrusTheAxe DrusTheAxe added the area-Infrastructure Build, test, source layout, package construction (TODO: move to Deployment, DeveloperTools) label Nov 28, 2023
@DrusTheAxe DrusTheAxe added this to the 1.5 milestone Nov 28, 2023
@alexlamtest alexlamtest self-requested a review November 28, 2023 08:35
@alexlamtest alexlamtest assigned alexlamtest and unassigned kythant Nov 28, 2023
@kythant kythant enabled auto-merge (squash) November 28, 2023 08:43
@kythant kythant disabled auto-merge November 29, 2023 00:25
@kythant kythant merged commit b5c7d00 into main Nov 29, 2023
@kythant kythant deleted the user/kythant/RemoveCorefresh branch November 29, 2023 00:26
kythant added a commit that referenced this pull request Nov 30, 2023
* Remove corefresh from tests

* also remove Server22_DC_arm64fre
kythant added a commit that referenced this pull request Nov 30, 2023
* Remove corefresh from tests

* also remove Server22_DC_arm64fre
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Infrastructure Build, test, source layout, package construction (TODO: move to Deployment, DeveloperTools)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants