Jump to: navigation, search

Difference between revisions of "Neutron/ReviewDays"

Line 19: Line 19:
 
* EMEA: 9AM GMT - 5PM GMT
 
* EMEA: 9AM GMT - 5PM GMT
 
* Americas: 5PM GMT - 1AM GMT
 
* Americas: 5PM GMT - 1AM GMT
Th
+
 
/* end */
+
Th /* 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.
 
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.
Line 32: Line 32:
 
|-
 
|-
 
| Tuesday June 26th  
 
| Tuesday June 26th  
| Bob Kukura
+
| Bob Kukura  
 
| [[SalvatoreOrlando]]  
 
| [[SalvatoreOrlando]]  
 
|-
 
|-
Line 39: Line 39:
 
| TBA  
 
| TBA  
 
|-
 
|-
|rowspan="2" |Thursday June 28th  
+
|<style="text-align: center;" |2>|Thursday June 28th  
|rowspan="2" |TBA  
+
|<style="text-align: center;" |2>|TBA  
 
| Gary Kotton  
 
| Gary Kotton  
 
|-
 
|-
Line 57: Line 57:
 
|-
 
|-
 
| Monday July 2nd  
 
| Monday July 2nd  
| Edgar Magana  
+
| Edgar Magana  
| TBA  
+
| TBA  
 
|-
 
|-
 
| Tuesday July 3rd  
 
| Tuesday July 3rd  
| Dan Wendlandt  
+
| Dan Wendlandt  
| Salvatore Orlando  
+
| Salvatore Orlando  
 
|-
 
|-
 
| Wednesday July 4th  
 
| Wednesday July 4th  
 
| TBA  
 
| TBA  
| Gary Kotton  
+
| Gary Kotton  
 
|-
 
|-
 
| Thursday July 5th  
 
| Thursday July 5th  
| TBA  
+
| TBA  
| Salvatore Orlando  
+
| Salvatore Orlando  
 
|-
 
|-
 
| Friday July 6th  
 
| Friday July 6th  
| Sumit Naiksatam  
+
| Sumit Naiksatam  
| TBA  
+
| TBA  
 
|-
 
|-
 
| Saturday July 7th  
 
| Saturday July 7th  
| TBA  
+
| TBA  
| TBA  
+
| TBA  
 
|-
 
|-
 
| Sunday July 8th  
 
| Sunday July 8th  
| Edgar Magana  
+
| Edgar Magana  
| Gary Kotton  
+
| Gary Kotton  
 
|-
 
|-
 
| Monday July 9th  
 
| Monday July 9th  
| TBA  
+
| TBA  
| Gary Kotton  
+
| Gary Kotton  
 
|-
 
|-
 
| Tuesday July 10th  
 
| Tuesday July 10th  
| Salvatore Orlando  
+
| Salvatore Orlando  
| TBA  
+
| TBA  
 
|-
 
|-
 
| Wednesday July 11th  
 
| Wednesday July 11th  
 
| Dan Wendlandt  
 
| Dan Wendlandt  
| TBA  
+
| TBA  
 
|-
 
|-
 
| Thursday July 12th  
 
| Thursday July 12th  
| Salvatore Orlando  
+
| Salvatore Orlando  
| Gary Kotton  
+
| Gary Kotton  
 
|-
 
|-
 
| Friday July 13th  
 
| Friday July 13th  
| Sumit Naiksatam  
+
| Sumit Naiksatam  
| TBA  
+
| TBA  
 
|-
 
|-
 
| Saturday July 14th  
 
| Saturday July 14th  
| TBA  
+
| TBA  
| TBA  
+
| TBA  
 
|-
 
|-
 
| Sunday July 15th  
 
| Sunday July 15th  
| TBA  
+
| TBA  
| Gary Kotton
+
| Gary Kotton  
 +
|}

Revision as of 21:54, 9 July 2012

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

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 */

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

Th /* 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
Tuesday June 26th Bob Kukura SalvatoreOrlando
Wednesday June 27th Sumit Naiksatam TBA
2>|Thursday June 28th 2>|TBA Gary Kotton
Friday June 29th Dan Wendlandt SalvatoreOrlando
Saturday June 30th
Sunday July 1st Gary Kotton
Monday July 2nd Edgar Magana TBA
Tuesday July 3rd Dan Wendlandt Salvatore Orlando
Wednesday July 4th TBA Gary Kotton
Thursday July 5th TBA Salvatore Orlando
Friday July 6th Sumit Naiksatam TBA
Saturday July 7th TBA TBA
Sunday July 8th Edgar Magana Gary Kotton
Monday July 9th TBA Gary Kotton
Tuesday July 10th Salvatore Orlando TBA
Wednesday July 11th Dan Wendlandt TBA
Thursday July 12th Salvatore Orlando Gary Kotton
Friday July 13th Sumit Naiksatam TBA
Saturday July 14th TBA TBA
Sunday July 15th TBA Gary Kotton