New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: save Windows artifacts at end of job in case they fail #33510
Conversation
No Release Notes |
I was unable to backport this PR to "18-x-y" cleanly; |
I was unable to backport this PR to "17-x-y" cleanly; |
I was unable to backport this PR to "19-x-y" cleanly; |
/trop run backport |
The backport process for this PR has been manually initiated - here we go! :D |
@VerteDinde has manually backported this PR to "19-x-y", please check out #34313 |
Description of Change
Sometimes Appveyor has trouble uploading artifacts and this causes a build to be marked as failing when really the only thing that failed was saving artifacts to Azure storage. Since these artifacts are not necessary to determine whether a build was successful or not, this PR moves the uploads of the artifacts AFTER the tests instead of before the tests. This means that now the tests will run first and will not be blocked by artifact uploads failing.
Checklist
Release Notes
Notes: none