This failure might be related to https://github.com/apple/swift-compiler-rt/commit/6f8a608c6b2467140785338c5b7b7bb3b5bd95bf
Thanks,
Mishal Shah
This failure might be related to https://github.com/apple/swift-compiler-rt/commit/6f8a608c6b2467140785338c5b7b7bb3b5bd95bf
Thanks,
Mishal Shah
Yup.
It is normal that I do "@swift-ci please test and merge" and nothing happens?
Kuba
Unfortunately, yes. I usually try several times. Do you want me to force merge?
-Andy
Yup.
It is normal that I do "@swift-ci please test and merge" and nothing happens?
Kuba
Unfortunately, yes. I usually try several times. Do you want me to force merge?
Sigh. Mishal, anything that can be done about that?
I triggered the testing again, this time it actually started. If you feel this needs to be resolved sooner than the testing is done, please force-merge (both on 5.0 and master).
Thanks and sorry for the breakage!
Kuba
If you force push to PR branch, it takes time for GitHub servers to pickup the new sha.
Thanks,
Mishal Shah
I did see the new sha on the Github web page. Then I did "@swift-ci please ...". And then an hour later, I see that testing didn't start :/
Kuba
If you force push to PR branch, it takes time for GitHub servers to pickup the new sha.
Thanks,
Mishal Shah
But there’s neither any feedback nor automatic retry. So every attempt to issue a swift-ci command involves staring at the browser, hitting refresh for a few minutes. This is not conducive to our mental health.
-Andy
Kuba, just trying to understand why we did not see this failure before pushing the original change (https://github.com/apple/swift-compiler-rt/commit/6f8a608c6b2467140785338c5b7b7bb3b5bd95bf), did you run PR testing on swift-compiler-rt?
Thanks,
Mishal Shah
Andy, we do auto retry for all other PR testing except for test and merge.
Thanks,
Mishal Shah
So, I really don’t need to issue 3-4 “@swift-ci test” commands after I push? What’s the interval between retrying?
-Andy
It takes about 2-4mins, because it has to checkout the swift repertory and check the sha before it can re-trigger new build.
Thanks,
Mishal Shah
That's not my observation. See what happened on my PR. It didn't start even after 1 hour.
Kuba
Andy, we do auto retry for all other PR testing except for test and merge.
It does not currently support re-try on test and merge, I said earlier.
Ah, sorry, I missed that, thanks! Plans to fix that?
Kuba
No worries, hopefully soon.
I did not. Does PR testing actually work on swift-compiler-rt?
Kuba