> Come to find out the script actually isn't stateful and it will only try to
> download results which it doesn't yet have. I was led astray by the fact that
> some test results are missing 'testinfo.json' and the script will try and
> download those results every time.
Having said that I added the retry part to "list_remote_container" which actually builds the dictionary of known test results. Previously if that failed for any release container then the whole script would crash and no other releases would be updated. So yes the retry belongs in here.
> Come to find out the script actually isn't stateful and it will only try to
> download results which it doesn't yet have. I was led astray by the fact that
> some test results are missing 'testinfo.json' and the script will try and
> download those results every time.
Having said that I added the retry part to "list_remote_ container" which actually builds the dictionary of known test results. Previously if that failed for any release container then the whole script would crash and no other releases would be updated. So yes the retry belongs in here.