diff options
| author | Christian Grothoff <christian@grothoff.org> | 2020-03-19 19:33:20 +0100 |
|---|---|---|
| committer | Christian Grothoff <christian@grothoff.org> | 2020-03-19 19:33:20 +0100 |
| commit | f76e7c46e6609d39d5b28cb489a946833ca58bfb (patch) | |
| tree | f111ba1e0249c45374248e85fb4d7ca096e542b3 /src/testing/testing_api_cmd_auditor_exchanges.c | |
| parent | 5e98c0652a6a03a7ce70405fa53adf706412b24b (diff) | |
| download | exchange-f76e7c46e6609d39d5b28cb489a946833ca58bfb.tar.gz exchange-f76e7c46e6609d39d5b28cb489a946833ca58bfb.tar.bz2 exchange-f76e7c46e6609d39d5b28cb489a946833ca58bfb.zip | |
make do_retry not retry forever in tests/benchmark
Diffstat (limited to 'src/testing/testing_api_cmd_auditor_exchanges.c')
| -rw-r--r-- | src/testing/testing_api_cmd_auditor_exchanges.c | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/src/testing/testing_api_cmd_auditor_exchanges.c b/src/testing/testing_api_cmd_auditor_exchanges.c index c7acaab9a..b5a1cab0e 100644 --- a/src/testing/testing_api_cmd_auditor_exchanges.c +++ b/src/testing/testing_api_cmd_auditor_exchanges.c @@ -31,6 +31,12 @@ /** + * How often do we retry before giving up? + */ +#define NUM_RETRIES 5 + + +/** * State for a "deposit confirmation" CMD. */ struct ExchangesState @@ -72,9 +78,9 @@ struct ExchangesState const char *exchange_url; /** - * Should we retry on (transient) failures? + * How often should we retry on (transient) failures? */ - int do_retry; + unsigned int do_retry; }; @@ -132,8 +138,9 @@ exchanges_cb (void *cls, es->leh = NULL; if (es->expected_response_code != http_status) { - if (GNUNET_YES == es->do_retry) + if (0 != es->do_retry) { + es->do_retry--; if ( (0 == http_status) || (TALER_EC_DB_COMMIT_FAILED_ON_RETRY == ec) || (MHD_HTTP_INTERNAL_SERVER_ERROR == http_status) ) @@ -353,7 +360,7 @@ TALER_TESTING_cmd_exchanges_with_retry (struct TALER_TESTING_Command cmd) GNUNET_assert (&exchanges_run == cmd.run); es = cmd.cls; - es->do_retry = GNUNET_YES; + es->do_retry = NUM_RETRIES; return cmd; } |