archinstall icon indicating copy to clipboard operation
archinstall copied to clipboard

Fix 1716 - Retry on mirror list fetching

Open svartkanin opened this issue 1 year ago • 3 comments

Fixes #2716 by adding a retry for http calls and provide more logging information to make debugging easier

svartkanin avatar Oct 17 '24 10:10 svartkanin

On the phone so hard to search, but didn't we already have a except Exception to catch all errors related to mirror tests?

And if my memory servers, we also discussed adding retry would cause delays to mirror ranking? Maybe we need to curate a list of errors like the related one in the issue that are retry worthy?

Torxed avatar Oct 17 '24 12:10 Torxed

Right, this change is in the generic networking function used by the mirror list. Maybe it's better to attempt the retry on the consumer based on certain errors

svartkanin avatar Oct 17 '24 21:10 svartkanin

@Torxed I have moved the retry local into the mirror logic instead and if that fails it should fallback now to the local mirror list

svartkanin avatar Oct 18 '24 10:10 svartkanin