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 host process default path is showing as c:\ instead of c:\hpc on AKS cluster version 1.28.5 #4196

Closed
anilkumar1619 opened this issue Apr 5, 2024 · 3 comments

Comments

@anilkumar1619
Copy link

Hello Team,

I am trying to deploy windows host process for containers by following below document and when checked, I noticed the default exec path in the container is showing as c:\ instead of c:\hpc.

https://learn.microsoft.com/en-us/azure/aks/use-windows-hpc

This is working as expected in 1.27.x whereas it is incorrect in 1.28.5. In 1.28.x container, $env.CONTAINER_SANDBOX_MOUNT_POINT is set to c:\hpc but while connecting it is by default taking to c:\

As per below Github issues and the document it says C:\hpc is the right path.

#3885

https://github.com/kubernetes/enhancements/tree/master/keps/sig-windows/1981-windows-privileged-container-support#container-mounts

image

Below are the screenshots for reference.

Left, 1.27.9, default working folder is expected C:\C:{container-id} which is correct.
Right, 1.28.5, default working folder is supposed to be C:\hpc but it turns out to be C:\ instead.
image

Kindly check and assist if this is expected or not.

@AbelHu AbelHu self-assigned this Apr 8, 2024
@AbelHu
Copy link
Member

AbelHu commented Apr 15, 2024

Workaround: Switch the working directory to $env:CONTAINER_SANDBOX_MOUNT_POINT before running the commands
Will be fixed in containerd/containerd#10069

@AbelHu
Copy link
Member

AbelHu commented Jun 27, 2024

The fix containerd/containerd#10069 is included in Windows containerd v1.7.17 which is being rolled out in AKS RP release v20240609. You can track the release progress in https://releases.aks.azure.com/webpage/index.html#tabversion.

Copy link
Contributor

Thanks for reaching out. I'm closing this issue as it was marked with "Fix released" and it hasn't had activity for 7 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants