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

[18.0.0-proposed]Change weight assigned to podAntiAffinity to 100 #543

Merged

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

(cherry picked from commit c61e750)

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
(cherry picked from commit c61e750)
@stuggi stuggi requested review from olliewalsh and abays July 30, 2024 14:29
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 e526dc2 into openstack-k8s-operators:18.0.0-proposed Jul 30, 2024
2 checks passed
stuggi added a commit to stuggi/keystone-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/infra-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/mariadb-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/openstack-ansibleee-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/openstack-baremetal-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/ovn-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/ovn-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/barbican-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/cinder-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/openstack-baremetal-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/keystone-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/designate-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/heat-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/horizon-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/ironic-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/manila-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/neutron-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/nova-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/octavia-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/placement-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/glance-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/swift-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/telemetry-operator that referenced this pull request Jul 30, 2024
stuggi added a commit to stuggi/test-operator that referenced this pull request Jul 30, 2024
lpiwowar pushed a commit to openstack-k8s-operators/test-operator that referenced this pull request Jul 31, 2024
openshift-merge-bot bot pushed a commit to openstack-k8s-operators/nova-operator that referenced this pull request Jul 31, 2024
openshift-merge-bot bot pushed a commit to openstack-k8s-operators/placement-operator that referenced this pull request Jul 31, 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