Skip to content

Windows: add smoke test Dockerfiles with full Windows#533

Open
speednoisemovement wants to merge 2 commits intoswiftlang:mainfrom
speednoisemovement:smoke_test_full
Open

Windows: add smoke test Dockerfiles with full Windows#533
speednoisemovement wants to merge 2 commits intoswiftlang:mainfrom
speednoisemovement:smoke_test_full

Conversation

@speednoisemovement
Copy link
Copy Markdown
Contributor

The current Dockerfiles use Microsoft's servercore base images. These are cut down enough that they will not run many real Windows Swift applications: for example, swift-win32 fails because the DLL responsible for app lifecycle notifications, win-core-psm-appnotify-l1-1-0.dll, is missing on servercore:1809.

This introduces new Dockerfiles based on windows:1809 and windows/server:ltsc2022 which provide a better test environment.

Other than base images, the only difference between these and the existing servercore-based images is the addition of -ExecutionPolicy Bypass since the full images default to Restricted and this is incompatible with running a PowerShell script as the entry point.

The current Dockerfiles use Microsoft's `servercore`
base images. These are cut down enough that they will
not run many real Windows Swift applications: for
example, `swift-win32` fails because the DLL responsible
for app lifecycle notifications, `win-core-psm-appnotify-l1-1-0.dll`,
is missing on `servercore:1809`.

This introduces new Dockerfiles based on `windows:1809` and
`windows/server:ltsc2022` which provide a better test
environment.

Other than base images, the only difference between these
and the existing `servercore`-based images is the addition
of `-ExecutionPolicy Bypass` since the full images default
to `Restricted` and this is incompatible with running a
PowerShell script as the entry point.
@speednoisemovement
Copy link
Copy Markdown
Contributor Author

@justice-adams-apple @shahmishal does this seem OK?

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