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

Support use of PriorityClass for Appwrappers in the SDK #182

Closed
MichaelClifford opened this issue Jun 27, 2023 · 1 comment · Fixed by #96
Closed

Support use of PriorityClass for Appwrappers in the SDK #182

MichaelClifford opened this issue Jun 27, 2023 · 1 comment · Fixed by #96
Assignees

Comments

@MichaelClifford
Copy link
Collaborator

One feature of MCAD is that is allows for priority and preemption of jobs. The SDK needs to support this feature by allowing a user to define a priority that updates both the priorityClassName and the spec priority that gets added to the generated appwrapper (see this PR for an example AppWrapper #96).

However, the priorityClassNames are non-standard and will be defined and occasionally edited by the cluster admin, therefore the SDK needs to lookup the available priorityClassNames available on the cluster and expose them to the user so that they can use the correct priority class, and raise an error when they try to use on that's not available.

The SDK also need to handle the case where there are no relevant priority classes on the cluster.

@Maxusmusti
Copy link
Collaborator

Updated in: #337

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

2 participants