Allow infinite retries for workflows

[matheusandre1]

Description

I added the requested validation and created the test according to the project's naming convention.

Issue reference

Please reference the issue this PR will close: #870

• Code compiles correctly
• Created/updated tests
• Extended the documentation

[acroca]

Also, I think it'd be useful to add a scenario for the infinite retries to the examples/workflow/simple.py. The example could be just a counter that makes it fail 10 times, and succeed after all those failures, and assert the workflow completed successfully. makes sense?

[matheusandre1]

I adjusted the WorkflowRuntime start time so it doesn't fail when the worker isn't ready within the expected time.

[acroca]

Thanks!
It'd be better if you send different commits instead of keeping updating a single commit, this way I can track what's added every time instead of having to review everything every time.

[acroca]

Hm do you mind explaining this change? I feel it'd be good to raise an an error because the worker is not ready.

[matheusandre1]

It was due to a pipeline error, where the attempts were altered.

[matheusandre1]

I apologize, I thought I had to do a squash commit.

[matheusandre1]

pr a pipeline tb failed again, I don't understand the error.

[sicoyle]

Hi! 👋 can you try running these cmds pls to fix the DCO build step?

In your local branch, run: git rebase HEAD~4 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin issue870

[CasperGN]

@matheusandre1 can you address the comments?

[matheusandre1]

@CasperGN I didn't understand, I corrected what was requested.

[matheusandre1]

@CasperGN , I'm having trouble understanding the CI errors; check examples.

[CasperGN]

@CasperGN , I'm having trouble understanding the CI errors; check examples.

Sorry I wasn't explicit enough. There's a couple of test failures that looks related to the infinite retries. The problem is deeper though:

======================================================================
ERROR: test_allow_infinite_max_number_of_attempts (test_retry_policy.RetryPolicyTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/python-sdk/python-sdk/ext/dapr-ext-workflow/tests/test_retry_policy.py", line 39, in test_allow_infinite_max_number_of_attempts
    retry_policy = RetryPolicy(
  File "/home/runner/work/python-sdk/python-sdk/ext/dapr-ext-workflow/dapr/ext/workflow/retry_policy.py", line 63, in __init__
    self._obj = task.RetryPolicy(
  File "/home/runner/work/python-sdk/python-sdk/.tox/py310/lib/python3.10/site-packages/durabletask/task.py", line 582, in __init__
    raise ValueError("max_number_of_attempts must be >= 1")
ValueError: max_number_of_attempts must be >= 1

If you note the last line, the value error is thrown from durable task so this PR cannot be alone - it needs to follow with a PR in durable task that allow propagating 0 and -1 into the retries.

Would it also be possible, when you get both repo PRs ready, to run it locally with an example of infinite retries?