-
Notifications
You must be signed in to change notification settings - Fork 153
Individual builds for all rolled up PRs #1372
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
Individual builds for all rolled up PRs #1372
Conversation
The existing rust-timer build $SHA support should work okay for this, I think. We might want to have the automation added here leave a comment about the queued builds with the rollup PR to pushed commit mapping though, since it'll be annoying to get that otherwise - I think that's not yet implemented? (I may have missed it) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One small nit left
Every time bors merges a rollup PR, we will now create rustc artifacts for all of the individual PRs in the roll up.
This is accomplished by creating a merge commit between the master commit at the time the rollup was merged and the head of each rollup PR. Each of these merge commits is then pushed to the try-perf where a try build CI run will be kicked off.
In and of itself, this does not help with roll up perf problems as this only builds the artifacts - it does not perform any performance run. After we have this working, we'll need a way to kick off perf runs for a particular rolled up PR.