Skip to content

Commit

Permalink
Merge pull request #4751 from consideRatio/pr/step3
Browse files Browse the repository at this point in the history
eksctl: default to using modern r7i instances over r5
  • Loading branch information
consideRatio authored Sep 9, 2024
2 parents 99b0ff8 + 931c701 commit 3784377
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 3784377

Please sign in to comment.