Skip to content

[helm] scheduling primitives#3154

Open
fresh-borzoni wants to merge 2 commits intoapache:mainfrom
fresh-borzoni:helm-scheduling-support
Open

[helm] scheduling primitives#3154
fresh-borzoni wants to merge 2 commits intoapache:mainfrom
fresh-borzoni:helm-scheduling-support

Conversation

@fresh-borzoni
Copy link
Copy Markdown
Contributor

@fresh-borzoni fresh-borzoni commented Apr 21, 2026

closes #3153

Exposes affinity, nodeSelector, tolerations, and topologySpreadConstraints on both coordinator and tablet. Required to spread replicas across zones/nodes under replication factor 3 - without these, a single node failure can take out all tablet replicas for segments not yet tiered.

@fresh-borzoni
Copy link
Copy Markdown
Contributor Author

@morazow @affo @polyzos PTAL 🙏

Copy link
Copy Markdown
Contributor

@affo affo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Loving where we are going with the Helm charts, thanks @fresh-borzoni for this one, another step towards full flexibility 🚀

I just have a question, why do you mention "With replication factor 3, a single node failure could take out all replicas simultaneously..."? I mean, this would hold with any replication factor if nodes are co-located, isn't it?

Copy link
Copy Markdown
Contributor

@morazow morazow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor nit from my side in the values file 🤝

Thanks, shaping nicely 🚀

Comment thread helm/values.yaml Outdated
@fresh-borzoni
Copy link
Copy Markdown
Contributor Author

I just have a question, why do you mention "With replication factor 3, a single node failure could take out all replicas simultaneously..."? I mean, this would hold with any replication factor if nodes are co-located, isn't it?

Well, it's just me missing "Even" before with, bc we had exactly this scenario and the first reaction was - but how all 3 simultaneously :)

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.

[helm] Add pod scheduling primitives (affinity, nodeSelector, tolerations, topologySpreadConstraints)

3 participants