Closed Bug 912345 Opened 12 years ago Closed 12 years ago

Fix up tryload.js

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philor, Assigned: philor)

References

Details

(Keywords: trychooser)

Attachments

(5 files)

First round: * we aren't looking at Ubuntu for test load, so we're missing nearly all the Linux test load * there's a ton of "call the same function three times with the same three arguments we just used" copy-paste for Linux build load, and there would be for test load except we're pretending that emulator tests (and asan-to-be, which is what started me on this) don't cause or consume any load
Attachment #799260 - Flags: review?(emorley)
Cute how the build load is == linux, and the test load is == linux64, no?
Attachment #799276 - Flags: review?(emorley)
At least things get slightly less confusing once "panda" isn't both a platform (that we no longer build) and also the test load thingy that always loses the Math.max race with tegra.
Attachment #799277 - Flags: review?(emorley)
Depends on: 912350
Can't fix Win64, since the data isn't there, bug 912350.
Imagine my horror when I saw that Mac wasn't working, and at first thought the "build_macos64x_gecko" was actually the name of the platform.
Attachment #799290 - Flags: review?(emorley)
todo: add win2012x64, add win8 tests (though they probably never win the Math.max race).
And I think that's it for this bug.
Attachment #799943 - Flags: review?(emorley)
Attachment #799260 - Flags: review?(emorley) → review+
Attachment #799276 - Flags: review?(emorley) → review+
Attachment #799277 - Flags: review?(emorley) → review+
Attachment #799290 - Flags: review?(emorley) → review+
Comment on attachment 799943 [details] [diff] [review] [tools] Add Win64 and Win8 load Review of attachment 799943 [details] [diff] [review]: ----------------------------------------------------------------- Lgtm & works well locally :-)
Attachment #799943 - Flags: review?(emorley) → review+
Thank you for doing these :-)
Depends on: 913007
Component: Tools → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: