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

flux account-priority-update: add max-preemptible-after to bank information sent over to plugin #554

Open
Tracked by #552
cmoussa1 opened this issue Dec 19, 2024 · 0 comments
Assignees
Labels
commands related to the flux-accounting commands/bindings new feature new feature

Comments

@cmoussa1
Copy link
Member

Once #553 is completed, the bank information that is sent to the priority-plugin via flux account-priority-update should also include the max-preemptible-after information for a bank.

The plugin should be responsible for unpacking this attribute and storing it. Because the plugin does not store general bank information right now, perhaps I could look into adding a new Bank object or something that is composed of just the name of the bank and its max-preemptible-after value (if any). Then, the plugin can keep an internal map of Bank objects and reference this when looking to either set a preemptible-after default or enforce a preemptible-after limit on an association's job.

@cmoussa1 cmoussa1 added the new feature new feature label Dec 19, 2024
@cmoussa1 cmoussa1 self-assigned this Dec 19, 2024
@cmoussa1 cmoussa1 changed the title flux-account-priority-update: add max-preemptible-after to bank information sent over to plugin flux account-priority-update: add max-preemptible-after to bank information sent over to plugin Dec 19, 2024
@cmoussa1 cmoussa1 added the commands related to the flux-accounting commands/bindings label Dec 19, 2024
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 new feature new feature
Projects
None yet
Development

No branches or pull requests

1 participant