-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Strange metric values when using counter defined in templateDefaults
#12821
Comments
templateDefaults
Root cause: forgot to merge argo-workflows/workflow/controller/dag.go Lines 433 to 440 in e547b2d
By the way, if you use |
Signed-off-by: oninowang <[email protected]>
Thank you very much for a quick response and the fix! |
Signed-off-by: oninowang <[email protected]>
Signed-off-by: oninowang <[email protected]>
Signed-off-by: oninowang <[email protected]> Co-authored-by: jswxstw <[email protected]>
Signed-off-by: oninowang <[email protected]> Co-authored-by: jswxstw <[email protected]> (cherry picked from commit 465c7b6)
Pre-requisites
:latest
What happened/what did you expect to happen?
I want to implement a simple counter of the workflows steps for every workflow. In order to do that I'm using
templateDefaults
in theworkflowDefaults
parameter like this:I'm testing this on this example workflow https://github.com/argoproj/argo-workflows/blob/main/examples/workflow-template/dag.yaml
Surprisingly I got wrong counter value in the end of the workflow execution: 2, however in my mind the result should by the amount of pods used in this workflow: 8
Version
v3.5.5
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
Logs from the workflow controller
Logs from in your workflow's wait container
The text was updated successfully, but these errors were encountered: