Change upstream location of testdata to UHH Gitlab#1099
Change upstream location of testdata to UHH Gitlab#1099olemke wants to merge 9 commits intoatmtools:mainfrom
Conversation
|
@riclarsson: I decided to not move the download mechanism into cmake. Reason being that the cmake |
|
Ok. But if it's done at configuration, then the build fails after 1 minute instead of 40. I hope the new error output will solve the true issue |
|
We'll see how it goes now and what the actual error is if it happens again. If necessary, we can always change it later. Did it happen to you again? I couldn't reproduce the download failure a single time while working on my branch. :-( |
|
Yeah, happened earlier this week. Or today? The random branch merge failed so I reran it. |
|
Hehe, I just saw the ❌ |
|
Guess I managed to reproduce it now. Windows build failed. And the error is a connection timeout. Unfortunately not clear if the outgoing connection from Github is causing it or the server on the UHH side is temporarily blocking. |
From the debug output it becomes evident that we trigger a rate or connection limit on the upstream server.
|
Superseded by #1100 |
If a download fails, output the actual RequestException error message.
Reduce outer retry attempts from 30 to 2 (each of those already retries up to 5 times inside the
download_filefunction). Past experience has shown if the download fails once, it'll keep failing no matter how many retries are done.