Skip to content

Conversation

@lillie-dae
Copy link
Collaborator

@lillie-dae lillie-dae commented Jul 6, 2022

Changes required for #127

Description

Adds execution stats to the status

"workflowId" : string
"duration" : int (if its -1 means it came back null on argo side)
"resourceDuration": dict<string, long> ( https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/ for further info)
"nodeInfo": dict<string, NodeStatus> key is id of the pod value is NodeStatus(see "io.argoproj.workflow.v1alpha1.NodeStatus" on https://raw.githubusercontent.com/argoproj/argo-workflows/master/api/openapi-spec/swagger.json for further info)
"startedAt": datetimeoffset
"finishedAt": datetimeoffset

Status

DONE

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • All tests passed locally.
  • Documentation comments included/updated.
  • User guide updated.
  • I have updated the changelog
  • Any new files have copyright headers
  • Code coverage above 70%
  • TODO items resolved

@lillie-dae lillie-dae merged commit fb68cc4 into develop Jul 6, 2022
@lillie-dae lillie-dae linked an issue Jul 6, 2022 that may be closed by this pull request
@mocsharp mocsharp deleted the 127-support-execution-stats branch June 11, 2024 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support execution stats

3 participants