-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Description
unity/pr-head started failing due to a concurrent merge of both #16612 and #16563. This issue was fixed in #16649, however, PR's that haven't been rebased on the latest main are still failing.
Other CI runs e.g. cpu/pr-head, gpu/pr-head, etc automatically rebase the PR before a CI run. This ensures the PR's are tested against the latest main, rather than a previous head of main. unity/pr-head doesn't seem to do this which can result in PR's failing even after triggering a rerun of CI:
- https://discuss.tvm.apache.org/t/is-there-an-anomaly-in-ci-unity-pr-head/16446/3
- [TOPI] improve inclusive_scan for thrust #16652 (comment)
The behaviour of unity/pr-head should be consistent with cpu/pr-head, gpu/pr-head, etc. Taking a look at the jenkinsfiles, it seems unity/pr-head is missing a call to merge_with_main()(link) when a PR branch is detected.
Originally posted here, creating an issue for better visibility.