This repository has been archived by the owner on Dec 11, 2024. It is now read-only.
forked from mlcommons/cm4mlops
-
Notifications
You must be signed in to change notification settings - Fork 2
Enable retry on network failures #11
Labels
enhancement
New feature or request
Comments
Hi @arjunsuresh , this would be useful. Should this be kept on by default or should it be controlled through any env variable? I'm wondering if there is a case where user wants to turn it off |
@anandhu-eng I think we should enable it by default and let users an ENV variable to turn it off for any reason. But first we need to list out the places where we need this. Below are some of them. We should probably try it on one, and if it works as expected move to the remaining places.
|
The below failure is seen many times in our github actions. Trying the fix
|
git clone failures
|
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
We often see CM script runs failing due to netwok failures like this. It'll be good to add a retry mechanism for such failures to improve the user experience and reduce the failures of automatic runs.
The text was updated successfully, but these errors were encountered: