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

tinderbox panel output should be sorted by build start, not build end

RESOLVED FIXED

Status

Webtools Graveyard
Tinderbox
RESOLVED FIXED
13 years ago
3 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

Details

(Whiteboard: [patch])

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

13 years ago
The tinderbox panel output is currently sorted in reverse order by build end. 
This isn't all that useful for telling how things are changing.  I think it
would be more useful for it to be sorted by build start -- this allows one to
tell if things are turning orange or turning green much more easily.

Patch to follow, except I can't test it.  The patch also makes the static
version link to the static full page (no change), and the dynamic one link to
the dynamic full page (change).

Anybody have a tinderbox server this could be tested on?
(Assignee)

Comment 1

13 years ago
Created attachment 170321 [details] [diff] [review]
patch
(Assignee)

Comment 2

13 years ago
Created attachment 170322 [details] [diff] [review]
patch
Attachment #170321 - Attachment is obsolete: true
(Assignee)

Updated

13 years ago
Whiteboard: [patch]

Comment 3

13 years ago
Comment on attachment 170322 [details] [diff] [review]
patch

+    print "http://tinderbox.mozilla.org/$::tree/";
+  } else {
+    print "http://tinderbox.mozilla.org/showbuilds.cgi?tree=$::tree";

please make that "/$::tree/" and "/showbuilds.cgi?tree=$::tree"
if i did have a tinderbox1 handy, those items wouldn't work because they're
hard coded :(
Attachment #170322 - Flags: review+
(Assignee)

Comment 4

13 years ago
Created attachment 170347 [details] [diff] [review]
patch

This makes them even more relative than suggested.
Attachment #170322 - Attachment is obsolete: true

Updated

13 years ago
Attachment #170347 - Flags: review+
(Assignee)

Comment 5

12 years ago
Landed tbglobals.pl comment change; still need to test showbuilds.cgi change.

Comment 6

12 years ago
When you have it, please offer a link to an example of this change in action.

Updated

12 years ago
Blocks: 333778
(Assignee)

Comment 7

12 years ago
Checked in, since this was tested on the live instance (bug 333778).
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.