Summary: Allow patches to be batched by themselves only.
This RFC is to allow users to configure batch size limit by providing an option to allow patches to be batched by themselves only.
Appending "single on"or "single=on" to the bors review command will force a patch to be batched by itself only and will never be put it a batch with other patches nor will other patches join its batch, like so:
bors r+ single on bors r+ single=on
Once a patch is reviewed with "single on" appended, all future reviews on the patch will remember the option and the patch will always be batched alone.
If this behavior is no longer desired, it can be disabled by appending "single off" or "single=off" to the review command and can be re-enable later by appending "single on" to a review command.
bors r+ single off bors r+ single=off
Additionally, the single command may be used without a review command like to turn the behavior on or off without having to review.
bors single=on bors single=off
The following options to the existing "bors r+" command, along with its alias, will be added:
bors r+ single on bors r+ single off bors merge single on bors merge single off bors single=on bors single=off
These commands will enable and disable the functionality to force a patch to always be batched by itself. Because the option is appended to the review command, once the option is handled, the review command will also be handled.
To support the option, the "patches" table will require a column to store the flag which will be checked before batching the patch.
Checking whether a patch should be batched alone will be performed during the handling of the review command and during the bisection of a batch. Batches that are currently running or waiting will be left alone.
There shouldn't be any drawbacks since the option only affects patches which have received the specified command and can be disabled if desired.
- This design is a simple solution to providing the users a method to limit batch size.
- An alternative solution would be allow the user to specify the max number of patches a batch can hold which would affect all patches in a project.
- The original bors system used implemented this solution. Unsure if other CI/CD systems implemented this system.
- Is it necessary for the bisecting logic to honor the "single on" option? If the patch is already in a batch, it would either be merged in without any issues or fail sooner or later. Additionally, since the option is appended to the review command, the patch shouldn't ever be seen by the bisecting logic unless the patched r+'ed again while it's already in a batch.
- Should the option also handle patches that are already in a running/waiting batch? Should it pull out the patch and place it in a separate batch by itself?
Implemented by pull request #839 on GitHub
- Pull request: https://github.com/bors-ng/bors-ng/pull/839