Hello,<br><br>Just a quick reminder for the best practice on the review process as defined at [1], which can be summarized as follow:<br><br> * first core revieviewer should just add +2 and leave the approved flag alone<br>
 * second core reviewer add +2 and +1 approved<br><br>So that you know, and unless there is a good reason to bypass the process, we do want to have at least 2 core reviewer to validate a patch before merging it.  When you mark is +1 approved, it is immediately merged, which may not have been obvious to everyone in the ceilometer core team.<br>
<br>[1] <a href="http://wiki.openstack.org/GerritJenkinsGithub#Reviewing_a_Change">http://wiki.openstack.org/GerritJenkinsGithub#Reviewing_a_Change</a><br><br>This finishes this process reminder interruption, you can now go back to coding :)<br>
<br>Nick<br>