canmove, Confirmed users
1,394
edits
(→Syntax) |
No edit summary |
||
| Line 7: | Line 7: | ||
* monitor and handle any problems as a result of the commit | * monitor and handle any problems as a result of the commit | ||
The last point is particularly important, since puppet masters automatically update to the latest commit, and start deploying that to hosts. | The last point is particularly important, since puppet masters automatically update to the latest commit, and start deploying that to hosts. | ||
== Exceptions to Review Requirement == | |||
* <tt>a=nodechange</tt> - self-approval if you're only changing node definitions, and only for nodes you're responsible for | |||
= Patch Guidelines / Review Checklist = | = Patch Guidelines / Review Checklist = | ||