Jump to: navigation, search

Difference between revisions of "Neutron/ReviewDays"

(Schedules)
 
(92 intermediate revisions by 18 users not shown)
Line 1: Line 1:
__NOTOC__
 
We are thinking about having an assigned reviewer for each day.
 
  
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.
+
As we enter the Havana 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.
 +
 
 +
----
 +
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:  
 
* 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.
+
* 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 */
+
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 */
+
== Schedules ==
* APAC: 1AM GMT - 9AM GMT
 
* EMEA: 9AM GMT - 5PM GMT
 
* Americas: 5PM GMT - 1AM GMT
 
  
/* end */
+
Approximately a core reviewer would be available in the following hours:
 +
* APAC: 0100 - 0900 UTC
 +
* EMEA: 0900 - 1700 UTC
 +
* Americas: 1700 - 0100 UTC
  
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 [http://www.openstack.org/community/members/ OpenStack Member Directory] for mappings between real names and IRC handles.
 
+
{| class="wikitable"
See [[People]] for mappings between real names and IRC handles.
 
{| border="1" cellpadding="2" cellspacing="0"
 
|-
 
| '''Day'''
 
| Americas
 
| EMEA
 
|-
 
| Monday July 30th
 
| TBA
 
| TBA
 
|-
 
| Tuesday July 31st
 
| TBA
 
| Salvatore Orlando
 
|-
 
| Wednesday August 1st
 
| Bob Kukura
 
| Salvatore Orlando
 
|-
 
| Thursday August 2nd
 
| Sumit Naiksatam
 
| TBA
 
|-
 
| Friday August 3rd
 
| Dan Wendlandt
 
| Gary Kotton
 
|-
 
| Saturday August 4th
 
| TBA
 
| TBA
 
|-
 
| Sunday August 5th
 
| Somik Behera
 
| Gary Kotton
 
|-
 
| Monday August 6th
 
| Edgar Magana
 
| Gary Kotton
 
|-
 
| Tuesday August 7th
 
| Somik Behera
 
| TBA
 
|-
 
| Wednesday August 8th
 
| Bob Kukura
 
| TBA
 
|-
 
| Thursday August 9th
 
| Sumit Naiksatam / Aaron Rosen
 
| Salvatore Orlando Gary Kotton
 
 
|-
 
|-
| Friday August 10th
+
!colspan="2" |'''Day'''
| Dan Wendlandt
+
!Americas
| Salvatore Orlando
+
!EMEA
 +
!APAC
 
|-
 
|-
| Saturday August 11th
+
| Monday ||  April 29th || || ||  
| TBA
 
| TBA
 
 
|-
 
|-
| Sunday August 12th
+
| Tuesday ||  April 30th || Mark McClain || Gary Kotton || 
| TBA
 
| Gary Kotton  
 
 
|-
 
|-
| Monday August 13th
+
| Wednesday || May 1st || Dan Wendlandt || ||
| Edgar Magana
 
| Salvatore Orlando
 
 
|-
 
|-
| Tuesday August 14th
+
| Thursday || May 2nd || Edgar Magana || Gary Kotton ||     
| Bob Kukura/Aaron Rosen
 
| Salvatore Orlando
 
 
|-
 
|-
| Wednesday August 15th
+
| Friday || May 3rd || Mark McClain,Sumit Naiksatam  || Salvatore Orlando || Akihiro Motoki
| Dan Wendlandt
 
| Gary Kotton
 
 
|-
 
|-
| Thursday August 16th
+
| Saturday || May 4th || || ||
| Aaron Rosen
 
| Gary Kotton
 
 
|-
 
|-
| Friday August 17th
+
| Sunday || May 5th || || Gary Kotton ||
| Sumit Naiksatam
 
| TBA
 
 
|-
 
|-
| Saturday August 18th
+
| Monday || May 6th || Bob Kukura || ||  Akihiro Motoki
| TBA
 
| TBA
 
 
|-
 
|-
| Sunday August 19th
+
| Tuesday ||  May 7th || Mark McClain || Gary Kotton ||  
| TBA
 
| Gary Kotton  
 
|}
 
 
 
For Folsom-3, we have many LARGE reviews.  Please sign up for the set of reviews that you will continue to revisit until they are merged for F-3.
 
{| border="1" cellpadding="2" cellspacing="0"
 
 
|-
 
|-
| Review Topic''' '''
+
| Wednesday || May 8th || Dan Wendlandt || ||
| Review Link
 
| Core Dev #1
 
 
|-
 
|-
| Provider Networks
+
| Thursday || May 9th || Edgar Magana || Gary Kotton ||     
| https://review.openstack.org/#/c/10938/
 
| danwent
 
 
|-
 
|-
| DB models for dns/routes
+
| Friday || May 10th || Mark McClain,Sumit Naiksatam  || Salvatore Orlando || Akihiro Motoki
| https://review.openstack.org/#/c/10791/
 
| sorlando
 
 
|-
 
|-
| RPC support for LB Plugin
+
| Saturday || May 11th || || ||
| https://review.openstack.org/#/c/9591/
 
| sorlando
 
 
|-
 
|-
| RPC support for OVS Plugin
+
| Sunday || May 12th || || Gary Kotton ||
| https://review.openstack.org/#/c/10879/
 
| sorlando
 
 
|-
 
|-
| RPC support for DHCP
+
!colspan="2" <#88FF88>| '''Weekly Defaults'''
| https://review.openstack.org/#/c/10997/
+
!<#88FF88>|Americas
| Gary Kotton
+
!<#88FF88>|EMEA
 +
!<#88FF88>|APAC
 
|-
 
|-
| quotas API
+
| Monday || || Bob Kukura, Nachi Ueno || ||  Akihiro Motoki
| https://review.openstack.org/#/c/10484/
 
| danwent
 
 
|-
 
|-
| Public/shared Networks
+
| Tuesday || || Mark McClain || Gary Kotton || Yong Sheng Gong(irc:gongysh)
| https://review.openstack.org/#/c/9845/
 
| Gary Kotton  
 
 
|-
 
|-
| L3 + Floating IPs
+
| Wednesday || || Dan Wendlandt, Nachi Ueno || ||
| https://review.openstack.org/#/c/10993/
 
| Aaron Rosen
 
 
|-
 
|-
| L3 + Floating IP CLI/client
+
| Thursday || || Edgar Magana || Gary Kotton || Yong Sheng Gong(irc:gongysh)
| https://review.openstack.org/#/c/10994/
 
|  sorlando
 
 
|-
 
|-
| v2 devstack exercise scripts
+
| Friday || || Mark McClain,Sumit Naiksatam || Salvatore Orlando || Akihiro Motoki
| https://review.openstack.org/#/c/10828/
 
| danwent
 
 
|-
 
|-
| <s>Per-network DHCP enable</s>
+
| Saturday || || || ||
| <s>https://review.openstack.org/#/c/10320/</s>
 
| <s>sorlando</s>
 
 
|-
 
|-
| bulk API operations
+
| Sunday || || || Gary Kotton ||
| https://review.openstack.org/#/c/10732/
 
|
 
|-
 
| v1 code removal
 
| https://review.openstack.org/#/c/10998/
 
| Sumit Naiksatam
 
|-
 
| NEC plugin
 
| https://review.openstack.org/#/c/10664/
 
|  Gary Kotton
 
|-
 
| ISC DHCP
 
| https://review.openstack.org/#/c/10443/
 
Gary Kotton  
 
|-
 
| Meta plugin
 
| https://review.openstack.org/#/c/10181/
 
|  Gary Kotton
 
 
|}
 
|}
 
Note: if you click through to a review and already seen that two other core devs are active on it, please sign up for other reviews so we can maximize coverage.
 
 
Note: there are many smaller reviews that we're not listing here. It does not mean that they are less important, just that they require less coordination regarding reviewer cycles.  Also, if a review was for a plugin with an obvious set of core maintainers who would review (Nicira, Cisco), I omitted it.
 

Latest revision as of 00:29, 2 July 2013


As we enter the Havana 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.


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

  • 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.
  • 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!)

