django-q
django-q copied to clipboard
"Retry and timeout are misconfigured" as per default settings
On a fresh installation without altered configuration, the new warning pops up:
/usr/lib/python3.9/site-packages/django_q/conf.py:136: UserWarning: Retry and timeout are misconfigured. Set retry larger than timeout,
failure to do so will cause the tasks to be retriggered before completion.
See https://django-q.readthedocs.io/en/latest/configure.html#retry for details.
warn("""Retry and timeout are misconfigured. Set retry larger than timeout,
Since I did not change any configuration, I looked up the default conf.py in this repo to find that timeout is set to none while retry is set to 60s. Should the the timeout in the default conf.py be set to an appropriate value so that it does not trigger the new warning after an unaltered default installation?
Yes, and the docs need to be updated too. It happened because of a recent PR. Either that or we don't show the warning on no timeout configured.
Awesome! I can create the pull request, but I don't know what timeout value would be sane. 30? 59?
I just had this issue on a fresh install with version 1.3.9 which is surprising since it looks like this PR was merged back in May...
It was not merged, only the modification in the documentation. I suggested the corresponding code modification in a separate pull request, but my code edits seem to create trouble elsewhere. Unfortunately, I do not know any python and cannot help any further.
#537
Ah ok, thanks for confirming!
Is this warning safe to ignore if there is no timeout configured?
The same issue comes when we set the timeout value to None.
How can we set timeout value to None and not get this warning? Tried keeping retry value as None, but same warning persist.