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

[Help] Managed Windows Node Groups creating Linux nodes? #6158

Closed
dan-tw opened this issue Jan 16, 2023 · 7 comments · Fixed by #7681
Closed

[Help] Managed Windows Node Groups creating Linux nodes? #6158

dan-tw opened this issue Jan 16, 2023 · 7 comments · Fixed by #7681
Labels

Comments

@dan-tw
Copy link

dan-tw commented Jan 16, 2023

What help do you need?

I'm currently creating a new cluster to test out Managed Windows Node Groups by doing the following:

# cluster.yaml
# An example of ClusterConfig containing Windows and Linux node groups to support Windows workloads
---
apiVersion: eksctl.io/v1alpha5
kind: ClusterConfig

metadata:
  name: windows-cluster
  region: ap-southeast-2

managedNodeGroups:
  - name: windows-managed-ng
    amiFamily: WindowsServer2022FullContainer
    instanceType: g4dn.2xlarge
    minSize: 1
    maxSize: 1

Using the following version of eksctl: 0.125.0

The result is a cluster with a single Managed Node group that is running an Amazon Linux 2 EKS node (AMI ID: ami-0760f8e90a1753cfa) running kubernetes 1.24

I would've thought that the above config would create a Windows Managed Node Group?

eksctl output also states that it is enabling Windows IP addressing, so it must've registered the AMI Family:

2023-01-16 15:52:26 [ℹ]  
2 sequential tasks: { create cluster control plane "windows-cluster", 
    2 sequential sub-tasks: { 
        2 sequential sub-tasks: { 
            wait for control plane to become ready,
            enable Windows IP address management,
        },
        create managed nodegroup "windows-managed-ng",
    } 
}

Any idea what's gone wrong?

@dan-tw dan-tw added the kind/help Request for help label Jan 16, 2023
@github-actions
Copy link
Contributor

Hello dan-tw 👋 Thank you for opening an issue in eksctl project. The team will review the issue and aim to respond within 1-5 business days. Meanwhile, please read about the Contribution and Code of Conduct guidelines here. You can find out more information about eksctl on our website

@yuxiang-zhang
Copy link
Member

It looks like because of

[!]  WindowsServer2022FullContainer does not ship with NVIDIA GPU drivers installed, hence won't support running GPU-accelerated workloads out of the box

The WindowsServer2022FullContainer AMI Family using GPU instances is not mapped to any corresponding AMI Type.
https://github.com/weaveworks/eksctl/blob/e13718238bc9fc394a701f1095868bb927d3fda4/pkg/cfn/builder/managed_nodegroup.go#L295-L297

@yuxiang-zhang
Copy link
Member

Creating GPU instances with Windows AMI is not allowed, see #4238.

@dan-tw
Copy link
Author

dan-tw commented Jan 17, 2023

That explains it.

Perhaps eksctl should fail earlier, rather than spin up a cluster that does not actually do what is being requested?

For Windows, we are actually using our own custom AMI that does have GPU drivers installed and does advertise available GPU's to containers. I was mostly interested in seeing how the "Managed Windows Node Group" implementation worked for eksctl

@github-actions
Copy link
Contributor

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the stale label Feb 17, 2023
@github-actions
Copy link
Contributor

This issue was closed because it has been stalled for 5 days with no activity.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Feb 23, 2023
@TiberiuGC TiberiuGC reopened this Mar 20, 2024
@TiberiuGC
Copy link
Collaborator

Related to #7645.

Perhaps eksctl should fail earlier, rather than spin up a cluster that does not actually do what is being requested?

@dan-tw - I agree with you on this. I shall work on a fix for both issues.

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

Successfully merging a pull request may close this issue.

3 participants