Retry policy keeps iterating if the workflow is paused

[Vitalii Solodilov]

If I try to pause an execution, the retry iteration doesn't stop.

Actual:

* start task1
* 1 retry iteration
* 2 retry iteration
...
* 50 retry iteration
-> pause workflow
* 51 retry iteration
* 52 retry iteration
...
* 500 retry iteration

But it would be cool if a retry iteration was paused after pausing the execution. And it should be continued after resuming the execution.

Expected:

* start task1
* 1 retry iteration
* 2 retry iteration
...
* 50 retry iteration
-> pause workflow
... some time later
-> resume workflow
* 51 retry iteration
* 52 retry iteration
...
* 500 retry iteration

[Matt B (matty-brennan)]

We have also seen this issue with asynchronous tasks and retries. If the execution is cancelled but an async tasks then fails, the retry logic will kick in and start up the task again inside the cancelled execution.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/mistral/+/914514