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

placement constraints #48

Closed
joel-bluedata opened this issue Oct 3, 2018 · 2 comments
Closed

placement constraints #48

joel-bluedata opened this issue Oct 3, 2018 · 2 comments

Comments

@joel-bluedata
Copy link
Member

Think about how we could/should expose affinity and nodeSelector configuration for the deployed pods.

@joel-bluedata
Copy link
Member Author

Current thinking is that node selector, affinity, and labelling for pods are all things that could need to be enforced in the kdapp and/or requested in the kdcluster.

We currently have support for the "requested in the kdcluster" part of that story for labels. We should probably extend that to node selector and affinity, and also (for all those things) look into expressing them in the kdapp as well.

@kmathur2 kmathur2 assigned kmathur2 and unassigned kmathur2 Apr 2, 2021
@aleshchynskyi
Copy link
Collaborator

As discussed with @kmathur2 the implementation for now will be adding affinity field as part of RoleSpec specified in kdclusters and passing it to pods. Also a good idea is to add affinity to list of pods in kdclusters' status. The affinity is more advanced and therefore top priority for this feature. Node Selector and Node Name fields can be added later as well.

aleshchynskyi pushed a commit to aleshchynskyi/kubedirector that referenced this issue Apr 8, 2021
aleshchynskyi pushed a commit to aleshchynskyi/kubedirector that referenced this issue Apr 8, 2021
aleshchynskyi pushed a commit to aleshchynskyi/kubedirector that referenced this issue Apr 8, 2021
joel-bluedata added a commit that referenced this issue Apr 9, 2021
#48 Allow specifying Affinity for roles
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants