Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add nodeselector in stack deployment and make nodeselector global con… #140

Merged
merged 3 commits into from
Nov 5, 2024

Conversation

sepauli
Copy link
Contributor

@sepauli sepauli commented Nov 1, 2024

Description

I have added the nodeSelector in the deployment of the nginx in the stack so that the configuration from the stack/values.yaml works.

I have also modified the configuration for all deployments so that you can set the nodeSelector for all resources at the same time via "global.nodeSelector".
This makes it more easy to control on which nodes all tinkerbell resources should be deployed

How Has This Been Tested?

I have deployed tinkerbell with different nodeSelector settings and looked on which nodes the individual tinkerbell resources are scheduled

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

@jacobweinstock jacobweinstock added the ready-to-merge Mergify: Ready for Merging label Nov 5, 2024
@mergify mergify bot merged commit 1369d73 into tinkerbell:main Nov 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Mergify: Ready for Merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants