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

Add support for Windows Server 2022 #557

Merged
merged 1 commit into from
Nov 1, 2021
Merged

Conversation

bdwyertech
Copy link
Contributor

Description

Add support for Windows Server 2022

https://aws.amazon.com/about-aws/whats-new/2021/09/aws-microsoft-windows-server-2022-amazon-ec2/

Issues Resolved

N/A I dont think

Check List

  • All tests pass. See TESTING.md for details.
  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@@ -134,7 +134,7 @@ def windows_version_parts

def windows_name_filter # rubocop:disable Metrics/MethodLength
major, revision, service_pack = windows_version_parts
if major == 2019 || major == 2016
if major == 2022 || major == 2019 || major == 2016
"Windows_Server-#{major}-English-Full-Base-*"
elsif major == 1709 || major == 1803
Copy link
Contributor Author

@bdwyertech bdwyertech Oct 9, 2021

Choose a reason for hiding this comment

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

This stuff can probably go -- 1709 and 1803 are no longer available as builds and the core structure follows same convention as GUI now

Screen Shot 2021-10-08 at 9 59 16 PM

Signed-off-by: Brian Dwyer <brian.dwyer@broadridge.com>
@tas50 tas50 merged commit 3f1e196 into test-kitchen:master Nov 1, 2021
@tas50
Copy link
Member

tas50 commented Nov 1, 2021

@bdwyertech If you want to cleanup this logic for the modern era that'd be great. I'd love to never have to update this again.

@bdwyertech bdwyertech deleted the win2022 branch November 2, 2021 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants