Skip to content

Latest commit

 

History

History
56 lines (42 loc) · 3.01 KB

REVIEW.md

File metadata and controls

56 lines (42 loc) · 3.01 KB

HITMC Review Policy

该章节阐述对于服务器操作的 Review 要点

为什么需要 Review?

防止运维进行危险操作,不熟悉某领域,而做出更改,导致需要擦屁股。

运维的权限管理

  1. 简单来讲,你拥有运维权限意味着你可以写入服务器重要文件夹,你需要对你的更改负 责。
  2. 如果你不确定这项更改会不会是不合理操作,你认为这方面你不太熟悉,需要别人看, 则进行 review。
  3. 我们信任你作为运维的直觉,如果你能确保一件事情是正确的,则不需要 review 直接操 作,例如删除一个肯定用不上的文档 (没有功能性修改的)。
  4. 如果你的操作你自己认为可能产生了破坏性操作,那我们建议你先 review。
  5. 如果这个操作太简单,你应该相信自己,而不是等待被 review(会降低效率)。
  6. 如果这个操作比较重大,并且你相信自己不会出问题,那么请写一个 isseus,并且自己贴 上accept label。这样这个 issues 将会作为备忘录,出现问题能等待其他人员进行审核与回滚。

Review 的一般流程

  1. 简单描述某个操作要做什么事情,目的是什么
  2. 可以描述一系列想法以及操作,比如“用 python -m mcdreforged" 再 "cp ../plugins/" 之类
  3. 等待 review 意见。

谁可以给出 Review 意见(同意或反对)

  1. 同意意见,有效的同意意见(approval)可以由任何一个运维给出,但只有负责人的意见 代表最后能够进行操作。非负责人 A 给出的 approval 意见代表 A 看了你的操作,认为 没有问题,但不代表你能进行操作。
  2. 反对意见,任何人都可以给出反对意见,且只需要一个反对意见就可以阻止操作执行。

Approve 后怎么样进行操作

  1. 如果有一个此领域相关的负责人 approve 了你的请求,并且没有其他运维表示反对,则可以进行。
  2. 如果有人反对此操作,你需要修复你的问题(按照反对者的要求)
  3. 在全部通过后,将会有accept label 代表已经通过,可以执行。

相关的负责人列表 (有序)

@inclyc 和 @Alxndrt 作为主力运维负责所有领域的 review 意见。如果你在某一个领域贡 献足够多(有足够分量)也可以申请相关负责人。

领域 负责人
网络,公网,校园网访问 @inclyc
深圳 mod 服,活动服 @Phychias @Robert
目录结构,插件 @Alxndrt @inclyc
服务器硬件 @pmczx @Alxndrt
聊天中间件代码 @inclyc
聊天中间件运行 @Alxndrt @inclyc
服务器全局软件包 @Alxndrt @pmczx @inclyc
系统全局用户,权限管理 @Alxndrt @inclyc
认证服务器(hitmc.cc 网站) @inclyc @Alxndrt