808460 - Wait time reports don't exclude rebuilds created by self-serve

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Nick Thomas [:nthomas] (UTC+12)]

Eg:
Wait time report for testpool for jobs submitted between Fri, 02 Nov 2012 23:00:00 -0800 (PST) and Sat, 03 Nov 2012 23:00:00 -0800 (PST)

fedora: 1821
  0:     1768    97.09%
 15:       43     2.36%
 30:        8     0.44%
 45:        0     0.00%
 60:        0     0.00%
 75:        0     0.00%
 90+:        2     0.11%

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

If you chase down the buildrequests you find buildsetid's with:

mysql> select id, reason from buildsets where id in (5054437,5054440);
+---------+-----------------------------------------+
| id      | reason                                  |
+---------+-----------------------------------------+
| 5054437 | Self-serve: Rebuilt by [e-mail] |
| 5054440 | Self-serve: Rebuilt by [e-mail] |
+---------+-----------------------------------------+
2 rows in set (0.00 sec)

Comment 2

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: [buildapi] Tweak the excludes

http://mxr.mozilla.org/build/search?string=WAITTIMES_BUILDREQUESTS_BUILDERNAME_SQL&find=buildapi shows this exclude list is only used in the wait time reports.

Comment 3

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: [buildapi] Tweak the excludes

Leaving off the leading space allows us to catch the old and new strings.

Comment 4

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 678198 [details] [diff] [review]
[buildapi] Tweak the excludes

http://hg.mozilla.org/build/buildapi/rev/90ac9178ea14