<div dir="ltr">Skipped the argument because it was me who proposed Gerrit. ;p. Waiting for others' comments.<div><br></div><div style>The CI is in the progress. The Jenkins will check every patch committed to the Github repo. In the future, we can improve further by checking every patch committed to the maillist even before checking into the Github repo.</div>
<div style><br></div><div style>IMHO, what is more important is we need more tests. Jenkins-self should be enough for CI.</div><div style><br></div><div style>Regards,</div><div style>Wenhao</div><div style><br></div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 7:26 PM, Joseph Glanville <span dir="ltr"><<a href="mailto:joseph@cloudscaling.com" target="_blank">joseph@cloudscaling.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sun, Jun 16, 2013 at 6:52 PM, Wenhao Xu <<a href="mailto:wenhao@zelin.io">wenhao@zelin.io</a>> wrote:<br>

> Hi Joseph,<br>
><br>
> Thanks for the proposal. Integrating Gerrit (and other ticket systems) into<br>
> the review process has been discussed several times. However, the mail list<br>
> has been working well for sheepdog for a long time. And we don't see<br>
> sticking to maillist for bug reporting and code review is a bad idea at the<br>
> current stage.<br>
><br>
> But I think the community is still open to Gerrit if it could improve the<br>
> review process a lot more in the future, e.g. when we have more developers.<br>
><br>
><br>
> FYI, Paste some old arguments from our awesome developers on the maillist<br>
> about Gerrit:<br>
<br>
</div>I don't want to start and argument but I think it's worth debunking<br>
some of these.<br>
<div class="im"><br>
> "<br>
> I have some awful experience of Gerrit system, it just looks to me<br>
> lifting higher barrier and throwing more problems than it solves:<br>
><br>
>  1 the review process in hidden in the Gerrit and not as open as current<br>
> plain patch in the mailing list<br>
<br>
</div>You can have gerrit post the patchsets to the mailing list. You can<br>
also then discuss them and emails will appear as comments in the<br>
gerrit system. This can be annoying to setup but it definitely works.<br>
<div class="im"><br>
>  2 need more steps of submitting the patch set than current only 'git<br>
> send-email' one liner.<br>
<br>
</div>When you have your changeset simply type git review, this will<br>
automatically create a review and trigger any hooks on the gerrit side<br>
like triggering a jenkins build and emailing the patchset to the<br>
mailing list.<br>
<div class="im"><br>
>  3 the patch set and its history isn't indexable or logged by other<br>
> system like <a href="http://marc.info" target="_blank">marc.info</a><br>
<br>
</div>As already noted you can do some of this by mailing patchsets to the<br>
list, that aside gerrit is very indexable as can be seen if you try<br>
search for openstack or android changes.<br>
<div class="im"><br>
>  4 It is very cumbersome than plain mail text.<br>
<br>
</div>I disagree because now that patchsets are stored as git refs in the<br>
remote repository which is very easy to work with.<br>
But this is a matter of opinion. :)<br>
<div class="im"><br>
> "<br>
> "<br>
> IMHO, Gerrit is too complex to review patches.<br>
> When I contributed a patch to OpenStack, I thought that Gerrit was a<br>
> barrier for new developers to join community.  However, if I'm in a<br>
> minority group, I'm not against introducing it.<br>
> "<br>
> ...<br>
><br>
><br>
> Sheepdog is compatible with openstack anyway, even when we have two<br>
> different review systems, don't we? ;)<br>
<br>
</div>Hehe there is more to compatibility than the code working together. :P<br>
<br>
As the test suite becomes more fully fledged the importance of making<br>
use of it increases. Having a proper CI system like this will ensure<br>
tests are run against every patchset.<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
><br>
> Regards,<br>
> Wenhao<br>
><br>
><br>
><br>
><br>
> On Mon, Jun 17, 2013 at 9:19 AM, Joseph Glanville <<a href="mailto:joseph@cloudscaling.com">joseph@cloudscaling.com</a>><br>
> wrote:<br>
>><br>
>> How do people feel about integrating Gerrit at some stage for code review?<br>
>><br>
>> If Sheepdog wants to be closer aligned with OpenStack this would be a<br>
>> great way of making the 2 ecosystems more compatible. :)<br>
>><br>
>> On Sun, Jun 16, 2013 at 5:13 PM, MORITA Kazutaka<br>
>> <<a href="mailto:morita.kazutaka@gmail.com">morita.kazutaka@gmail.com</a>> wrote:<br>
>> > At Fri, 14 Jun 2013 11:44:25 +0800,<br>
>> > Kai Zhang wrote:<br>
>> >><br>
>> >> Hi all<br>
>> >><br>
>> >> I'm very happy to announce that a public Jenkins server for sheepdog<br>
>> >> has been set up.<br>
>> >> Both server and hostname are sponsored by <a href="http://zelin.io" target="_blank">zelin.io</a><br>
>> >><br>
>> >> Please check <a href="http://jenkins.sheepdog-project.org" target="_blank">http://jenkins.sheepdog-project.org</a> for more.<br>
>> ><br>
>> > Great, thanks a lot!<br>
>> ><br>
>> >><br>
>> >> Currently, there are only 2 jobs.<br>
>> >> One is for daily build. The other is for testing with local driver.<br>
>> >> (This is a downstream job of former one)<br>
>> >><br>
>> >> However, it is still far from enough.<br>
>> >> The fowling things will be done soon:<br>
>> >> - add github hook so that each commit will trigger a new build<br>
>> >> - generate graphical testing report<br>
>> >> - add jobs for testing zookeeper and corosync<br>
>> >> - fix testing errors<br>
>> >><br>
>> >> For security reason, only login user can configure jobs and sign up has<br>
>> >> been disabled.<br>
>> >> It would be very appreciated if you would like to contribute on<br>
>> >> Jenkins. And I will create an account for you.<br>
>> ><br>
>> > Can you create an account for me?  I'd be happy to help you.<br>
>> ><br>
>> > Thanks,<br>
>> ><br>
>> > Kazutaka<br>
>> > --<br>
>> > sheepdog mailing list<br>
>> > <a href="mailto:sheepdog@lists.wpkg.org">sheepdog@lists.wpkg.org</a><br>
>> > <a href="http://lists.wpkg.org/mailman/listinfo/sheepdog" target="_blank">http://lists.wpkg.org/mailman/listinfo/sheepdog</a><br>
>> --<br>
>> sheepdog mailing list<br>
>> <a href="mailto:sheepdog@lists.wpkg.org">sheepdog@lists.wpkg.org</a><br>
>> <a href="http://lists.wpkg.org/mailman/listinfo/sheepdog" target="_blank">http://lists.wpkg.org/mailman/listinfo/sheepdog</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>