Puppet/reviews

这些文档已过时，但为了历史原因和搜索访问而保留在此处。要查看最新文档，请参阅 https://docs.openstack.org/developer/puppet-openstack-guide/

评审政策

代码的合并基于 Gerrit 中模块的投票流程。所有提交的补丁都会自动触发一个任务，运行其 rspec-puppet 测试。这个任务被视为一道关卡，任何未通过这些测试的代码都不得合并。该任务的结果会以来自 Jenkins 的 +1 Verified 投票的形式列在每个补丁中。

任何用户都可以对提交 +/- 1 投票并添加评论，但只有 puppet-manager-core 组的成员才有 +2 投票和批准代码合并的权限。

废弃

如果一个变更被提交并收到 -1 投票，随后作者在几周内没有回应，评审人员应在评审中留下提醒评论，或尝试通过 IRC 或电子邮件联系原始作者。如果变更易于修复，任何人都可以随时检出该变更并使用相同的变更 ID 重新提交，以保留原始作者身份。核心评审人员不会放弃这样的变更。

如果一个变更被提交并收到 -2 投票，或者以其他方式明确该变更无法合并（例如，如果选择了另一个变更来解决问题），并且作者至少 3 个月没有回应，核心评审人员应放弃该变更。

随着我们审查处理遗忘补丁的带宽以及监控积压增长情况，本政策可能会发生变化。