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

Change weight assigned to podAntiAffinity to 100 #542

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

stuggi
Copy link
Contributor

@stuggi stuggi commented Jul 30, 2024

The affinity weight ca be between 1 and 100 for each instance of the preferredDuringSchedulingIgnoredDuringExecution affinity type. When the scheduler finds nodes that meet all the other scheduling requirements of the Pod, the scheduler iterates through every preferred rule that the node satisfies and adds the value of the weight for that expression to a sum.

To make the pods better spread bump the weight to the max. Still the anti affinty is "this Pod should not run in an X if that X is already running one or more Pods that meet rule Y", which means if there are less nodes, but enough resources, the pods get scheduled there.

Related: https://issues.redhat.com/browse/OSPRH-8870

The affinity weight ca be between 1 and 100 for each instance of
the preferredDuringSchedulingIgnoredDuringExecution affinity type.
When the scheduler finds nodes that meet all the other scheduling
requirements of the Pod, the scheduler iterates through every
preferred rule that the node satisfies and adds the value of the
weight for that expression to a sum.

To make the pods better spread bump the weight to the max. Still
the anti affinty is "this Pod should not run in an X if that X is
already running one or more Pods that meet rule Y", which means
if there are less nodes, but enough resources, the pods get
scheduled there.

Related: https://issues.redhat.com/browse/OSPRH-8870
@stuggi
Copy link
Contributor Author

stuggi commented Jul 30, 2024

Copy link
Contributor

@abays abays left a comment

Choose a reason for hiding this comment

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

/lgtm

@stuggi stuggi merged commit 7fd3da6 into openstack-k8s-operators:main Jul 30, 2024
2 checks passed
stuggi added a commit to openstack-k8s-operators/placement-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to openstack-k8s-operators/cinder-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to openstack-k8s-operators/horizon-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to openstack-k8s-operators/keystone-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to openstack-k8s-operators/neutron-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to openstack-k8s-operators/octavia-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to openstack-k8s-operators/ovn-operator that referenced this pull request Jul 30, 2024
openshift-merge-bot bot pushed a commit to openstack-k8s-operators/placement-operator that referenced this pull request Jul 30, 2024
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