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

[Windows] Update antrea-agent image using Server 2022 #6265

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wenyingd
Copy link
Contributor

@wenyingd wenyingd commented Apr 26, 2024

Bump up the basic image for Antrea Agent on Windows to Server 2022

The change includes,

  • Use servercore:ltsc2022 in "antrea/base-windows". This is because running "sh" command is not supported in nanoserver:ltsc2022, it returns error code "-1073741511", but in Server2019, it returns 0. This caused we can't use nanoserver:ltsc2022 as a base to compile the antrea-agent binary any more.
  • Use Windows "curl" command to download the utilities in base-windows, and drop the previous powershell command "Invoke-WebRequest -UseBasicParsing". This is because the Windows curl command is more efficiency.
  • The final antrea Windows image "antrea/antrea-windows" is built based on mcr.microsoft.com/powershell:lts-nanoserver-ltsc2022.

Fix: #6290

Bump up the basic image for Antrea Agent on Windows to Server 2022

The change includes,
 * Use servercore:ltsc2022 in "antrea/base-windows". This is because running
   "sh" command is not supported in nanoserver:ltsc2022, it returns error code
   "-1073741511", but in Server2019, it returns 0. This caused we can't use
   nanoserver:ltsc2022 as a base to compile the antrea-agent binary any more.
 * Use Windows "curl" command to download the utilities in base-windows, and
   drop the previous powershell command "Invoke-WebRequest -UseBasicParsing".
   This is because the Windows curl command is more efficiency.
 * The final antrea Windows image "antrea/antrea-windows" is built based on
   mcr.microsoft.com/powershell:lts-nanoserver-ltsc2022.

Signed-off-by: Wenying Dong <wenyingd@vmware.com>
@wenyingd wenyingd requested a review from XinShuYang May 6, 2024 06:29
@wenyingd wenyingd changed the title [WIP][Windows] Update antrea-agent image using Server 2022 [Windows] Update antrea-agent image using Server 2022 May 6, 2024
@@ -1 +1 @@
1809
ltsc2022
Copy link
Contributor

Choose a reason for hiding this comment

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

After adding support for Windows Server 2022 in Antrea, should we also announce that we will no longer maintain Antrea on Windows Server 2019 at the same time?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think so...

Copy link
Contributor

Choose a reason for hiding this comment

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

@wenyingd I also created a PR to add base-image verification pipeline in CI, could you review it first? #6262

@luolanzone luolanzone added the area/OS/windows Issues or PRs related to the Windows operating system. label May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/OS/windows Issues or PRs related to the Windows operating system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Antrea Agent Windows container image to use Server 2022 as a base
3 participants