← Back to branch summary

~ubuntu-branches/ubuntu/vivid/neutron/vivid

« back to all changes in this revision

Viewing changes to doc/source/policies/code-reviews.rst

  • Committer: Package Import Robot
  • Author(s): James Page
  • Date: 2015-03-30 11:17:19 UTC
  • mfrom: (1.1.21)
  • Revision ID: package-import@ubuntu.com-20150330111719-h0gx7233p4jkkgfh
Tags: 1:2015.1~b3-0ubuntu1
https://launchpad.net/bugs/1263539
* New upstream milestone release:
  - d/control: Align version requirements with upstream.
  - d/control: Add new dependency on oslo-log.
  - d/p/*: Rebase.
  - d/control,d/neutron-plugin-hyperv*: Dropped, decomposed into
    separate project upstream.
  - d/control,d/neutron-plugin-openflow*: Dropped, decomposed into
    separate project upstream.
  - d/neutron-common.install: Add neutron-rootwrap-daemon and 
    neutron-keepalived-state-change binaries.
  - d/rules: Ignore neutron-hyperv-agent when installing; only for Windows.
  - d/neutron-plugin-cisco.install: Drop neutron-cisco-cfg-agent as
    decomposed into separate project upstream.
  - d/neutron-plugin-vmware.install: Drop neutron-check-nsx-config and
    neutron-nsx-manage as decomposed into separate project upstream.
  - d/control: Add dependency on python-neutron-fwaas to neutron-l3-agent.
* d/pydist-overrides: Add overrides for oslo packages.
* d/control: Fixup type in package description (LP: #1263539).
* d/p/fixup-driver-test-execution.patch: Cherry pick fix from upstream VCS
  to support unit test exection in out-of-tree vendor drivers.
* d/neutron-common.postinst: Allow general access to /etc/neutron but limit
  access to root/neutron to /etc/neutron/neutron.conf to support execution
  of unit tests in decomposed vendor drivers.
* d/control: Add dependency on python-neutron-fwaas to neutron-l3-agent
  package.

Show diffs side-by-side

added added

removed removed

Lines of Context:
doc/source/policies/code-reviews.rst
 
1
Neutron Code Reviews
 
2
====================
 
3
 
 
4
Code reviews are a critical component of all OpenStack projects. Neutron accepts patches from many
 
5
diverse people with diverse backgrounds, employers, and experience levels. Code reviews provide a
 
6
way to enforce a level of consistency across the project, and also allow for the careful on boarding
 
7
of contributions from new contributors.
 
8
 
 
9
Neutron Code Review Practices
 
10
-----------------------------
 
11
Neutron follows the `code review guidelines <https://wiki.openstack.org/wiki/ReviewChecklist>`_ as
 
12
set forth for all OpenStack projects. It is expected that all reviewers are following the guidelines
 
13
set forth on that page.
 
14
 
 
15
Neutron Spec Review Practices
 
16
-----------------------------
 
17
In addition to code reviews, Neutron also maintains a BP specification git repository. Detailed
 
18
instructions for the use of this repository are provided `here <https://wiki.openstack.org/wiki/Blueprints>`_.
 
19
It is expected that Neutron core team members are actively reviewing specifications which are pushed out
 
20
for review to the specification repository. In addition, there is a neutron-drivers team, composed of a
 
21
handful of Neutron core reviewers, who can approve and merge Neutron specs.
 
22
 
 
23
Some guidelines around this process are provided below:
 
24
 
 
25
* Once a specification has been pushed, it is expected that it will not be approved for at least 3 days
 
26
  after a first Neutron core reviewer has reviewed it. This allows for additional cores to review the
 
27
  specification.
 
28
* For blueprints which the core team deems of High or Critical importance, core reviewers may be assigned
 
29
  based on their subject matter expertise.
 
30
* Specification priority will be set by the PTL with review by the core team once the specification is
 
31
  approved.
 
32
 
 
33
Tracking Review Statistics
 
34
--------------------------
 
35
Stackalytics provides some nice interfaces to track review statistics. The links are provided below. These
 
36
statistics are used to track not only Neutron core reviewer statistics, but also to track review statistics
 
37
for potential future core members.
 
38
 
 
39
* `30 day review stats <http://stackalytics.com/report/contribution/neutron-group/30>`_
 
40
* `60 day review stats <http://stackalytics.com/report/contribution/neutron-group/60>`_
 
41
* `90 day review stats <http://stackalytics.com/report/contribution/neutron-group/90>`_
 
42
* `180 day review stats <http://stackalytics.com/report/contribution/neutron-group/180>`_