Skip to content

Commit

Permalink
fix(ui): show correct podGC message for deleteDelayDuration. Fixes: a…
Browse files Browse the repository at this point in the history
…rgoproj#12395 (argoproj#12784)

Signed-off-by: shuangkun <tsk2013uestc@163.com>
Signed-off-by: shuangkun tian <72060326+shuangkun@users.noreply.github.com>
Signed-off-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com>
Co-authored-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com>
  • Loading branch information
shuangkun and agilgur5 committed Mar 12, 2024
1 parent c80b2e9 commit 03f9f75
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,8 @@ export function WorkflowLogsViewer({workflow, nodeId, initialPodName, container,
)}
{execSpec(workflow).podGC && (
<>
<WarningIcon /> Your pod GC settings will delete pods and their logs immediately on completion.
<WarningIcon /> Your pod GC settings will delete pods and their logs{' '}
{execSpec(workflow).podGC.deleteDelayDuration ? `after ${execSpec(workflow).podGC.deleteDelayDuration}` : 'immediately'} on completion.
</>
)}{' '}
Logs may not appear for pods that are deleted.{' '}
Expand Down
1 change: 1 addition & 0 deletions ui/src/models/workflows.ts
Original file line number Diff line number Diff line change
Expand Up @@ -825,6 +825,7 @@ export interface WorkflowSpec {
*/
podGC?: {
strategy?: string;
deleteDelayDuration?: string;
};
/**
* SecurityContext holds pod-level security attributes and common container settings.
Expand Down

0 comments on commit 03f9f75

Please sign in to comment.