<br><br><div class="gmail_quote">On Wed, May 30, 2012 at 8:42 AM, Andrew Hutchings <span dir="ltr"><<a href="mailto:andrew@linuxjedi.co.uk" target="_blank">andrew@linuxjedi.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Doug,<br>
<div class="im"><br>
On 30/05/12 13:14, Doug Hellmann wrote:<br>
> I started working on setting one up yesterday, and plan to continue<br>
> working on it later today. I would appreciate any feedback you could<br>
> give me on:<br>
><br>
> <a href="https://github.com/dhellmann/ceilometer/commit/d2855c656c5442e46a4a891ff91d6be0abff9706" target="_blank">https://github.com/dhellmann/ceilometer/commit/d2855c656c5442e46a4a891ff91d6be0abff9706</a><br>
<br>
</div>I'm not an expert but it looks good to me.  Best thing to do is to<br>
propose a change via. gerrit.  This will run a check test with the new<br>
tox.ini.  You can then amend the commit and review again to upload<br>
further patchsets if you find some things need changing.<br>
<br>
Once this is in your other commits can simply be rebased to use it.<br></blockquote><div><br></div><div>It looks like we've got it worked out. </div><div><br></div><div>Thanks,</div><div>Doug</div><div><br></div></div>