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.

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

[Feature] Allow additional selector terms to be defined in storage config #446

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[Feature] Allow additional selector terms to be defined in storage config #446

wants to merge 2 commits into from

Commits on Sep 16, 2024

  1. Support any number of node names in affinity. 

    When we process our PVs, We shouldn't care about the amount of nodes.
If none of them exists we can just clear it out and if any one of them
does exist, we should skip it.
    @Omar007
    Omar007 committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    105673e View commit details
    Browse the repository at this point in the history

  2. Allow additional node selector terms to be set 

    This enables additional node selector terms to be added aside from the provisioner node,
enabling use-cases such as shared disks, as affinity can not be changed after provisioning the PV.

Limitation in the current implementation is that the provisioner can /not/ be ANDed with additional terms.
This change only allows for additional terms that will be /ORed/ with the provisioner name term.
    @Omar007
    Omar007 committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    177c340 View commit details
    Browse the repository at this point in the history