Skip to content

Commit

Permalink
Merge pull request #5160 from GeorgianaElena/dask-hub-split
Browse files Browse the repository at this point in the history
 jmte: put dask workers into their own nodepool per hub and tag them
  • Loading branch information
GeorgianaElena authored Nov 20, 2024
2 parents e7a9b25 + 57bbf6e commit 485cb25
Show file tree
Hide file tree
Showing 3 changed files with 35 additions and 1 deletion.
10 changes: 10 additions & 0 deletions config/clusters/jupyter-meets-the-earth/prod.values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,3 +32,13 @@ basehub:
nodeSelector:
2i2c/hub-name: prod
node.kubernetes.io/instance-type: m5.xlarge
dask-gateway:
gateway:
scheduler:
extraPodConfig:
nodeSelector:
2i2c/hub-name: prod
worker:
extraPodConfig:
nodeSelector:
2i2c/hub-name: prod
10 changes: 10 additions & 0 deletions config/clusters/jupyter-meets-the-earth/staging.values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,13 @@ basehub:
2i2c/hub-name: staging
extraEnv:
SCRATCH_BUCKET: s3://jupyter-meets-the-earth-scratch-staging/$(JUPYTERHUB_USER)
dask-gateway:
gateway:
scheduler:
extraPodConfig:
nodeSelector:
2i2c/hub-name: staging
worker:
extraPodConfig:
nodeSelector:
2i2c/hub-name: staging
16 changes: 15 additions & 1 deletion eksctl/jupyter-meets-the-earth.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,18 @@ 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"] }},
{
namePrefix: "dask-staging",
labels+: { "2i2c/hub-name": "staging" },
tags+: { "2i2c:hub-name": "staging" },
instancesDistribution+: { instanceTypes: ["r5.4xlarge"] }
},
{
namePrefix: "dask-prod",
labels+: { "2i2c/hub-name": "prod" },
tags+: { "2i2c:hub-name": "prod" },
instancesDistribution+: { instanceTypes: ["r5.4xlarge"] }
},
];


Expand Down Expand Up @@ -320,6 +331,9 @@ local daskNodes = [
labels+: {
"k8s.dask.org/node-purpose": "worker"
},
tags+: {
"k8s.dask.org/node-purpose": "worker"
},
taints+: {
"k8s.dask.org_dedicated" : "worker:NoSchedule",
"k8s.dask.org/dedicated" : "worker:NoSchedule"
Expand Down

0 comments on commit 485cb25

Please sign in to comment.