Created attachment 533584 [details] [diff] [review] Patch v1 Because of bug 657889, we can't access to build.mozilla.org from the Paris office which makes TBPL always showing "Loading 50%" with no error message. This patch adds a timeout value (10s) so an error message is shown when the URL isn't accessible.
Attachment #533584 - Flags: review?
Attachment #533584 - Flags: review? → review?(mstange)
Attachment #533584 - Flags: review?(mstange) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [needs review]
Ah, this was why I was claiming in bug 657540 comment 6 that tip was broken, and would only load pushes and pending but not tinderbox, or pushes and tinderbox but not pending. I'm real happy that you all have nice fast connections, but if tbpl.m.o is going to insist that every part must load in 10 seconds or less, even when I'm on my fringe 3G connection restoring a session with three trees open, then I won't be able to use it anymore.
Can we add a query param for the timeout so it's possible to override the default value?
And apparently we only show the error message for a few parts, since I've only seen it once in 30 or 40 failed broken partial loads.
Backed out: http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/a1838a15ad48 http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/512e82d808ca
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Marking this as WONTFIX given that I'm not planning to fix this. Feel free to reopen if someone wants to fix this bug.
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago → 6 years ago
Resolution: --- → WONTFIX
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.