Recipes are the build scripts in chrome infra. More info is here.
Your CL will cause downstream recipe expectations to change. Examine the output of the tryjob ; there should be steps like
build tests. The red steps correspond to the repo where, if your change was rolled, expectations would be changed.
If your change needs a manual patch, because you are doing backwards incompatible changes which require non machine creatable patches downstream, Use a flag.
If your change is dangerous, please Use a flag. “Dangerous” means the author, infra people, or reviewers are scared about this breaking bots. This is ultimately up to your discretion, but please be thoughtful.
Otherwise, just use the Autoroller.
The autoroller will automatically land your change. You will have to LGTM the other CLs which it will create (if there are expectation changes), when it rolls your change into downstream repos. It is assumed there will be a small number of these; if there is a large number of them, you should probably Use a flag. If your change has expectation changes and you still want the autoroller to land it, see I just want to commit a change.
This means that you guard your change behind a flag, and disable it by default. It is your responsibility to then roll the change out manually (e.g. go to users of the feature and issue CLs to enable the new functionality). Once everything uses the new functionality:
Write the new code you want to roll out. Example CL
Add a new
Single value to the
recipe_module’s config which controls the behavior
pragma: no coverfor the if statement which triggers the new behavior
Commit the change; it should pass all tryjobs and roll safely to all dependencies with no effects/expectation changes.
Flip users of your recipe module over to the new behavior
Once everything is flipped over, remove the flag and the old behavior.
If you bypass the tryjob, and you break other bots, your change may be reverted by a sheriff/trooper, and you may receive a Post Mortem with your name plastered all over it by the relevant owners of the code, and/or other interested parties. These recipe changes can end up breaking the entire chrome infra bot fleet (~4000 machines), so please be careful. The tryjob is there for a reason.
Recipe-Tryjob-Bypass-Reason: your reason here to the bottom of your CL description (example here), and the tryjob will turn green.