[ovs-dev] [PATCH v2] Introduce Emeritus Committer status.

Justin Pettit jpettit at ovn.org
Thu Oct 26 22:13:44 UTC 2017


Acked-by: Justin Pettit <jpettit at ovn.org>

--Justin


> On Oct 26, 2017, at 2:33 PM, Ben Pfaff <blp at ovn.org> wrote:
> 
> From: Russell Bryant <russell at ovn.org>
> 
> This patch introduces an Emeritus status for OVS committers. An
> Emeritus Committer is recognized as having made a significant impact
> to the project and having been a committer in the past.  It is
> intended as an option for those that do not currently have the time or
> interest to fulfill committer responsibilities based on their current
> responsibilities.  While in this status, they are not included in
> voting for governance purposes.
> 
> An emeritus committer may be re-instated as a full committer at any
> time.
> 
> The OVS committers voted approval of this change.
> 
> See documentation contents for full details.
> 
> Suggested-by: Ethan J. Jackson <ejj at eecs.berkeley.edu>
> Signed-off-by: Russell Bryant <russell at ovn.org>
> Signed-off-by: Ben Pfaff <blp at ovn.org>
> ---
> v1->v2:
>  - Deleted the previous requirements for inactivity revocation,
>    which are now supplanted by emeritus status.
>  - Wordsmithing.
>  - Obtained committer approval via voting.
> 
> Documentation/automake.mk                          |  1 +
> Documentation/index.rst                            |  3 +-
> .../internals/committer-emeritus-status.rst        | 63 ++++++++++++++++++++++
> .../internals/committer-grant-revocation.rst       | 63 ++--------------------
> Documentation/internals/index.rst                  |  1 +
> MAINTAINERS.rst                                    | 14 ++++-
> 6 files changed, 84 insertions(+), 61 deletions(-)
> create mode 100644 Documentation/internals/committer-emeritus-status.rst
> 
> diff --git a/Documentation/automake.mk b/Documentation/automake.mk
> index 630fdf197b7c..3be185414928 100644
> --- a/Documentation/automake.mk
> +++ b/Documentation/automake.mk
> @@ -81,6 +81,7 @@ DOC_SOURCE = \
> 	Documentation/internals/index.rst \
> 	Documentation/internals/authors.rst \
> 	Documentation/internals/bugs.rst \
> +	Documentation/internals/committer-emeritus-status.rst \
> 	Documentation/internals/committer-grant-revocation.rst \
> 	Documentation/internals/committer-responsibilities.rst \
> 	Documentation/internals/documentation.rst \
> diff --git a/Documentation/index.rst b/Documentation/index.rst
> index 17b7b7a0e8a0..c737a6f6c238 100644
> --- a/Documentation/index.rst
> +++ b/Documentation/index.rst
> @@ -108,7 +108,8 @@ Learn more about the Open vSwitch project and about how you can contribute:
> 
> - **Maintaining:** :doc:`internals/maintainers` |
>   :doc:`internals/committer-responsibilities` |
> -  :doc:`internals/committer-grant-revocation`
> +  :doc:`internals/committer-grant-revocation` |
> +  :doc:`internals/committer-emeritus-status`
> 
> - **Documentation:** :doc:`internals/contributing/documentation-style` |
>   :doc:`Building Open vSwitch Documentation <intro/install/documentation>` |
> diff --git a/Documentation/internals/committer-emeritus-status.rst b/Documentation/internals/committer-emeritus-status.rst
> new file mode 100644
> index 000000000000..b2589ac6729e
> --- /dev/null
> +++ b/Documentation/internals/committer-emeritus-status.rst
> @@ -0,0 +1,63 @@
> +..
> +      Licensed under the Apache License, Version 2.0 (the "License"); you may
> +      not use this file except in compliance with the License. You may obtain
> +      a copy of the License at
> +
> +          http://www.apache.org/licenses/LICENSE-2.0
> +
> +      Unless required by applicable law or agreed to in writing, software
> +      distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
> +      WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
> +      License for the specific language governing permissions and limitations
> +      under the License.
> +
> +      Convention for heading levels in Open vSwitch documentation:
> +
> +      =======  Heading 0 (reserved for the title in a document)
> +      -------  Heading 1
> +      ~~~~~~~  Heading 2
> +      +++++++  Heading 3
> +      '''''''  Heading 4
> +
> +      Avoid deeper levels because they do not render well.
> +
> +==================================
> +Emeritus Status for OVS Committers
> +==================================
> +
> +OVS committers are nominated and elected based on their impact on the Open
> +vSwitch project.  Over time, as committers' responsibilities change, some may
> +become unable or uninterested in actively participating in project governance.
> +Committer "emeritus" status provides a way for committers to take a leave of
> +absence from OVS governance responsibilities.  The following guidelines clarify
> +the process around the emeritus status for committers:
> +
> +* A committer may choose to transition from active to emeritus, or from
> +  emeritus to active, by sending an email to the committers mailing list.
> +
> +* If a committer hasn't been heard from in 6 months, and does not respond to
> +  reasonable attempts to contact him or her, the other committers can vote as a
> +  majority to transition the committer from active to emeritus.  (If the
> +  committer resurfaces, he or she can transition back to active by sending an
> +  email to the committers mailing list.)
> +
> +* Emeritus committers may stay on the committers mailing list to continue to
> +  follow any discussions there.
> +
> +* Emeritus committers do not nominate or vote in committer elections.  From a
> +  governance perspective, they are equivalent to a non-committer.
> +
> +* Emeritus committers cannot merge patches to the OVS repository.
> +
> +* Emeritus committers will be listed in a separate section in the
> +  MAINTAINERS.rst file to continue to recognize their contributions to the
> +  project.
> +
> +Emeritus status does not replace the procedures for forcibly removing a
> +committer.
> +
> +Note that just because a committer is not able to work on the project on a
> +day-to-day basis, we feel they are still capable of providing input on the
> +direction of the project.  No committer should feel pressured to move
> +themselves to this status.  Again, it's just an option for those that do not
> +currently have the time or interest.
> diff --git a/Documentation/internals/committer-grant-revocation.rst b/Documentation/internals/committer-grant-revocation.rst
> index 48446264f968..c011df4aec0a 100644
> --- a/Documentation/internals/committer-grant-revocation.rst
> +++ b/Documentation/internals/committer-grant-revocation.rst
> @@ -102,47 +102,10 @@ The process to grant commit access to a candidate is simple:
> Revoking Commit Access
> ----------------------
> 
> -There are two situations in which commit access might be revoked.
> -
> -The straightforward situation is a committer who is no longer active in the
> -project and has no plans to become active in the near future. The process in
> -this case is:
> -
> -- Any time after a committer has been inactive for more than 6 months any other
> -  committer to the project may identify that committer as a candidate for
> -  revocation of commit access due to inactivity.
> -
> -- The plans of revocation should be sent in a private email to the candidate.
> -
> -- If the candidate for removal states plans to continue participating no action
> -  is taken and this process terminates.
> -
> -- If the candidate replies they no longer require commit access then commit
> -  access is removed and a notification is sent to the candidate and all
> -  existing committers.
> -
> -- If the candidate can not be reached within 1 week of the first attempting to
> -  contact this process continues.
> -
> -- A message proposing removal of commit access is sent to the candidate and all
> -  other committers.
> -
> -  - If the candidate for removal states plans to continue participating no
> -    action is taken.
> -
> -  - If the candidate replies they no longer require commit access then their
> -    access is removed.
> -
> -  - If the candidate can not be reached within 2 months of the second
> -    attempting to contact them, access is removed.
> -
> -- In any case, where access is removed, this fact is published through an email
> -  to all existing committers (including the candidate for removal).
> -
> -The more difficult situation is a committer who is behaving in a manner that is
> -viewed as detrimental to the future of the project by other committers. This is
> -a delicate situation with the potential for the creation of division within the
> -greater community and should be handled with care. The process in this case is:
> +When a committer behaves in a manner that other committers view as detrimental
> +to the future of the project, it raises a delicate situation with the potential
> +for the creation of division within the greater community.  These situations
> +should be handled with care.  The process in this case is:
> 
> - Discuss the behavior of concern with the individual privately and explain why
>   you believe it is detrimental to the project. Stick to the facts and keep the
> @@ -276,24 +239,6 @@ Invitation to Accepted Committer
>     we'll set up access. We're looking forward continuing to work together to
>     advance the Open vSwitch project.
> 
> -Proposal to Remove Commit Access for Inactivity
> ------------------------------------------------
> -
> -    Committer *[candidate]* has been inactive for *[duration]*. I have
> -    attempted to privately contacted *[him/her]* and *[he/she]* could not be
> -    reached.
> -
> -    Based on this I would like to formally propose removal of commit access.
> -    If a response to this message documenting the reasons to retain commit
> -    access is not received by *[date]* access will be removed.
> -
> -Notification of Commit Removal for Inactivity
> ----------------------------------------------
> -
> -    Committer *[candidate]* has been inactive for *[duration]*. *[He/she]*
> -    *[stated no commit access is required/failed to respond]* to the formal
> -    proposal to remove access on *[date]*. Commit access has now been removed.
> -
> Proposal to Revoke Commit Access for Detrimental Behavior
> ---------------------------------------------------------
> 
> diff --git a/Documentation/internals/index.rst b/Documentation/internals/index.rst
> index cf70f5566a06..50f59a88f4ee 100644
> --- a/Documentation/internals/index.rst
> +++ b/Documentation/internals/index.rst
> @@ -39,6 +39,7 @@ itself and how they might involved.
>    release-process
>    bugs
>    security
> +   committer-emeritus-status
>    committer-responsibilities
>    committer-grant-revocation
>    authors
> diff --git a/MAINTAINERS.rst b/MAINTAINERS.rst
> index bb389da9f6db..a6deb82294e6 100644
> --- a/MAINTAINERS.rst
> +++ b/MAINTAINERS.rst
> @@ -34,7 +34,7 @@ The responsibilities of an Open vSwitch committer are documented
> The process for adding or removing committers is documented
> `here <Documentation/internals/committer-grant-revocation.rst>`__.
> 
> -This is the current list of Open vSwitch committers:
> +This is the current list of active Open vSwitch committers:
> 
> .. list-table:: OVS Maintainers
>    :header-rows: 1
> @@ -75,3 +75,15 @@ This is the current list of Open vSwitch committers:
>      - tgraf at noironetworks.com
>    * - YAMAMOTO Takashi
>      - yamamoto at midokura.com
> +
> +The project also maintains a list of Emeritus Committers (or Maintainers).
> +More information about Emeritus Committers can be found
> +`here <Documentation/internals/committer-emeritus-status.rst>`__.
> +
> +.. list-table:: OVS Emeritus Maintainers
> +   :header-rows: 1
> +
> +   * - Name
> +     - Email
> +   * -
> +     -
> -- 
> 2.10.2
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev



More information about the dev mailing list