Active reviews for both the server and client/CLI repos:

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.

Schedules

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

  • APAC: 0100 - 0900 UTC
  • EMEA: 0900 - 1700 UTC
  • Americas: 1700 - 0100 UTC

See OpenStack Member Directory for mappings between real names and IRC handles.

Day Americas EMEA APAC
Monday April 29th
Tuesday April 30th Mark McClain Gary Kotton
Wednesday May 1st Dan Wendlandt
Thursday May 2nd Edgar Magana Gary Kotton
Friday May 3rd Mark McClain,Sumit Naiksatam Salvatore Orlando Akihiro Motoki
Saturday May 4th
Sunday May 5th Gary Kotton
Monday May 6th Bob Kukura Akihiro Motoki
Tuesday May 7th Mark McClain Gary Kotton
Wednesday May 8th Dan Wendlandt
Thursday May 9th Edgar Magana Gary Kotton
Friday May 10th Mark McClain,Sumit Naiksatam Salvatore Orlando Akihiro Motoki
Saturday May 11th
Sunday May 12th Gary Kotton
Weekly Defaults Americas EMEA APAC
Monday Bob Kukura, Nachi Ueno Akihiro Motoki
Tuesday Mark McClain Gary Kotton Yong Sheng Gong(irc:gongysh)
Wednesday Dan Wendlandt, Nachi Ueno
Thursday Edgar Magana Gary Kotton Yong Sheng Gong(irc:gongysh)
Friday Mark McClain,Sumit Naiksatam Salvatore Orlando Akihiro Motoki
Saturday
Sunday Gary Kotton