From 1321c11ef3267599398fba6128bf89e17bf76aab Mon Sep 17 00:00:00 2001 From: Alexander Kjeldaas Date: Sat, 18 Apr 2020 23:51:29 +0200 Subject: [PATCH] Fix retry logic Make the `RETRY` and `RETRY_INTERVAL` environment variables work. --- app/app.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/app/app.py b/app/app.py index 36c0c5d..bad7f9f 100644 --- a/app/app.py +++ b/app/app.py @@ -84,7 +84,7 @@ def reconfigure(version): # sometimes we can get an error back from DFP, this can happen when DFP is not fully loaded. # resend the request until response status code is 200 (${RETRY} times waiting ${RETRY_INTERVAL} seconds between retries) t = 0 - while t < os.environ.get('RETRY', 10): + while t < int(os.environ.get('RETRY', 10)): t += 1 logger.debug('forwarding request to docker-flow-proxy ({})'.format(t)) @@ -96,9 +96,9 @@ def reconfigure(version): except Exception, e: logger.error('Error while trying to forward request: {}'.format(e)) logger.debug('waiting for retry') - time.sleep(os.environ.get('RETRY_INTERVAL', 5)) + time.sleep(int(os.environ.get('RETRY_INTERVAL', 5))) return "OK" if __name__ == "__main__": - app.run(host='0.0.0.0', port=8080, debug=True, threaded=True) \ No newline at end of file + app.run(host='0.0.0.0', port=8080, debug=True, threaded=True)