1249614 - Release Tracking Report missed bug

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, defect)

Description

[Carsten Book [:Tomcat]]

Sylvestre pinged me about https://bugzilla.mozilla.org/show_bug.cgi?id=1208226 that apprently missed the uplift.

I did a search and found out something is wrong.

Steps to reproduce:
2016-02-15 02:57:22 PST - Bug 1208226 was marked as approval beta+

so if you use https://bugzilla.mozilla.org/page.cgi?id=release_tracking_report.html and set 
: Show where was changed to (and is currently) approval-beta + between 2016-01-05 and 2016-02-15 and firefox 45 not fixed doesnt' have this bug 

if also not 2016-02-16 -> now range don't have this bug 

The only query where this bug is shown up is 2015-11-24 -> 2016-01-04 
which was the Firefox 44 release i think..

But that bug was not marked as approval between that time range ?

Comment 1

[Carsten Book [:Tomcat]]

(In reply to Carsten Book [:Tomcat] from comment #0)
> Sylvestre pinged me about
> https://bugzilla.mozilla.org/show_bug.cgi?id=1208226 that apprently missed
> the uplift.
> 
> I did a search and found out something is wrong.

oh never mind my bad - that bug above is complete missing in the bug queries:

     Bug ID: 1218473, 1233182, 1235855, 1212133, 1238892, 1239823, 1242641, 1242293, 1240848 

so its still valid for so if you use https://bugzilla.mozilla.org/page.cgi?id=release_tracking_report.html and set 
: Show where was changed to (and is currently) approval-beta + between 2016-01-05 and 2016-02-15 and firefox 45 not fixed doesnt' have this bug  that bug 1208226 is missing

Comment 2

[Carsten Book [:Tomcat]]

also bug 1208226 don't show up here for the aurora query :(

https://bugzilla.mozilla.org/buglist.cgi?bug_id=1218473%2C1237775%2C1240783%2C1240559%2C1242293%2C1242720%2C1243413%2C1229763%2C1240848&list_id=12863752

Comment 3

[Carsten Book [:Tomcat]]

just as idea, could the problem be that the bug state in question is not fixed (reopen in this case) and that we miss such cases ?

Comment 4

[David Lawrence [:dkl]]

(In reply to Carsten Book [:Tomcat] from comment #0)
> Steps to reproduce:
> 2016-02-15 02:57:22 PST - Bug 1208226 was marked as approval beta+
> 
> so if you use
> https://bugzilla.mozilla.org/page.cgi?id=release_tracking_report.html and
> set 
> : Show where was changed to (and is currently) approval-beta + between
> 2016-01-05 and 2016-02-15 and firefox 45 not fixed doesnt' have this bug 

One thing to note is the actual dates in the SQL query end up being:

AND (a.bug_when >= '2016-01-05 00:00:00')
AND (a.bug_when < '2016-02-15 00:00:00')

So the 2016-02-15 02:57:22 PST date will not fall within that range which would explain why it is missing.

And the next date range in the list will not get it either as it is between 2016-02-16 and 2016-03-28.
So this is an issue in that there is a complete day in which we cannot use in our results.

I will see if I can fix this properly.

> The only query where this bug is shown up is 2015-11-24 -> 2016-01-04 
> which was the Firefox 44 release i think..
> 
> But that bug was not marked as approval between that time range ?

I do not see the bug id in the list when using that search criteria. I see 1208876 which is close.

dkl

Comment 5

[Carsten Book [:Tomcat]]

(In reply to David Lawrence [:dkl] from comment #4)
> I do not see the bug id in the list when using that search criteria. I see
> 1208876 which is close.
> 
> dkl

yeah sorry my bad, was more confusing myself :)

Comment 6

[David Lawrence [:dkl]]

Attachment: 1249614_1.patch

Quick sanity check on my solution. Thanks!

Comment 7

[:glob ✱]

Comment on attachment 8721862 [details] [diff] [review]
1249614_1.patch

Review of attachment 8721862 [details] [diff] [review]:
-----------------------------------------------------------------

::: extensions/BMO/lib/Reports/ReleaseTracking.pm
@@ +370,4 @@
>              push @where, "(a.bug_when >= ?)";
>              push @params, $q->{start_date} . ' 00:00:00';
>              push @where, "(a.bug_when < ?)";
> +            push @params, $q->{end_date} . ' 23:59:59';

the operator should be <= to match end_dates on the last second of the day.
otherwise lgtm

Comment 8

[David Lawrence [:dkl]]

Thanks!

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   d78003b..4541fb3  master -> master

dkl