Skip to content

Comments

Add configurable nodeSelectors to Fluid core workloads#117

Open
gdagil wants to merge 1 commit intofluid-cloudnative:masterfrom
gdagil:feature/add-node-selector
Open

Add configurable nodeSelectors to Fluid core workloads#117
gdagil wants to merge 1 commit intofluid-cloudnative:masterfrom
gdagil:feature/add-node-selector

Conversation

@gdagil
Copy link

@gdagil gdagil commented Feb 16, 2026

Summary

  • Add default kubernetes.io/os: linux nodeSelectors for all core Fluid workloads (controllers, dataset, runtime controllers, CSI daemonset, webhook, CRD upgrade job) in values.
  • Wire nodeSelector blocks into every relevant pod template so users can override or extend scheduling constraints.
  • Keep existing tolerations and affinity behavior unchanged.

Changes

  • values.yaml: new nodeSelector defaults for dataset, CSI, webhook, fluidapp, CRD upgrader, and each runtime controller (alluxio, jindo, goosefs, juicefs, thin, efc, vineyard).
  • Templates: inject nodeSelector into controller Deployments, CSI DaemonSet, webhook Deployment, and CRD upgrade Job.

Testing

  • helm template charts/fluid (manual) to confirm each rendered pod includes the nodeSelector stanza.

Notes for reviewers

  • Defaults are minimal and match Kubernetes’ standard label; users can override per component via values.yaml.
  • No behavior change for clusters already scheduling on Linux; adds explicit scheduling hook for constrained clusters.

Checklist

  • Changes limited to Helm values/templates for scheduling.
  • Tested with helm template.
  • Commit includes Signed-off-by.

Branch

feature/add-node-selector

How to override (example)

dataset:
  nodeSelector:
    kubernetes.io/os: linux
    my.custom/zone: az1

Signed-off-by: Gudynin Danila <dggitdev@bk.ru>
@gdagil gdagil force-pushed the feature/add-node-selector branch from 5d5eecc to 42516f9 Compare February 16, 2026 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant