Skip to content

Commit

Permalink
Merge pull request #825 from yiannistri/revert-790
Browse files Browse the repository at this point in the history
Revert "Merge pull request #790 from mjura/gpu-v2.9"
  • Loading branch information
yiannistri authored Sep 12, 2024
2 parents bfd4eaa + 54a247a commit 710f4f9
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
5 changes: 3 additions & 2 deletions controller/external.go
Original file line number Diff line number Diff line change
Expand Up @@ -207,8 +207,9 @@ func BuildUpstreamClusterState(ctx context.Context, name, managedTemplateID stri
ngToAdd.Ec2SshKey = ng.Nodegroup.RemoteAccess.Ec2SshKey
}
}

if ng.Nodegroup.AmiType == ekstypes.AMITypesAl2023X8664Nvidia {
// TODO: Update AMITypesAl2X8664Gpu to Amazon Linux 2023 when it is available
// Issue https://github.com/rancher/eks-operator/issues/568
if ng.Nodegroup.AmiType == ekstypes.AMITypesAl2X8664Gpu {
ngToAdd.Gpu = aws.Bool(true)
} else if ng.Nodegroup.AmiType == ekstypes.AMITypesAl2023X8664Standard {
ngToAdd.Gpu = aws.Bool(false)
Expand Down
2 changes: 1 addition & 1 deletion pkg/eks/create.go
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,7 @@ func CreateNodeGroup(ctx context.Context, opts *CreateNodeGroupOptions) (string,
} else if arm := opts.NodeGroup.Arm; aws.ToBool(arm) {
nodeGroupCreateInput.AmiType = ekstypes.AMITypesAl2023Arm64Standard
} else if gpu := opts.NodeGroup.Gpu; aws.ToBool(gpu) {
nodeGroupCreateInput.AmiType = ekstypes.AMITypesAl2023X8664Nvidia
nodeGroupCreateInput.AmiType = ekstypes.AMITypesAl2X8664Gpu
} else {
nodeGroupCreateInput.AmiType = ekstypes.AMITypesAl2023X8664Standard
}
Expand Down
2 changes: 1 addition & 1 deletion pkg/eks/create_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -956,7 +956,7 @@ var _ = Describe("CreateNodeGroup", func() {
InstanceTypes: createNodeGroupOpts.NodeGroup.SpotInstanceTypes,
Subnets: createNodeGroupOpts.NodeGroup.Subnets,
NodeRole: aws.String("test"),
AmiType: ekstypes.AMITypesAl2023X8664Nvidia,
AmiType: ekstypes.AMITypesAl2X8664Gpu,
}).Return(nil, nil)

launchTemplateVersion, generatedNodeRole, err := CreateNodeGroup(ctx, createNodeGroupOpts)
Expand Down

0 comments on commit 710f4f9

Please sign in to comment.