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

Add the kubernetes.io/os selector too #46

Open
aleskandro opened this issue Feb 27, 2024 · 1 comment
Open

Add the kubernetes.io/os selector too #46

aleskandro opened this issue Feb 27, 2024 · 1 comment

Comments

@aleskandro
Copy link
Member

In the future, it might be good for us to include the OS matchExpression too.
it is a good piece of information that is also worth considering for the scheduler predicates.

For example, windows container images shouldn't "risk" landing into a linux node.

The current official recommendation (for OCP) is to leverage runtime classes that instruct the scheduler as a kind of 'side-effect'/additional feature, so it's just a "possible" future thing that we can onboard for free to consider better the platform information in the manifest list images we inspect

xref #44 (comment)

@Prashanth684
Copy link
Contributor

Prashanth684 commented Feb 29, 2024

As we do more additions like this, we should start thinking when we want to expose our features with opt-in flags or options in the CR rather than doing it implicitly under the covers, like something along the lines of:

affinitySelector:
     - architecture
     - osType
     - custom

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

No branches or pull requests

2 participants