Jump to: navigation, search

Difference between revisions of "Neutron/ReviewDays"

(147 intermediate revisions by 14 users not shown)
Line 1: Line 1:
 
__NOTOC__
 
__NOTOC__
−
----
+
We are thinking about having an assigned reviewer for each day.
−
As we enter the Grizzly release cycle, we are also back to normal with review days. All quantum core devs are therefore invited to sign up in the list below.
+
 
 +
'''DISCLAIMER: This page is a proposal only at the moment! The idea of review days has not yet been officialy approved!!!'''
  
−
----
 
 
Review duty rotates between [https://launchpad.net/~quantum-core/+members#active every (human) member of quantum-core]. If you are the assigned reviewer, it's your responsibility to: /* start */
 
Review duty rotates between [https://launchpad.net/~quantum-core/+members#active every (human) member of quantum-core]. If you are the assigned reviewer, it's your responsibility to: /* start */
 
* Set blueprint proposals to "Approved" if appropriate, or request more information.
 
* Set blueprint proposals to "Approved" if appropriate, or request more information.
 
* Review patches.
 
* Review patches.
−
* Assign domain experts to patches withouth a review. In this case the domain expert might be or might not be a core dev. In the latter case a core dev should probably be selected as well for the branch. NOTE: Following input for the design summit we should be identifying domains and respective experts for Quantum.
+
* Assign domain experts to patches withouth a review. In this case the domain expert might be or might not be a core dev. In the latter case a core dev should probably be selected as well for the branch.
 
* Respond to queries from contributors concerning pending reviews or blueprint approvals.
 
* Respond to queries from contributors concerning pending reviews or blueprint approvals.
 
* Pester assigned reviewers to review branches.
 
* Pester assigned reviewers to review branches.
−
* Assign new bug report to domain experts, or triage then if you happen to be an expert for the domain the bug report refers to.
 
−
* Answer questions on Launchpad or on the Openstack-dev mailing list if they are pertinent to your area of expertise (Actually core devs are invited to do so not just on their review day!)
 
  
 
/* end */
 
/* end */
  
−
Active reviews for both the server and client/CLI repos:
+
Active reviews for trunk: https://review.openstack.org/#q,status:open+project:openstack/quantum,n,z
−
 
 
−
* https://review.openstack.org/#q,status:open+project:openstack/quantum,n,z
 
−
* https://review.openstack.org/#/q/status:open+project:openstack/python-quantumclient,n,z
 
  
 
Every human member of quantum-core should be part of this rotation. As we have core developers in several we will try to leverage this as much as possible. For this reason, the table below has 3 columns corresponding to 3 different time "areas": Americas, EMEA, and APAC.
 
Every human member of quantum-core should be part of this rotation. As we have core developers in several we will try to leverage this as much as possible. For this reason, the table below has 3 columns corresponding to 3 different time "areas": Americas, EMEA, and APAC.
  
 
Approximately a core reviewer would be available in the following hours: /* start */
 
Approximately a core reviewer would be available in the following hours: /* start */
 +
* Americas: 5PM GMT - 1AM GMT
 +
* EMEA: 9AM GMT - 5PM GMT
 
* APAC: 1AM GMT - 9AM GMT
 
* APAC: 1AM GMT - 9AM GMT
−
* EMEA: 9AM GMT - 5PM GMT
 
−
* Americas: 5PM GMT - 1AM GMT
 
  
 
/* end */
 
/* end */
 +
 +
It would be great if we could fill every column for each day, but having just one reviewer on duty in a given time area 4 days out of 5 would probably be an acceptable result. The idea is to start this scheme on Monday June 18th, and Update this schedule every 2 weeks,  in order to accomodate as much as possible for holydays or other things.
  
 
See [[People]] for mappings between real names and IRC handles.
 
See [[People]] for mappings between real names and IRC handles.
Line 36: Line 33:
 
| EMEA  
 
| EMEA  
 
|-
 
|-
−
| Monday December 3rd
+
| Monday June 18th
−
| 
 
−
| 
 
−
|-
 
−
| Tuesday December 4th
 
−
| Mark [[McClain]]*
 
−
| Gary Kotton
 
−
|-
 
−
| Wednesday December 5th
 
−
| Dan Wendlandt*
 
−
| 
 
−
|-
 
−
| Thursday December 6th
 
 
| Edgar Magana  
 
| Edgar Magana  
 
| Gary Kotton  
 
| Gary Kotton  
 
|-
 
|-
−
| Friday December 7th
+
| Tuesday June 19th
−
| Mark [[McClain]]*
+
| TBA
−
| Salvatore Orlando
+
| TBA
 
|-
 
|-
−
| Saturday December 8th
+
| Wednesday June 20th
−
|
+
| TBA
−
|
+
| TBA
 
|-
 
|-
−
| Sunday December 9th
+
| Thursday June 21st
−
|
+
| TBA
 
| Gary Kotton  
 
| Gary Kotton  
 
|-
 
|-
−
| Monday December 10th
+
| Friday June 22nd
−
| Bob Kukura
+
| Dan Wendlandt
−
|
+
| TBA
 
|-
 
|-
−
| Tuesday December 11th
+
| Saturday June 23rd
−
| Mark [[McClain]]*
 
−
| Gary Kotton
 
−
|-
 
−
| Wednesday December 12th
 
−
|  (dan traveling)
 
 
|   
 
|   
−
|-
 
−
| Thursday December 13th
 
−
| Edgar Magana
 
−
| Gary Kotton
 
−
|-
 
−
| Friday December 14th
 
−
| Mark [[McClain]]* 
 
 
|   
 
|   
 
|-
 
|-
−
| Saturday December 15th
+
| Sunday June 24th
−
|  Dan Wendlandt
 
 
|   
 
|   
−
|-
 
−
| Sunday December 16th
 
 
|   
 
|   
−
| Gary Kotton
 
 
|-
 
|-
−
| Monday December 17th
+
| Monday June 25th
−
| Bob Kukura
 
−
|  Salvatore Orlando
 
−
|-
 
−
| Tuesday December 18th
 
−
| Mark [[McClain]]*
 
−
| Gary Kotton
 
−
|-
 
−
| Wednesday December 19th
 
−
| Dan Wendlandt*
 
−
|  Salvatore Orlando
 
−
|-
 
−
| Thursday December 20th
 
 
| Edgar Magana  
 
| Edgar Magana  
 
| Gary Kotton  
 
| Gary Kotton  
 
|-
 
|-
−
| Friday December 21st
+
| Tuesday June 26th
−
| Mark [[McClain]]*
+
| TBA
−
| Salvatore Orlando
+
| [[SalvatoreOrlando]]  
 
|-
 
|-
−
| Saturday December 22nd
+
| Wednesday June 27th
−
|
+
| TBA
−
|
+
| TBA
 
|-
 
|-
−
| Sunday December 23rd
+
| Thursday June 28th
−
|
+
| TBA
 
| Gary Kotton  
 
| Gary Kotton  
 
|-
 
|-
−
| Monday December 24th
+
| Friday June 29th
−
| Mark [[McClain]]  
+
| Dan Wendlandt
−
| 
+
| [[SalvatoreOrlando]]  
−
|-
 
−
| Tuesday December 25th
 
−
| 
 
−
| Gary Kotton
 
 
|-
 
|-
−
| Wednesday December 26th
+
| Saturday June 30th
−
| Dan Wendlandt*
 
 
|   
 
|   
−
|-
 
−
| Thursday December 27th
 
−
| Mark [[McClain]]*
 
−
| Gary Kotton
 
−
|-
 
−
| Friday December 28th
 
 
|   
 
|   
−
| Salvatore Orlando
 
 
|-
 
|-
−
| Saturday December 29th
+
| Sunday July 1st
 
|   
 
|   
 
|   
 
|   
−
|-
+
|}
−
| Sunday December 30th
 
−
| 
 
−
|
 

Revision as of 17:53, 11 June 2012

We are thinking about having an assigned reviewer for each day.

DISCLAIMER: This page is a proposal only at the moment! The idea of review days has not yet been officialy approved!!!

Review duty rotates between every (human) member of quantum-core. If you are the assigned reviewer, it's your responsibility to: /* start */

  • Set blueprint proposals to "Approved" if appropriate, or request more information.
  • Review patches.
  • Assign domain experts to patches withouth a review. In this case the domain expert might be or might not be a core dev. In the latter case a core dev should probably be selected as well for the branch.
  • Respond to queries from contributors concerning pending reviews or blueprint approvals.
  • Pester assigned reviewers to review branches.

/* end */

Active reviews for trunk: https://review.openstack.org/#q,status:open+project:openstack/quantum,n,z

Every human member of quantum-core should be part of this rotation. As we have core developers in several we will try to leverage this as much as possible. For this reason, the table below has 3 columns corresponding to 3 different time "areas": Americas, EMEA, and APAC.

Approximately a core reviewer would be available in the following hours: /* start */

  • Americas: 5PM GMT - 1AM GMT
  • EMEA: 9AM GMT - 5PM GMT
  • APAC: 1AM GMT - 9AM GMT

/* end */

It would be great if we could fill every column for each day, but having just one reviewer on duty in a given time area 4 days out of 5 would probably be an acceptable result. The idea is to start this scheme on Monday June 18th, and Update this schedule every 2 weeks, in order to accomodate as much as possible for holydays or other things.

See People for mappings between real names and IRC handles.

Day Americas EMEA
Monday June 18th Edgar Magana Gary Kotton
Tuesday June 19th TBA TBA
Wednesday June 20th TBA TBA
Thursday June 21st TBA Gary Kotton
Friday June 22nd Dan Wendlandt TBA
Saturday June 23rd
Sunday June 24th
Monday June 25th Edgar Magana Gary Kotton
Tuesday June 26th TBA SalvatoreOrlando
Wednesday June 27th TBA TBA
Thursday June 28th TBA Gary Kotton
Friday June 29th Dan Wendlandt SalvatoreOrlando
Saturday June 30th
Sunday July 1st
Retrieved from "https://wiki.openstack.org/w/index.php?title=Neutron/ReviewDays&oldid=14837"