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

Background Analysis runs when rolling back to a revision within defined Rollback Window #3669

Closed
2 tasks done
Psukhe opened this issue Jun 25, 2024 · 0 comments
Closed
2 tasks done
Labels
bug Something isn't working

Comments

@Psukhe
Copy link
Contributor

Psukhe commented Jun 25, 2024

Checklist:

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

Describe the bug

Background Analysis runs when rolling back to a revision within defined Rollback Window

To Reproduce

Files to reproduce are present here: https://github.com/Psukhe/background-analysis-runs-when-within-rollback-window/tree/main

  • kubectl apply -f analysis-template.yml
  • kubectl apply -f rollout-with-background-analysis-and-rollback-window-step-1.yml
  • kubectl apply -f rollout-with-background-analysis-and-rollback-window-step-2.yml
  • kubectl argo rollouts undo rollout-with-background-analysis-and-rollback-window

Expected behavior

I would expect no Analysis Runs when rolling back within a defined Rollback Window

Version

v1.7.1+6a99ea9

Logs

time="2024-06-25T20:52:29Z" level=info msg="Rollback within the window: 0 (3)" namespace=default rollout=rollout-with-background-analysis-and-rollback-window
...
time="2024-06-25T20:52:29Z" level=info msg="Background Analysis Run 'rollout-with-background-analysis-and-rollback-window-cd6c94ffd-3' Status New: 'Running' Previous: ''" event_reason=AnalysisRunRunning namespace=default rollout=rollout-with-background-analysis-and-rollback-window

All logs are present here: https://github.com/Psukhe/background-analysis-runs-when-within-rollback-window/tree/main


Message from the maintainers:

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

@Psukhe Psukhe added the bug Something isn't working label Jun 25, 2024
zachaller pushed a commit that referenced this issue Aug 13, 2024
…ysis Runs. Fixes #3669 (#3670)

* Ensure that BackgroundAnalysisRun does not run when rolling back within RollbackWindow

Signed-off-by: Alex Dunn <[email protected]>

* Fix linting

Signed-off-by: Alex Dunn <[email protected]>

---------

Signed-off-by: Alex Dunn <[email protected]>
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
Development

No branches or pull requests

1 participant