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

Canary rollout: analysis run from previous step remains running after promotion #3220

Closed
2 tasks
oleksandr-codefresh opened this issue Dec 6, 2023 · 0 comments · Fixed by #3221
Closed
2 tasks
Labels
bug Something isn't working

Comments

@oleksandr-codefresh
Copy link
Contributor

oleksandr-codefresh commented Dec 6, 2023

Checklist:

  • I've included steps to reproduce the bug.
  • I've included the version of argo rollouts.

Describe the bug

If canary rollout has 2 or more analysis steps one-by-one and you perform promotion on n-1 step with analysis. Analysis run of previous step will remain running while appropriate to new step won't be created.

To Reproduce

  1. create canary rollout with 3 steps and 2 analysis steps one-by-one
- setWeight: 50
- analysis:
     templates:
     - templateName: test-template-a
- analysis:
     templates:
     - templateName: test-template-b     
  1. try to call promote on test-template-a analysis step

Expected behavior

Old analysis run should be terminated and new one (for current step) should be created.

Screenshots
-

Version

tested on 1.5.0 and 1.6.3


Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritize the issues with the most 👍.

@oleksandr-codefresh oleksandr-codefresh added the bug Something isn't working label Dec 6, 2023
zachaller pushed a commit that referenced this issue Dec 11, 2023
… promote action being called. Fixes #3220 (#3221)

* fix: canary step analysis run wasn't terminated as keep running after promote action being called

Signed-off-by: oleksandr-codefresh <[email protected]>

* fix unit test: added test case for canary step analysis run wasn't terminated as keep running after promote action being called

Signed-off-by: oleksandr-codefresh <[email protected]>

---------

Signed-off-by: oleksandr-codefresh <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>
oleksandr-codefresh added a commit to codefresh-io/argo-rollouts that referenced this issue Jan 18, 2024
… promote action being called. Fixes argoproj#3220 (argoproj#3221)

* fix: canary step analysis run wasn't terminated as keep running after promote action being called

Signed-off-by: oleksandr-codefresh <[email protected]>

* fix unit test: added test case for canary step analysis run wasn't terminated as keep running after promote action being called

Signed-off-by: oleksandr-codefresh <[email protected]>

---------

Signed-off-by: oleksandr-codefresh <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>
(cherry picked from commit 0300af1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
1 participant