Skip to content

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

Merged
merged 12 commits into from
Aug 1, 2022

Conversation

rylev
Copy link
Member

@rylev rylev commented Jul 31, 2022

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.

@rylev rylev requested a review from Mark-Simulacrum July 31, 2022 17:50
@Mark-Simulacrum
Copy link
Member

After we have this working, we'll need a way to kick off perf runs for a particular rolled up PR.

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)

@rylev rylev requested a review from Mark-Simulacrum August 1, 2022 08:05
Copy link
Member

@Mark-Simulacrum Mark-Simulacrum left a 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants