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

Ben Pfaff blp at ovn.org
Fri Oct 27 17:37:21 UTC 2017


Thanks Ethan and Justin.  I applied this to master.

On Thu, Oct 26, 2017 at 02:40:48PM -0700, Ethan J. Jackson wrote:
> Acked-by: Ethan J. Jackson <ethan at kelda.io>
> 
> On Thu, 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
> >
> 
> 
> 
> -- 
> Ethan J. Jackson
> ejj.sh


More information about the dev mailing list