Skip to content

refactor podAffinity generation #2156

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 3, 2023
Merged

refactor podAffinity generation #2156

merged 1 commit into from
Jan 3, 2023

Conversation

@FxKu
Copy link
Member

@FxKu FxKu commented Jan 3, 2023

Follow up to #2048

  • List new option pod_antiaffinity_preferred_during_scheduling in all example manifests
  • Swap with pod_antiaffinity_topology_key for alphabetical sorting
  • Have one function to generate PodAffinity and PodAntiAffinity
  • Unify unit tests about different pod anti affinity settings

@FxKu FxKu added this to the 1.9 milestone Jan 3, 2023
@jopadi
Copy link
Member

jopadi commented Jan 3, 2023

👍

1 similar comment
@FxKu
Copy link
Member Author

FxKu commented Jan 3, 2023

👍

@FxKu FxKu merged commit 819e410 into master Jan 3, 2023
@FxKu FxKu deleted the anti-affinity-preferred branch January 3, 2023 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sdudoladov sdudoladov Awaiting requested review from sdudoladov sdudoladov is a code owner

@Jan-M Jan-M Awaiting requested review from Jan-M Jan-M is a code owner

@jopadi jopadi Awaiting requested review from jopadi jopadi is a code owner

@idanovinda idanovinda Awaiting requested review from idanovinda idanovinda is a code owner

@hughcapet hughcapet Awaiting requested review from hughcapet hughcapet is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

1.9

Development

Successfully merging this pull request may close these issues.

3 participants

@FxKu @jopadi