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

bank_table: add max-preemptible-after column #553

Open
4 tasks done
cmoussa1 opened this issue Dec 19, 2024 · 0 comments · May be fixed by #556
Open
4 tasks done

bank_table: add max-preemptible-after column #553

cmoussa1 opened this issue Dec 19, 2024 · 0 comments · May be fixed by #556
Assignees
Labels
commands related to the flux-accounting commands/bindings database related to the flux-accounting database new feature new feature

Comments

@cmoussa1
Copy link
Member

cmoussa1 commented Dec 19, 2024

The bank_table does not have a way to track a max preemptible-after value for a certain bank. This kind of information is needed to enforce a limit on association-set preemptible-after values on their jobs as well as set a default on their job in the case where the association does not set one on their own.

Because this might be specific to a bank, its default should be to require no preemption. This would also follow the preemptible-after attribute defined in RFC 14.

The column in the bank_table should be a real number that has no default value (which would indicate that there is no preemption required for this bank and should retroactively support banks already defined), but when defined, represents:

A floating point number greater than or equal to zero representing job run time after which the job MAY be canceled in order to make way for other jobs. A value of zero indicates that the job is immediately preemptible. The attribute SHALL NOT be set if the job is never preemptible.

This value should be able to be specified on the command line when first adding a bank as well as be edited after the bank is added.


Tasks

Preview Give feedback
@cmoussa1 cmoussa1 added the new feature new feature label Dec 19, 2024
@cmoussa1 cmoussa1 self-assigned this Dec 19, 2024
@cmoussa1 cmoussa1 added commands related to the flux-accounting commands/bindings database related to the flux-accounting database labels Dec 19, 2024
@cmoussa1 cmoussa1 linked a pull request Dec 19, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commands related to the flux-accounting commands/bindings database related to the flux-accounting database new feature new feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant