Skip to content

Commit

Permalink
Remove redundant future retry [RHELDST-9679]
Browse files Browse the repository at this point in the history
Originally, the fast purge client would retry failed futures. Now with
the HTTP retries it doesn't make sense to have both. The interaction
between the two causes failing purges to run much longer than the
desired 5 minutes.

This change removes the Future retry in favour of the urllib3 retry.
  • Loading branch information
amcmahon-rh committed Feb 28, 2024
1 parent 19a45cc commit cd35d73
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion fastpurge/_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,6 @@ def __executor(self):
sync(name="fastpurge").\
with_poll(self.__poll_purges).\
with_throttle(count=self.MAX_REQUESTS).\
with_retry().\
with_cancel_on_shutdown()
return self.___executor

Expand Down

0 comments on commit cd35d73

Please sign in to comment.