<div dir="ltr"><div>I know that there is jenkins plugin for Sonar. I wonder whether we could use it to make sure that<br></div>new code is following good practices.<br><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 5, 2016 at 11:52 AM, Roy Golan <span dir="ltr"><<a href="mailto:rgolan@redhat.com" target="_blank">rgolan@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Thu, May 5, 2016 at 11:05 AM, Roman Mohr <span dir="ltr"><<a href="mailto:rmohr@redhat.com" target="_blank">rmohr@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Thu, May 5, 2016 at 9:48 AM, Yaniv Kaul <span dir="ltr"><<a href="mailto:ykaul@redhat.com" target="_blank">ykaul@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Thu, May 5, 2016 at 10:42 AM, Roman Mohr <span dir="ltr"><<a href="mailto:rmohr@redhat.com" target="_blank">rmohr@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Thu, May 5, 2016 at 9:32 AM, Eyal Edri <span dir="ltr"><<a href="mailto:eedri@redhat.com" target="_blank">eedri@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Nice,<div>So its an online hosted Sonar instance for open source projects?</div></div></blockquote><div><br></div></span><div>Exactly.<br></div><span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>What about other projects in oVirt like VDSM?</div></div></blockquote></span><div><br>We can ask them. Don't see any problems there.<br>I would just start with the engine and see how the whole process works.<br></div></div></div></div></blockquote><div><br></div></span><div>On one hand Engine would be nice because we can compare it to Coverity. <br></div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>On the other hand, I'd start with VDSM, as we don't have today something for it - and it's a smaller project.</div></div></div></div></blockquote><div><br></div></span><div>I have not much experience with sonar and python. Further I don't know if travis support is already don for VDSM. There was a thread a few weeks ago regarding this topic.<br></div><div><br>Doing it with the engine will be very very easy for me. I can just do it while I work on other stuff ;)<br></div></div></div></div></blockquote><div><br></div></span><div><br>I also suggest starting with the engine since the effort is small. Can they report by mail on severity? <br><br></div><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><div><div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><font color="#888888"><div>Y.</div></font></span><div><div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div></div><div><div><div class="gmail_quote"><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>E.</div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Thu, May 5, 2016 at 10:13 AM, David Caro Estevez <span dir="ltr"><<a href="mailto:dcaro@redhat.com" target="_blank">dcaro@redhat.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On 05/05 09:13, David Caro Estevez wrote:<br>
><br>
> Hey Roman,<br>
><br>
> Adding the infra list<br>
<br>
Forgot to add them XP<br>
<br>
><br>
> On 05/05 08:57, Roman Mohr wrote:<br>
> > Hi David,<br>
> ><br>
> > I have asked sonarqube if they would add ovirt-engine to<br>
> > <a href="https://nemo.sonarqube.org/" rel="noreferrer" target="_blank">https://nemo.sonarqube.org/</a>.<br>
> ><br>
> > sonarqube is a pretty nice tool for source code analysis. It has a slightly<br>
> > different focus than coverity and could be very useful for us.<br>
><br>
> Have you discussed this with the ovirt-engine maintainers/devs? Not that I<br>
> think it would be an issue, but usually people don't like surprises :)<br>
><br>
> ><br>
> > They are happy to add us. In the past they just built everything on nemo<br>
> > and published the results but they are switching to building on travis and<br>
> > just upload the results.<br>
> ><br>
> > Do you think you could give me access to our ovirt-engine github repo?<br>
><br>
> I can add the project, no problem, you can just make sure to create the new<br>
> branch with the travis yaml (if noone has issues with it).<br>
><br>
> ><br>
> > I would do the following:<br>
> > - prepare a .travis.yml file on a separate branch<br>
> > - configure an account on nemo with the help of a sonarqube guy<br>
><br>
> ^ the accounts are free? Can we create a project and add multiple admin<br>
> accounts? If not, we should find a way to share that account to avoid a single<br>
> maintainer<br>
><br>
> > - enable travis builds<br>
> > - when everything works I would add the .travis.yml file through a normal<br>
> > gerrit patch<br>
> > - give up my github permissions if required ;)<br>
> ><br>
> > Roman<br>
<span><font color="#888888">><br>
> --<br>
> David Caro<br>
><br>
> Red Hat S.L.<br>
> Continuous Integration Engineer - EMEA ENG Virtualization R&D<br>
><br>
> Tel.: <a href="tel:%2B420%20532%20294%20605" value="+420532294605" target="_blank">+420 532 294 605</a><br>
> Email: <a href="mailto:dcaro@redhat.com" target="_blank">dcaro@redhat.com</a><br>
> IRC: dcaro|dcaroest@{freenode|oftc|redhat}<br>
> Web: <a href="http://www.redhat.com" rel="noreferrer" target="_blank">www.redhat.com</a><br>
> RHT Global #: 82-62605<br>
<br>
<br>
<br>
--<br>
David Caro<br>
<br>
Red Hat S.L.<br>
Continuous Integration Engineer - EMEA ENG Virtualization R&D<br>
<br>
Tel.: <a href="tel:%2B420%20532%20294%20605" value="+420532294605" target="_blank">+420 532 294 605</a><br>
Email: <a href="mailto:dcaro@redhat.com" target="_blank">dcaro@redhat.com</a><br>
IRC: dcaro|dcaroest@{freenode|oftc|redhat}<br>
Web: <a href="http://www.redhat.com" rel="noreferrer" target="_blank">www.redhat.com</a><br>
RHT Global #: 82-62605<br>
</font></span><br></div></div>_______________________________________________<br>
Infra mailing list<br>
<a href="mailto:Infra@ovirt.org" target="_blank">Infra@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman/listinfo/infra</a><br>
<br></blockquote></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr"><div>Eyal Edri<br>Associate Manager</div><div>RHEV DevOps<br>EMEA ENG Virtualization R&D<br>Red Hat Israel<br><br>phone: <a href="tel:%2B972-9-7692018" value="+97297692018" target="_blank">+972-9-7692018</a><br>irc: eedri (on #tlv #rhev-dev #rhev-integ)</div></div></div></div></div>
</font></span></div>
</blockquote></div><br></div></div></div></div>
</blockquote></div></div></div><br></div></div>
</blockquote></div></div></div><br></div></div>
</blockquote></div></div></div><br></div></div>
</blockquote></div><br></div></div></div></div>