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 netapi32.dll to windows images #1265

Merged
merged 1 commit into from
May 18, 2021
Merged

Add netapi32.dll to windows images #1265

merged 1 commit into from
May 18, 2021

Conversation

johnSchnake
Copy link
Contributor

@johnSchnake johnSchnake commented May 17, 2021

This missing dll is required to run may containerized golang apps
since anything checking the current user via the os package
relies on this. It so happens that klog does this and so its common
for some dependency to end up with the issue.

We can continue to shift to klog/v2 which may fix the issue, but
it would could get hit via another import as well.

Fixes #1263

Signed-off-by: John Schnake [email protected]

This missing dll is required to run may containerized golang apps
since anything checking the current user via the os package
relies on this. It so happens that klog does this and so its common
for some dependency to end up with the issue.

We can continue to shift to klog/v2 which may fix the issue, but
it would could get hit via another import as well.

Fixes #1263

Signed-off-by: John Schnake <[email protected]>
@johnSchnake johnSchnake requested a review from wilsonehusin May 17, 2021 19:30
@johnSchnake johnSchnake merged commit 63d1573 into vmware-tanzu:master May 18, 2021
@johnSchnake johnSchnake deleted the windowsDLLFix branch May 18, 2021 13:50
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.

Nanoserver is missing DLLs required
1 participant