Skip to content

Commit

Permalink
eksctl: default to using modern r7i instances over r5
Browse files Browse the repository at this point in the history
  • Loading branch information
consideRatio committed Sep 9, 2024
1 parent 99b0ff8 commit 931c701
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions eksctl/template.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ local nodeAz = "<< cluster_region >>a";
// A `node.kubernetes.io/instance-type label is added, so pods
// can request a particular kind of node with a nodeSelector
local notebookNodes = [
{ instanceType: "r5.xlarge" },
{ instanceType: "r5.4xlarge" },
{ instanceType: "r5.16xlarge" },
{ instanceType: "r7i.xlarge" },
{ instanceType: "r7i.4xlarge" },
{ instanceType: "r7i.16xlarge" },
];
<% if hub_type == "daskhub" %>
local daskNodes = [
Expand All @@ -52,7 +52,7 @@ local daskNodes = [
// A not yet fully established policy is being developed about using a single
// node pool, see https://github.com/2i2c-org/infrastructure/issues/2687.
//
{ instancesDistribution+: { instanceTypes: ["r5.4xlarge"] }},
{ instancesDistribution+: { instanceTypes: ["r7i.4xlarge"] }},
];
<% else %>
local daskNodes = [];
Expand Down

0 comments on commit 931c701

Please sign in to comment.