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

Extend the ReclaimablePods feature to other built-in integrations #4237

Open
3 tasks done
mimowo opened this issue Feb 12, 2025 · 3 comments
Open
3 tasks done

Extend the ReclaimablePods feature to other built-in integrations #4237

mimowo opened this issue Feb 12, 2025 · 3 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@mimowo
Copy link
Contributor

mimowo commented Feb 12, 2025

What would you like to be added:

Currently the ReclaimablePods feature is only supported for batch/Jobs.

I would like to support it for all built-in integrations, unless for some this is not possible.

However, the graduation status of the KEP https://github.com/kubernetes-sigs/kueue/blob/main/keps/78-dynamically-reclaiming-resources/README.md is unclear. I would like to revisit the state, and review for which Jobs the support is possible.

Also, solve the related bug #3141 which may or may not require API update.

As we support the other Jobs we may need to protect the support by feature gates, so I would like to refresh the KEP and extend with the feature gates.

Why is this needed:

  • to provide complete functionality to the users
  • to solve the remaining bugs in the functionality
  • to make a clear statement about the graduation of the feature.

Completion requirements:

This enhancement requires the following artifacts:

  • Design doc
  • API change
  • Docs update

The artifacts should be linked in subsequent comments.

@mimowo mimowo added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 12, 2025
@mimowo
Copy link
Contributor Author

mimowo commented Feb 12, 2025

@tenzen-y
Copy link
Member

+1

@dgrove-oss
Copy link
Contributor

It's been on our feature list for AppWrappers for a while (project-codeflare/appwrapper#51). The main hesitation was that only a handful of other integrations had implemented it so it didn't seem worth the effort yet.

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