Closed Bug 951469 Opened 8 years ago Closed 8 years ago

TryChooser is showing "N/A" for all queues

Categories

(Release Engineering :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dholbert, Unassigned)

References

Details

(Keywords: trychooser)

Attachments

(1 file)

STR:
 1. Visit http://trychooser.pub.build.mozilla.org/
 2. Look at the Build Queue / Test Queue entries

ACTUAL RESULTS: They're all N/A
EXPECTED RESULTS: Most should not be N/A
The Browser Console reports a syntax error at tryload.js:91, which I think is pointing to the redundant comma at the end of the final line here:
>        totalTestLoad["macosx64"] = Math.max(
>                                     Math.max(
>                                      load_try_test["mac10.6-rev4"] + load_nontry_test["mac10.6-rev4"],
>                                    , load_try_test["mac10.8"] + load_nontry_test["mac10.8"]);

It's possible that fixing that will fix this.
Attached patch fix_trychooserSplinter Review
Attachment #8349127 - Flags: review?(dholbert)
Comment on attachment 8349127 [details] [diff] [review]
fix_trychooser

This is just dropping the redundant (nested) max() and the redundant comma, yes?

Seems reasonable to me.
Attachment #8349127 - Flags: review?(dholbert) → review+
(aki: Do you know yet whether that tweak actually fixes the N/A-everywhere issue?)
Not yet, no, but it looks like http://hg.mozilla.org/build/tools/rev/e94ddbd4e309 introduced badness, so it's a guess.
Yeah, it does. You can checkout http://hg.mozilla.org/build/tools and open trychooser/index.html in Firefox to hack/test this stuff.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Keywords: trychooser
Component: Tools → General
You need to log in before you can comment on or make changes to this bug.