If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

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

RESOLVED FIXED

Status

Release Engineering
General
P2
normal
RESOLVED FIXED
5 years ago
5 months ago

People

(Reporter: nthomas, Assigned: nthomas)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
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%
(Assignee)

Comment 1

5 years ago
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 czchen@gmail.com |
| 5054440 | Self-serve: Rebuilt by czchen@gmail.com |
+---------+-----------------------------------------+
2 rows in set (0.00 sec)
Blocks: 786290
(Assignee)

Comment 2

5 years ago
Created attachment 678197 [details] [diff] [review]
[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.
Attachment #678197 - Flags: review?(catlee)
(Assignee)

Comment 3

5 years ago
Created attachment 678198 [details] [diff] [review]
[buildapi] Tweak the excludes

Leaving off the leading space allows us to catch the old and new strings.
Attachment #678197 - Attachment is obsolete: true
Attachment #678197 - Flags: review?(catlee)
Attachment #678198 - Flags: review?(catlee)

Updated

5 years ago
Attachment #678198 - Flags: review?(catlee) → review+
(Assignee)

Comment 4

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

http://hg.mozilla.org/build/buildapi/rev/90ac9178ea14
Attachment #678198 - Flags: checked-in+
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
Component: Tools → General
Product: Release Engineering → Release Engineering
You need to log in before you can comment on or make changes to this bug.