Masakari 团队流程

Masakari 始终在改进其流程，以确保我们社区所有成员之间进行高效的沟通。

OpenStack 广泛模式

Masakari 遵循 OpenStack 项目普遍采用的大多数规范。您可以在这里获取更多详细信息

如果您是 Masakari 的新手，请首先阅读此内容：如何（更深入地）参与 Masakari。

如何让我的代码合并？

好的，所以您是 Masakari 的新手，并且您被分配了一个需要实现的功能。我该如何实现？

您可以在这里找到大部分问题的答案

但是让我们为 Masakari 增加一些特定的内容…

概述

Flow chart showing the Masakari bug/feature process

你们在哪里跟踪错误？

我们在这里跟踪错误

如果您修复了一个问题，请提交一个错误报告，以便其他人发现该问题时可以找到您为他们提供的修复方案。

另外，在提交补丁之前，值得检查一下是否有人已经为您修复了它（Launchpad 在您创建错误报告时会提供一些帮助）。

我什么时候需要蓝图，什么时候需要规范？

要理解这个问题，我们需要理解为什么蓝图和规范有用。

但这是大致的想法

  • 如果需要规范，那么就需要蓝图。

  • 如果涉及 API 更改，则需要规范。

  • 如果只是一个小的补丁，只涉及少量代码，并且对部署者和文档的影响有限，那么可能不需要规范。

如果您不确定，请咨询 PTL (masakari-ptl) 或其他 masakari-core 成员，通过 IRC 询问。

如何让我的蓝图获得批准？

所以您需要您的蓝图获得批准？以下是如何操作

  • 如果您不需要规范，请将您的蓝图链接添加到下一次 masakari 会议的议程中：https://wiki.openstack.org/wiki/Meetings/Masakari

    • 确保您的蓝图描述包含足够的上下文，以便在会议上进行审查。

  • 如果您需要规范，请提交 masakari-spec 以供审查。

还有其他问题吗？联系 PTL (masakari-ptl) 或其他在您醒着时也在的 masakari-core 成员。IRC 是最佳选择，因为您通常会立即得到回复。如果他们太忙，请给他们发送电子邮件。

如何移除我的补丁上的程序性 -2？

当功能冻结来临时，任何仍在审查中的蓝图补丁都会收到程序性 -2，以阻止它们合并。在 Masakari 中，蓝图仅在一个发布版本中获得批准。要移除 -2，您需要让蓝图获得当前版本的批准（请参阅如何让我的蓝图获得批准？）。

我的代码审查似乎卡住了，我该怎么办？

首先，解决所有 -1 和 -2！以下是一些技巧

  • 精确。确保您没有各说各话。

  • 尝试理解审查者的想法。他们可能具有与您不同的视角和/或用例。

  • 如果您不理解问题，请让他们解释 - 这是一种常见且有帮助的行为。

  • 保持积极。每个人的补丁都有问题，包括核心审查者。一旦问题解决，没有人会在意。

  • 尽量不要反复无常。当两位审查者将您拉向不同的方向时，停止推送代码并协商最佳前进方式。

  • 如果审查者没有回复补丁集上留下的回复，请通过 IRC 或电子邮件联系他们。如果他们仍然没有回复，您可以尝试询问他们的同事是否在休假（或者只是等待）。最后，您可以在 Masakari 会议上要求调解，将其添加到议程中（https://wiki.openstack.org/wiki/Meetings/Masakari）。如果您无法协商解决问题，也应该这样做。

最终，您应该从正在处理审查队列的人那里获得一些 +1。预计也会收到 -1。您可以在公司内部要求审查，1-2 个就足够了（不要更多），特别是如果这些审查者以提供良好的审查而闻名。在等待期间，您可以花一些时间审查其他人的代码 - 他们可能会以同等的回报，并且您可能会学到一些东西（为什么要在我不属于核心团队时进行代码审查？）。

如果您等待了足够的时间并且没有收到任何 +1，您可以在 IRC 上要求进行审查。请不要立即要求核心审查，尤其是不要直接要求（IRC 或电子邮件）。核心审查者的时间非常宝贵，获得一些 +1 是表明您的补丁符合基本质量标准的好方法。

一旦您获得了一些 +1，请耐心等待。记住平均等待时间。您可以每周在 IRC 上要求审查，在核心成员醒着时要求审查会有所帮助。

漏洞

应用正确的跟踪信息会有所帮助。

  • 在提交消息标签中根据需要放置“Closes-Bug”、“Partial-Bug”或“Related-Bug”。

  • 如果您首先需要在 Launchpad 中提交错误报告，请这样做 - 这有助于其他人找到您的修复方案。

  • 确保错误报告具有正确的优先级和标签设置。

特性

再次说明，应用正确的跟踪信息会有所帮助。对于仅限蓝图的功能

  • 在提交消息中放置您的蓝图，例如“blueprint simple-feature”。

  • 如果您完成，请将蓝图标记为 NeedsCodeReview。

  • 维护蓝图上的白板，以便于理解哪些补丁需要审查。

  • 为所有相关补丁使用单个主题。一个蓝图的所有补丁应共享一个主题。

对于蓝图和规范功能，请执行仅限蓝图功能的所有操作，并

如果它不是优先级，您的蓝图/规范已获得该周期的批准，并且您已经耐心等待，您可以在 Masakari 会议期间提出它。结果可能是您的规范未获得该周期的批准，以便优先级项目可以集中精力。如果发生这种情况，很抱歉 - 它不应该一开始就获得批准，Masakari 团队咬得太多了，这是他们的错误而不是您的错误。您可以重新在下一个周期中提出它。

如果它不是优先级，并且您的规范尚未获得批准，您的代码将不会在此周期内合并。请在下一个周期重新提出您的规范。

发布说明

发布说明在单独的页面上介绍：发布说明