Jump to: navigation, search

Difference between revisions of "Nova/CoreTeam"

(Membership Expectations)
(Adding or Removing Members)
(19 intermediate revisions by 5 users not shown)
Line 1: Line 1:
 +
[[Category:Compute]]
 
= nova-core =
 
= nova-core =
  
The [https://launchpad.net/~nova-core nova-core] team is responsible for reviewing all changes proposed to the following repositories:
+
The [https://review.openstack.org/#/admin/groups/25,members nova-core] team is responsible for reviewing all changes proposed to the projects under nova in governance:
  
* openstack/nova
+
https://governance.openstack.org/tc/reference/projects/nova.html
* openstack/python-novaclient
+
 
* openstack-doc/compute-api
+
Note that everyone is encouraged to help review changes, even if you are not a member of this team.  All reviews are very useful and are taken into account by the core team members.  Participating in the review process is most critical task on the road to joining the team.
  
 
= Adding or Removing Members =
 
= Adding or Removing Members =
 +
 +
While not strictly required, core team nominations are generally discussed within the nova-core team privately to let members of the core team provide input to other members of the core team before any nominations are proposed publicly in the mailing list. This is done to avoid any awkward -1s in the public nomination if they can be handled privately first.
  
 
A new member may be proposed on the openstack-dev list at any time.  A proposal can come from anyone, but typically comes from the project's PTL or an existing member of the core team.  Once a proposal has been made, five existing members of the core team must respond with a +1.  If any existing member of the team objects, they may respond to the proposal with a -1 to veto the nomination.
 
A new member may be proposed on the openstack-dev list at any time.  A proposal can come from anyone, but typically comes from the project's PTL or an existing member of the core team.  Once a proposal has been made, five existing members of the core team must respond with a +1.  If any existing member of the team objects, they may respond to the proposal with a -1 to veto the nomination.
Line 15: Line 18:
 
= Membership Expectations =
 
= Membership Expectations =
  
Membership in the nova-core team is a significant commitment and should not be taken lightly.  Team members are expected to participate in code reviews on a regular (near daily) basis. Members are also expected to stay on top of discussions happening within the project, primarily on the openstack-dev mailing list.  These activities are critical to be able to provide high quality reviews based on the current state of the project that are consistent with the reviews being done by others on the team.
+
Membership in the nova-core team is a significant commitment and should not be taken lightly.  Maintaining membership on this team takes a lot of time.  Further, it is important that the time invested is ''consistent''.  It is harmful to the team and the project overall for the core team members' participation to be inconsistent.
 +
 
 +
Team members are expected to participate in code reviews on a regular (near daily) basis. Members are also expected to stay on top of discussions happening within the project, primarily on the openstack-dev mailing list.  Note that IRC is also used quite heavily, so members should be in <tt>#openstack-nova</tt> as much as they can.  These activities are critical to be able to provide high quality reviews based on the current state of the project that are consistent with the reviews being done by others on the team and consistent with the [http://docs.openstack.org/infra/manual/developers.html#peer-review documented review guidelines].
 +
 
 +
One metric used to determine the level of participation in reviews is just the number of reviews being done.  While there is no hard line for an expectation on the number of reviews you are doing, members are generally expected to be in the same ballpark as the majority of the rest of the team.  You can find stats on that here:
 +
 
 +
* [http://stackalytics.com/report/contribution/nova/30 Last 30 Days]
 +
* [http://stackalytics.com/report/contribution/nova/90 Last 90 days]
 +
* [http://stackalytics.com/report/contribution/nova/180 Last 180 Days]
 +
 
 +
The number of reviews is certainly not the only important thing.  It is also important that reviews are high quality, such that you gain respect from the other core team members over time.  This is done by regularly providing high quality constructive criticism.  Your well thought out recommendations for changes are what build credibility for your +1 of a patch.
 +
 
 +
All of these things are important, but the most important thing of all is to gain trust from the majority of the -core team.  There's not a well defined list of things you must do to gain this trust, but all of the expectations described here are a starting point.
 +
 
 +
= Review Expectations =
 +
 
 +
Not all reviews are created equal.  Team members should take care in prioritizing where they invest their review time.  In general, priority should be based on the priority of the bug or blueprint the patch is associated with.  Beyond that, older reviews should get priority.  Take a look at the [[ReviewWorkflowTips]] page for tips on how to work prioritization into your workflow.
 +
 
 +
Please keep up to date with the current [http://docs.openstack.org/developer/nova/code-review.html code review guidelines].
  
One metric used to determine the level of participation in reviews is just the number of reviews being done.  You can find stats on that here:
+
= Resources =
  
* [http://russellbryant.net/openstack-stats/nova-reviewers-30.txt Last 30 Days]
+
* [http://docs.openstack.org/infra/manual/developers.html#peer-review Criteria To Use When Doing Code Reviews]
* [http://russellbryant.net/openstack-stats/nova-reviewers-90.txt Last 90 days]
+
* [https://docs.openstack.org/nova/latest/contributor/index.html#reviewing Nova specific review guidelines]
* [http://russellbryant.net/openstack-stats/nova-reviewers-180.txt Last 180 Days]
+
* [[ReviewWorkflowTips|Tips on Review Workflow]]

Revision as of 16:09, 31 August 2018

nova-core

The nova-core team is responsible for reviewing all changes proposed to the projects under nova in governance:

https://governance.openstack.org/tc/reference/projects/nova.html

Note that everyone is encouraged to help review changes, even if you are not a member of this team. All reviews are very useful and are taken into account by the core team members. Participating in the review process is most critical task on the road to joining the team.

Adding or Removing Members

While not strictly required, core team nominations are generally discussed within the nova-core team privately to let members of the core team provide input to other members of the core team before any nominations are proposed publicly in the mailing list. This is done to avoid any awkward -1s in the public nomination if they can be handled privately first.

A new member may be proposed on the openstack-dev list at any time. A proposal can come from anyone, but typically comes from the project's PTL or an existing member of the core team. Once a proposal has been made, five existing members of the core team must respond with a +1. If any existing member of the team objects, they may respond to the proposal with a -1 to veto the nomination.

A member of the team may be removed at any time by the PTL. This is typically due to a drop off of involvement by the member such that they are no longer meeting expectations to maintain team membership.

Membership Expectations

Membership in the nova-core team is a significant commitment and should not be taken lightly. Maintaining membership on this team takes a lot of time. Further, it is important that the time invested is consistent. It is harmful to the team and the project overall for the core team members' participation to be inconsistent.

Team members are expected to participate in code reviews on a regular (near daily) basis. Members are also expected to stay on top of discussions happening within the project, primarily on the openstack-dev mailing list. Note that IRC is also used quite heavily, so members should be in #openstack-nova as much as they can. These activities are critical to be able to provide high quality reviews based on the current state of the project that are consistent with the reviews being done by others on the team and consistent with the documented review guidelines.

One metric used to determine the level of participation in reviews is just the number of reviews being done. While there is no hard line for an expectation on the number of reviews you are doing, members are generally expected to be in the same ballpark as the majority of the rest of the team. You can find stats on that here:

The number of reviews is certainly not the only important thing. It is also important that reviews are high quality, such that you gain respect from the other core team members over time. This is done by regularly providing high quality constructive criticism. Your well thought out recommendations for changes are what build credibility for your +1 of a patch.

All of these things are important, but the most important thing of all is to gain trust from the majority of the -core team. There's not a well defined list of things you must do to gain this trust, but all of the expectations described here are a starting point.

Review Expectations

Not all reviews are created equal. Team members should take care in prioritizing where they invest their review time. In general, priority should be based on the priority of the bug or blueprint the patch is associated with. Beyond that, older reviews should get priority. Take a look at the ReviewWorkflowTips page for tips on how to work prioritization into your workflow.

Please keep up to date with the current code review guidelines.

Resources