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

feat: adds windows server 2022 #660

Merged
merged 2 commits into from
Sep 28, 2021
Merged

Conversation

nick5616
Copy link
Contributor

Reason for Change:

Adds Windows Server 2022 to Makefile, BASEIMAGE, and BASEIMAGE_CORE

Requirements

  • squashed commits
  • included documentation
  • added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

  • Yes
  • No

If "Yes," did you notify that project's maintainers and provide attribution?

Special Notes for Reviewers:
ltsc2022 is the MCR tag for Windows Server 2022
https://hub.docker.com/_/microsoft-windows-nanoserver
There is a PR for the same thing here:
kubernetes-sigs/azuredisk-csi-driver#1030
kubernetes-sigs/secrets-store-csi-driver#756

@nilekhc
Copy link
Contributor

nilekhc commented Sep 27, 2021

@nick5616 Thanks for the PR!

Could you re-base?

@codecov-commenter
Copy link

Codecov Report

Merging #660 (882e126) into master (9c49995) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #660   +/-   ##
=======================================
  Coverage   62.74%   62.74%           
=======================================
  Files           7        7           
  Lines         765      765           
=======================================
  Hits          480      480           
  Misses        252      252           
  Partials       33       33           

@aramase
Copy link
Member

aramase commented Sep 28, 2021

/azp run pr-e2e

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@aramase
Copy link
Member

aramase commented Sep 28, 2021

/azp run pr-e2e-azure

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@nilekhc nilekhc left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@aramase aramase left a comment

Choose a reason for hiding this comment

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

/lgtm

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.

None yet

4 participants