...
Priority
Generally you shouldn’t have to provide a priority for your PR when merging with bors. As an exception to this rule, some PR’s should be merged with the standard priority p=99.
...
The above policy applies broadly and in general across PR’s to cockroach. However, breaking these rules can be legitimately useful at times for solving operational problems, or for fast-tracking very high-priority changes that legitimately need to take precedence over everything else for a reason unrelated to CI stability. If you think your PR merits an exception to the policy, ask in the #dev-inf channel to make sure.
FAQ
Craig (bors) told me "Permission denied"!
If you run bors and get a “Permission denied” response, go to #dev-inf and ask them to:
1. explicitly add you to the access list for cockroach
2. click Update for the Reviewers list in bors
Other FAQs:
How is Bors configured? There's not many configuration options, but what there is can be found in
bors.tomlin the.githubdirectory.When does Bors process the config file? At the time commands are issued, Bors looks for a
bors.tomlin the PR to be merged. This means if you haven't changed it, the config file from your PR's target branch is used.Is there a log of what Bors does somewhere? Yep! For this repo, you can find it here.
How do I add Craig to other repos? Craig is already listening to all the repos under @cockroachdb. There are two steps to making Craig respond to commands somewhere else: part one is to add a
bors.tomlconfig file to the repo that tells Bors what CI status to wait on, and part two is adding everyone as reviewers on the repo's settings page on the Bors dashboard.Craig told me "Permission denied"! You are not currently a reviewer, so just ask someone who is to add you to the list of reviewers on the settings page.Should I use bors for backports? No, use the green merge button instead.
My question isn't answered here! Ask on the mailing list, or if you're an internal Cockroach dev, on Slack in the channel #bors.
...