Skip to content

Commit

Permalink
Read TaskPriorityClass from config
Browse files Browse the repository at this point in the history
Signed-off-by: Victor Chang <[email protected]>
  • Loading branch information
mocsharp committed Sep 6, 2023
1 parent 0aca28d commit 556f373
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
3 changes: 3 additions & 0 deletions src/Common/Configuration/TaskManagerConfiguration.cs
Original file line number Diff line number Diff line change
Expand Up @@ -74,5 +74,8 @@ public class ArgoPluginArguments

[ConfigurationKeyName("messageSenderContainerMemoryLimit")]
public string MessageSenderContainerMemoryLimit { get; set; } = "500Mi";

[ConfigurationKeyName("taskPriorityClass")]
public string TaskPriorityClass { get; set; } = string.Empty;
}
}
2 changes: 1 addition & 1 deletion src/TaskManager/Plug-ins/Argo/ArgoPlugin.cs
Original file line number Diff line number Diff line change
Expand Up @@ -432,7 +432,7 @@ private async Task<Workflow> BuildWorkflowWrapper(CancellationToken cancellation
private void ProcessTaskPluginArguments(Workflow workflow)
{
Guard.Against.Null(workflow, nameof(workflow));
var priorityClassName = Event.GetTaskPluginArgumentsParameter(ArgoParameters.TaskPriorityClassName) ?? "standard";
var priorityClassName = Event.GetTaskPluginArgumentsParameter(ArgoParameters.TaskPriorityClassName) ?? _options.Value.TaskManager.ArgoPluginArguments.TaskPriorityClass;

foreach (var template in workflow.Spec.Templates)
{
Expand Down
3 changes: 2 additions & 1 deletion src/TaskManager/TaskManager/appsettings.json
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,8 @@
"messageGeneratorContainerCpuLimit": "1",
"messageGeneratorContainerMemoryLimit": "500Mi",
"messageSenderContainerCpuLimit": "1",
"messageSenderContainerMemoryLimit": "500Mi"
"messageSenderContainerMemoryLimit": "500Mi",
"taskPriorityClass": ""
},
"argoExitHookSendMessageContainerImage": "ghcr.io/project-monai/monai-deploy-task-manager-callback:0.2.0-beta.211"
},
Expand Down

0 comments on commit 556f373

Please sign in to comment.