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

Default LocalQueue #2936

Open
3 of 6 tasks
KPostOffice opened this issue Aug 29, 2024 · 5 comments
Open
3 of 6 tasks

Default LocalQueue #2936

KPostOffice opened this issue Aug 29, 2024 · 5 comments
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@KPostOffice
Copy link

What would you like to be added:

I would like to be able to indicate a default LocalQueue per namespace to be used for a workload

Why is this needed:

If an administrator creates a single LocalQueue per namespace that all workloads are expected to use, defaulting the queue let's quota consumers ignore the need to include Queue specific request labels in their workloads and enforces the need to be associated with a Queue in order to be scheduled.

Completion requirements:

  • API changes to LocalQueue spec which allow admin to specify it as default
  • Mutating webhook which will get the default LocalQueue and apply the appropriate label
  • Validating webhook which ensures that only one LocalQueue is marked as default

This enhancement requires the following artifacts:

  • Design doc
  • API change
  • Docs update

The artifacts should be linked in subsequent comments.

@KPostOffice KPostOffice added the kind/feature Categorizes issue or PR as related to a new feature. label Aug 29, 2024
@tenzen-y
Copy link
Member

tenzen-y commented Oct 8, 2024

@KPostOffice Do you assume that automatically https://kueue.sigs.k8s.io/docs/tasks/manage/setup_job_admission_policy/ generation?

@KPostOffice
Copy link
Author

@KPostOffice Do you assume that automatically https://kueue.sigs.k8s.io/docs/tasks/manage/setup_job_admission_policy/ generation?

Are you referring to the example AdmissionPolicy outlined in the documentation?

@tenzen-y
Copy link
Member

https://kueue.sigs.k8s.io/docs/tasks/manage/setup_job_admission_policy/

Yes, I indicated the ValidatingAdmissionPolicy.

@yaroslava-serdiuk
Copy link
Contributor

/assign

@KPostOffice
Copy link
Author

Yeah, the assumption here would be that the policy would be in place

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

3 participants