smarkets / marge-bot

A merge-bot for GitLab
BSD 3-Clause "New" or "Revised" License
693 stars 136 forks source link

Add a retry mechanism on all API calls #349

Closed z8v closed 1 year ago

z8v commented 1 year ago

The PR simply adds the retry package and wraps all the API calls with a retry mechanism.

xtermi2 commented 1 year ago

Is here any progress? We need this retry mechanism because he have a lot of failures which could be solved by retries 🙏

z8v commented 1 year ago

@xtermi2 The repo seems abandoned. I think the best way forward would be to fork and apply any patches you need.

xtermi2 commented 1 year ago

Thanks, I've also noticed https://gitlab.com/marge-org/marge-bot which should be the future of marge bot

nejch commented 1 year ago

Hey @z8v @xtermi2! Yes, we're trying to revive this project on GitLab.com. Feel free to follow/star that project :)

This PR was imported during the community fork migration as https://gitlab.com/marge-org/marge-bot/-/merge_requests/349.

However, see also https://gitlab.com/marge-org/marge-bot/-/issues/375 for what I personally think might be a more maintainable approach that fixes other issues. If you disagree just let us know in the fork MR/issues!