Skip to content

feat(chart): supporting nodeSelector and tolerations#149

Open
caiofralmeida wants to merge 1 commit into
google:mainfrom
caiofralmeida:node-selec-tolerations-chart
Open

feat(chart): supporting nodeSelector and tolerations#149
caiofralmeida wants to merge 1 commit into
google:mainfrom
caiofralmeida:node-selec-tolerations-chart

Conversation

@caiofralmeida
Copy link
Copy Markdown

This pull request adds support for customizing node scheduling for the controllerManager deployment in the kube-startup-cpu-boost Helm chart. Users can now specify nodeSelector and tolerations in the values file to control which nodes the controller pods are scheduled on and which taints they tolerate.

Helm chart enhancements for pod scheduling:

  • Added support for nodeSelector and tolerations in the controllerManager section of values.yaml, allowing users to specify node selection and taint toleration preferences.
  • Updated the deployment.yaml template to include the nodeSelector and tolerations fields when provided, enabling the new configuration options in the rendered deployment manifest.

@google-cla
Copy link
Copy Markdown

google-cla Bot commented Apr 24, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@mikouaj
Copy link
Copy Markdown
Member

mikouaj commented Apr 27, 2026

@caiofralmeida I'm happy to accept the PR but you need to sign Google's Contributor License Agreement (https://cla.developers.google.com) - that is required for any google owned repo, sorry for inconvience.

@caiofralmeida
Copy link
Copy Markdown
Author

caiofralmeida commented Apr 27, 2026

Hi @mikouaj, I've already accepted/signed. Can you run the verification again?

@caiofralmeida
Copy link
Copy Markdown
Author

@mikouaj the PR checks have passed successfully :)

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.

2 participants