Race condition for bulk project creation

[almahmoud]

Race condition for bulk project creation

[edeediong]

@almahmoud I just forked the repository. How do you advise I begin?

[almahmoud]

If you look at the old commit: 75cfd31 could give you an idea of what was added before (essentially a name is created based on a template from the project name, and it fetches the secret with the KubeClient).
In terms of what to do for this specific issue, it's a matter of adding a retry on the try part of the the try/except here:

cloudman/cloudman/projman/api.py

Line 194 in 87b3aaa

try:

. Perhaps retrying 3 times, at 5 second intervals, before returning None, for a first contribution.
You can use the tenacity library, (here is a quick blog about tenacity: https://julien.danjou.info/python-tenacity/ )

Down the line we can make it a bit smarter, like checking for the chart that is being installed, but this would be a good start!

[edeediong]

Alrighty... Let me get that working locally

[edeediong]

Hi @almahmoud, I was able to come up with

retryer = tenacity.Retrying(
            stop=tenacity.stop_after_attempt(3),
            retry=tenacity.retry_if_exception_type(),
            wait=tenacity.wait_fixed(5))

retryer.call()

Question I have is do I need the try/except block after using tenacity?