225043 - Migration of old chart data should populate "All Open"

Status: RESOLVED FIXED

(Bugzilla :: Reporting/Charting, enhancement, P1)

Description

[Albert Ting]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20031021 Firebird/0.7
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20031021 Firebird/0.7

When upgrading to 2.17.5, the checksetup.pl does not appear to propagate all the
mining data.  In particular, none of the "All Open" categories were updated. 
But the individual bug_status mining data for each product did get stored into
the series tables.

Reproducible: Always

Steps to Reproduce:
After the upgrade, the new charts shows 0 data points when you try to display a
category with "All Open" status.

Comment 1

[Albert Ting]

Attachment: script to fix the tables

Here's a script that I used to propagate the "All Open" series data.  Seems to
have fixed the problem.

Comment 2

[Gervase Markham [:gerv]]

Oh, I get it.

You are saying that the new "All Open" query should have historical data
populated by adding up the relevant columns from the imported data set?

That's a good idea - an enhancement, but a good idea.

Gerv

Comment 3

[Albert Ting]

Yea, I had thought it was a "bug" or an "admin error" since the "All Open"
category was inconsistent with the existing historical data.  

In any case, thanks for earmarking this for 2.18.  It'll prevent confusion when
upgrading.
 

Comment 4

[Gervase Markham [:gerv]]

Attachment: Patch v.1

This patch populates the new "All Open" series at migration time, based on the
sum of the individual bug counts for all the opened states (NEW + ASSIGNED +
REOPENED + UNCONFIRMED).

Gerv

Comment 5

[Gervase Markham [:gerv]]

Comment on attachment 139832 [details] [diff] [review]
Patch v.1

Dave, Myk: is this worth having in time for the upgrade? b.m.o.'s data will
only get migrated once, so if having historical "All Open" data is useful, we
need to check this patch in.

Gerv

Comment 6

[Gervase Markham [:gerv]]

Dave: can I get an approval on this, since it was applied to b.m.o.?

Gerv

Comment 7

[Dave Miller [:justdave]]

Comment on attachment 139832 [details] [diff] [review]
Patch v.1

it seems to have worked on b.m.o...

Comment 8

[Gervase Markham [:gerv]]

Fixed.

Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.259; previous revision: 1.258
done

Gerv