Code review comment for autopkgtest-cloud:retry-download-all-results

Revision history for this message
Brian Murray (brian-murray) wrote :

> 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.

« Back to merge proposal