Browse Source

*: create project governance

Create project governance.
Sahdev P. Zala 6 years ago
parent
commit
036cb96fd1
3 changed files with 9 additions and 18 deletions
  1. 3 0
      CHANGELOG-3.5.md
  2. 5 13
      GOVERNANCE.md
  3. 1 5
      MAINTAINERS

+ 3 - 0
CHANGELOG-3.5.md

@@ -102,6 +102,9 @@ Note that any `etcd_debugging_*` metrics are experimental and subject to change.
 - Require [*Go 1.13+*](https://github.com/etcd-io/etcd/pull/11110).
 - Compile with [*Go 1.13*](https://golang.org/doc/devel/release.html#go1.13)
 
+### Project Governance
+
+- The etcd team has added, a well defined and openly discussed, project [governance](https://github.com/etcd-io/etcd/pull/11175).
 
 <hr>
 

+ 5 - 13
GOVERNANCE.md

@@ -42,7 +42,7 @@ below.
 Contributors who are interested in becoming a maintainer, if performing these
 responsibilities, should discuss their interest with the existing maintainers. New
 maintainers must be nominated by an existing maintainer and must be elected by a
-supermajority of  maintainers. Likewise, maintainers can be removed by a supermajority
+supermajority of maintainers. Likewise, maintainers can be removed by a supermajority
 of the maintainers and moved to emeritus status.
 
 Life priorities, interests, and passions can change. If a maintainer needs to step
@@ -51,22 +51,14 @@ to pick up the related work. At the very least, ensure the related work can be c
 Afterward, create a pull request to remove yourself from the [MAINTAINERS](./MAINTAINERS)
 file.
 
-## Approvers
-
-[Approvers](./MAINTAINERS) are contributors with merge rights. They are active reviewers
-and have created significant numbers of PRs in the various code areas to add new features,
-fix bugs and improve code. New approvers must be nominated by an existing maintainer
-and must be elected by a supermajority of maintainers. Likewise, approvers can be removed
-by a supermajority of the maintainers or can resign by notifying the maintainers.
-
 ## Reviewers
 
 [Reviewers](./MAINTAINERS) are contributors who have demonstrated greater skill in
 reviewing the code contribution from other contributors. Their LGTM counts towards
-merging a code change into the project. New reviewers must be nominated by an existing
-maintainer and must be elected by a supermajority of maintainers. Likewise, reviewers
-can be removed by a supermajority of the  maintainers or can resign by notifying the
-maintainers.
+merging a code change into the project. A reviewer is generally on the ladder towards
+maintainership. New reviewers must be nominated by an existing maintainer and must be
+elected by a supermajority of maintainers. Likewise, reviewers can be removed by a
+supermajority of the  maintainers or can resign by notifying the maintainers.
 
 ## Decision making process
 

+ 1 - 5
MAINTAINERS

@@ -1,4 +1,4 @@
-# The official list of maintainers, approvers, and reviewers for the project maintenance.
+# The official list of maintainers and reviewers for the project maintenance.
 #
 # Refer to the GOVERNANCE.md for description of the roles.
 #
@@ -21,9 +21,5 @@ Xiang Li <xiangli.cs@gmail.com> (@xiang90) pkg:*
 Ben Darnell <ben@cockroachlabs.com> (@bdarnell) pkg:go.etcd.io/etcd/raft
 Tobias Grieger <tobias.schottdorf@gmail.com> (@tbg) pkg:go.etcd.io/etcd/raft
 
-# APPROVERS
-Anthony Romano <romanoanthony061@gmail.com> (@heyitsanthony) pkg:*
-Fanmin Shi <fashi@redhat.com> (@fanminshi) pkg:*
-
 # REVIEWERS
 Wenjia Zhang <wenjiazhang@google.com> (@wenjiaswe) pkg:*