<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 3 September 2015 at 07:15, Reyna, David L (Wind River) <span dir="ltr"><<a href="mailto:david.reyna@windriver.com" target="_blank">david.reyna@windriver.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It appears that a fix for 8187 in included in "[Toaster] [review-request][PATCH 1/3] toaster: hide irrelevant".<br>
<br>
-  Â  Â  Â  queryset = Build.objects.filter(outcome__lte = Build.IN_PROGRESS)<br>
+  Â  Â  Â  queryset = Build.objects.filter(project_id = pid)<br>
<br>
We might as well make my 8187 a duplicate of Elliot's 8236, and be done with it.<br></blockquote><div><br></div><div>I think you're right, David. However, the patch you submitted has some extra stuff in it which mine doesn't. It could be that I've missed something. I will compare the two today and see if there's anything we need from your patch which isn't in mine.</div><div><br></div><div>Let me just mention that the patch I'm submitting is an update of a stray inherited branch I'm trying to clean up. The branch did several things, some of which weren't attached to bugs and were incidental fixes. It was accidental that it overlapped with the work you were doing. I'll try to avoid duplication like this in future.</div><div><br></div><div>Elliot</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
- David<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> -----Original Message-----<br>
> From: Barros Pena, Belen [mailto:<a href="mailto:belen.barros.pena@intel.com">belen.barros.pena@intel.com</a>]<br>
> Sent: Wednesday, September 02, 2015 9:22 AM<br>
> To: SMITH, ELLIOT; <a href="mailto:toaster@yoctoproject.org">toaster@yoctoproject.org</a><br>
> Cc: Reyna, David<br>
> Subject: Re: [Toaster] [review-request][PATCH 0/3] Hide irrelevant builds<br>
><br>
><br>
><br>
> On 02/09/2015 17:04, "<a href="mailto:toaster-bounces@yoctoproject.org">toaster-bounces@yoctoproject.org</a> on behalf of Elliot<br>
> Smith" <<a href="mailto:toaster-bounces@yoctoproject.org">toaster-bounces@yoctoproject.org</a> on behalf of<br>
> <a href="mailto:elliot.smith@intel.com">elliot.smith@intel.com</a>> wrote:<br>
><br>
> >Hide "in progress" builds everywhere, and only show builds for<br>
> >the currently-open project on the /project/X/builds page.<br>
> ><br>
> >Changes since 71b0568fa43285f0946fae93fb43cea5f3bbecec are in<br>
> >git://<a href="http://git.yoctoproject.org/poky-contrib" rel="noreferrer" target="_blank">git.yoctoproject.org/poky-contrib</a>, elliot/toaster/hide_builds-8236<br>
> ><a href="http://git.yoctoproject.org/cgit.cgi/poky-" rel="noreferrer" target="_blank">http://git.yoctoproject.org/cgit.cgi/poky-</a><br>
> contrib/log/?h=elliot/toaster/hi<br>
> >de_builds-8236<br>
><br>
> The UI seems to be working correctly:<br>
><br>
> * Builds in progress no longer show<br>
> * Only the builds for the selected project are shown<br>
><br>
> The latter means that this bug also fixes<br>
><br>
> <a href="https://bugzilla.yoctoproject.org/show_bug.cgi?id=8187" rel="noreferrer" target="_blank">https://bugzilla.yoctoproject.org/show_bug.cgi?id=8187</a><br>
><br>
> Which was assigned to David Reyna, who had a patch out for review, I<br>
> believe.<br>
><br>
> Just bringing it up so you can decide which one should go in.<br>
><br>
> Cheers<br>
><br>
> Belén<br>
><br>
> ><br>
> >Related bug: <a href="https://bugzilla.yoctoproject.org/show_bug.cgi?id=8236" rel="noreferrer" target="_blank">https://bugzilla.yoctoproject.org/show_bug.cgi?id=8236</a><br>
> ><br>
> >Alexandru DAMIAN (1):<br>
> >  toaster: hide irrelevant builds in the project builds view<br>
> ><br>
> >Elliot Smith (2):<br>
> >  toaster: Remove cast to date which causes errors during test<br>
> >  toaster: Add tests for /project/X/builds page<br>
> ><br>
> > bitbake/lib/toaster/toastergui/tests.py | 91<br>
> >++++++++++++++++++++++++++++++++-<br>
> > bitbake/lib/toaster/toastergui/views.py | 59 +++++++++++++++------<br>
> > bitbake/toaster-requirements.txt  Â  Â  Â  |  1 +<br>
> > 3 files changed, 134 insertions(+), 17 deletions(-)<br>
> ><br>
> >--<br>
> >Elliot Smith<br>
> >Software Engineer<br>
> >Intel OTC<br>
> ><br>
> >---------------------------------------------------------------------<br>
> >Intel Corporation (UK) Limited<br>
> >Registered No. 1134945 (England)<br>
> >Registered Office: Pipers Way, Swindon SN3 1RJ<br>
> >VAT No: 860 2173 47<br>
> ><br>
> >This e-mail and any attachments may contain confidential material for<br>
> >the sole use of the intended recipient(s). Any review or distribution<br>
> >by others is strictly prohibited. If you are not the intended<br>
> >recipient, please contact the sender and delete all copies.<br>
> ><br>
> >--<br>
> >_______________________________________________<br>
> >toaster mailing list<br>
> ><a href="mailto:toaster@yoctoproject.org">toaster@yoctoproject.org</a><br>
> ><a href="https://lists.yoctoproject.org/listinfo/toaster" rel="noreferrer" target="_blank">https://lists.yoctoproject.org/listinfo/toaster</a><br>
><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Elliot Smith<br>Software Engineer<br>Intel Open Source Technology Centre</div></div></div>
</div></div>