-
Notifications
You must be signed in to change notification settings - Fork 147
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
feat: Add missing fields of keypair resource policy #2146
feat: Add missing fields of keypair resource policy #2146
Conversation
Your org has enabled the Graphite merge queue for merging into mainAdd the label “flow:merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “flow:hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @agatha197 and the rest of your teammates on Graphite |
max_pending_session_count
and max_pending_session_resource_slots
fields to keypair_resource_policy cli6203cb2
to
5b7729a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works well!
could you add those missing options to CLI keypair-resource-policy
command?
5b7729a
to
91b9288
Compare
91b9288
to
147ba34
Compare
…rce_slots` fields to keypair_resource_policy
606c7f7
to
0e43a26
Compare
0e43a26
to
09ba91a
Compare
I added more missing options such as |
Co-authored-by: Kyujin Cho <kyujin.cho@lablup.com> Backported-from: main (24.09) Backported-to: 24.03 Backport-of: 2146
Backport to 24.09 is failed. Please backport manually. |
follows #2013
TL;DR
Added missing fields of
keypair_resource_policy
such asmax_pending_session_count
,max_pending_session_resource_slots
,max_session_lifetime
, andmax_concurrent_sftp_sessions
.What changed?
max_pending_session_count
andmax_pending_session_resource_slots
to the keypair resource policy CLI commands and functions.max_concurrent_sftp_sessions
is added toKeypairResourcePolicy
that is missed in feat: concurrency limit for system sessions #1299.How to test?
Ensure that the new options can be set and updated successfully in the keypair resource policy.
Why make this change?
The change introduces new flexibility by allowing users to set maximum pending sessions and resource slots for pending sessions in the keypair resource policy.
Checklist: (if applicable)
ai.backend.test
docs
directory