No description
Find a file
Florian Klink 3125b79f2f serie.tmpl.html: move missing table html
When refactoring the web interfaces, series was removed.

However, we still want to show an individual Serie, which should
properly be inside it's own table - so let's add this to the template.
2019-12-04 13:33:57 +01:00
.buildkite WIP: build binary and container 2019-11-22 19:51:16 +01:00
frontend add rotatingloghandler which removes older log entries 2019-12-02 12:03:46 +01:00
gerrit gerrit.client: update internal head after triggering a rebase 2019-12-04 11:43:32 +01:00
misc add rotatingloghandler which removes older log entries 2019-12-02 12:03:46 +01:00
public serie.tmpl.html: move missing table html 2019-12-04 13:33:57 +01:00
submitqueue submitqueue.runner: fix typo in comment 2019-12-04 11:43:32 +01:00
.envrc .envrc: enable go modules 2019-11-25 17:48:08 +01:00
.gitignore gitignore /gerrit-queue 2019-11-21 19:00:54 +01:00
go.mod Use Runner 2019-12-02 10:00:32 +01:00
go.sum add rotatingloghandler which removes older log entries 2019-12-02 12:03:46 +01:00
LICENSE Create LICENSE 2019-11-18 17:48:24 +01:00
main.go main.go: log errors if they are returned from runner.Trigger() 2019-12-03 11:07:44 +01:00
README.md README: document current runner logic in detail 2019-12-04 11:43:32 +01:00
shell.nix WIP: build binary and container 2019-11-22 19:51:16 +01:00

gerrit-queue

This daemon automatically rebases and submits changesets from a Gerrit instance, ensuring they still pass CI.

In a usual gerrit setup with a linear master history, different developers await CI feedback on a rebased changeset, then one clicks submit, and effectively makes everybody else rebase again. gerrit-queue is meant to remove these races to master.

Developers can add a specific tag submit_me to all changesets in a series, and if all preconditions on are met ("submittable" in gerrit speech, this usually means passing CI and passing Code Review), gerrit-queue takes care of rebasing and submitting it to master

How it works

Gerrit only knows about Changesets (and some relations to other changesets), but usually developers think in terms of multiple changesets.

Fetching changesets

gerrit-queue fetches all changesets from gerrit, and tries to identify these chains of changesets. We call them Series. All changesets need to have strict parent/child relationships to be detected (so if only half of the stack gets rebased by the Gerrit Web interface, these are considered individual series.

Series are sorted by the number of changesets in them. This ensures longer series are merged faster, and less rebases are triggered. In the future, this might be extended to other metrics.

Submitting changesets

The submitqueue has a Trigger() function, which gets periodically executed.

It can keep a reference to one single serie across multiple runs. This is necessary if it previously rebased one serie to current HEAD and needs to wait some time until CI feedback is there. If it wouldn't keep that state, it would pick another series (with +1 from CI) and trigger a rebase on that one, so depending on CI run times and trigger intervals, if not keepig this information it'd end up rebasing all unrebased changesets on the same HEAD, and then just pick one, instead of waiting for the one to finish.

The Trigger() function first instructs the gerrit client to fetch changesets and assemble series. If there is a wipSerie from a previous run, we check if it can still be found in the newly assembled list of series (it still needs to contain the same number of series. Commit IDs may differ, because the code doesn't reassemble a wipSerie after scheduling a rebase. If the wipSerie could be refreshed, we update the pointer with the newly assembled series. If we couldn't find it, we drop it.

Now, we enter the main for loop. The first half of the loop checks various conditions of the current wipSerie, and if successful, does the submit ("Submit phase"), the second half will pick a suitable new wipSerie, and potentially do a rebase ("Pick phase").

Submit phase

We check if there is an existing wipSerie. If there isn't, we immediately go to the "pick" phase.

The wipSerie still needs to be rebased on HEAD (otherwise, the submit queue advanced outside of gerrit), and should not fail CI (logical merge conflict) - otherwise we discard it, and continue with the picking phase.

If the wipSerie still contains a changeset awaiting CI feedback, we return from the Trigger() function (and go back to sleep).

If the changeset is "submittable" in gerrit speech, and has the necessary submit queue tag set, we submit it.

Pick phase

The pick phase finds a new wipSerie. It'll first try to find one that already is rebased on the current HEAD (so the loop can just continue, and the next submit phase simply submit), and otherwise fall back to a not-yet-rebased serie. Because the rebase mandates waiting for CI, the code returns the Trigger() function, so it'll be called again after waiting some time.

Compile and Run

go generate
GERRIT_PASSWORD=mypassword go run main.go --url https://gerrit.mydomain.com --username myuser --project myproject