<p dir="ltr"><br>
On Dec 21, 2015 10:19 AM, "Asselin, Ramy" <<a href="mailto:ramy.asselin@hpe.com">ramy.asselin@hpe.com</a>> wrote:<br>
><br>
> Hi Phillip,<br>
><br>
> Yes, please offer a patch to that repo Anita suggested. There's a small group of us actively working on improving the code and eventually getting ci-watch deployed in <a href="http://openstack.org">openstack.org</a>. Your patch and help would be very much appreciated.</p>
<p dir="ltr"> +1</p>
<p dir="ltr">><br>
> We also meet bi-weekly in the 3rd party ci working group: <a href="https://wiki.openstack.org/wiki/Meetings/ThirdParty">https://wiki.openstack.org/wiki/Meetings/ThirdParty</a><br>
> We also discuss some issues in #openstack-third-party-ci<br>
><br>
> Thanks!<br>
> Ramy<br>
><br>
> -----Original Message-----<br>
> From: Anita Kuno [mailto:<a href="mailto:anteaya@anteaya.info">anteaya@anteaya.info</a>]<br>
> Sent: Monday, December 21, 2015 6:52 AM<br>
> To: <a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a><br>
> Subject: Re: [openstack-dev] [ThirdParty][CI] [patch] Status page at <a href="http://ci-watch.tintri.com/project">http://ci-watch.tintri.com/project</a><br>
><br>
> On 12/21/2015 09:20 AM, Philipp Marek wrote:<br>
> > Hi all,<br>
> ><br>
> > I quite like the page at <a href="http://ci-watch.tintri.com/project">http://ci-watch.tintri.com/project</a> - it gives<br>
> > a very quick overview about the failures one should look into, and<br>
> > which to ignore ;)<br>
> ><br>
> ><br>
> > Please let me state before anything else that I don't know any of the<br>
> > restrictions that may have led into the current design - it's very<br>
> > likely that I'm just missing a few points, and that some or all of my<br>
> > comments below are invalid anyway. As always, take enough salt!<br>
> ><br>
> ><br>
> > One thing about that page that is bothering me is the performance...<br>
> > my<br>
> > (current) Firefox asks me several times whether I'd like to stop the<br>
> > JS, or whether it should be allowed to continue.<br>
> ><br>
> > With this patch (and a local exported copy of the page) I don't get<br>
> > asked about that any more; it seems to give me a speedup of ~200, as<br>
> > no intermediate lists need to be built and filtered any more:<br>
> ><br>
> > $ diff -u verified.js.orig verified.js<br>
> > --- verified.js.orig    2015-12-21 15:03:45.614529924 +0100<br>
> > +++ verified.js 2015-12-21 15:03:36.114432601 +0100<br>
> > @@ -33,9 +33,9 @@<br>
> >  $(document).ready(function () {<br>
> >    $("colgroup").each(function (i, elem) {<br>
> >      if ($(elem).hasClass("verified-1")) {<br>
> > -      $("#results").find("td").filter(":nth-child(" + (i + 1) + ")").addClass("verified-1");<br>
> > +      $("#results td:nth-child(" + (i + 1) +<br>
> > + ")").addClass("verified-1");<br>
> >      } else if ($(elem).hasClass("verified1")) {<br>
> > -      $("#results").find("td").filter(":nth-child(" + (i + 1) + ")").addClass("verified1");<br>
> > +      $("#results td:nth-child(" + (i + 1) +<br>
> > + ")").addClass("verified1");<br>
> >      }<br>
> >    });<br>
> >    $("#verified1-button").on("click", toggle_verified_plus);<br>
> ><br>
> ><br>
> > Furthermore, I'm wondering whether<br>
> ><br>
> >     <td class="result verified1"><br>
> >         <a href="..."><br>
> >             <p class="success"><span class="glyphicon glyphicon-ok"></span></p><br>
> >         </a><br>
> >     </td><br>
> ><br>
> > couldn't be simplified to<br>
> ><br>
> >     <td class=""><br>
> >         <a href="..." data-success="1"><br>
> >         </a><br>
> >     </td><br>
> ><br>
> > with the rest being done via CSS? Perhaps a <p></p> would be needed<br>
> > within the <a> to get the vertical size right, but everything else<br>
> > should be possible via CSS, I believe.<br>
> ><br>
> > This change should reduce the size of the generated HTML big some 50%<br>
> > or so, too.<br>
> ><br>
> ><br>
> ><br>
> > Thanks for listening - if you disagree, please ignore and continue<br>
> > working on something else ;)<br>
> ><br>
> ><br>
> > Regards,<br>
> ><br>
> > Phil<br>
> ><br>
> ><br>
> > ______________________________________________________________________<br>
> > ____ OpenStack Development Mailing List (not for usage questions)<br>
> > Unsubscribe:<br>
> > <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> > <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
> ><br>
><br>
> The repo is here if you would like to offer your patch via Gerrit.<br>
> <a href="http://git.openstack.org/cgit/openstack-infra/ciwatch/">http://git.openstack.org/cgit/openstack-infra/ciwatch/</a><br>
><br>
> Thanks Philipp,<br>
> Anita.<br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</p>