[webkit-reviews] review granted: [Bug 186635] [GTK][buildbot] Raise timeouts for the step benchmark-test on the GTK perf bot : [Attachment 342779] Patch

[bugzilla-daemon at webkit.org]

Michael Catanzaro <mcatanzaro at igalia.com> has granted Carlos Alberto Lopez
Perez <clopez at igalia.com>'s request for review:
Bug 186635: [GTK][buildbot] Raise timeouts for the step benchmark-test on the
GTK perf bot
https://bugs.webkit.org/show_bug.cgi?id=186635

Attachment 342779: Patch

https://bugs.webkit.org/attachment.cgi?id=342779&action=review




--- Comment #4 from Michael Catanzaro <mcatanzaro at igalia.com> ---
Comment on attachment 342779
  --> https://bugs.webkit.org/attachment.cgi?id=342779
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=342779&action=review

> Tools/BuildSlaveSupport/build.webkit.org-config/steps.py:-831
> -    # Buildbot default timeout without output for a step is 1200.
> -    # The current maximum timeout for a benchmark plan is also 1200.
> -    # So raise the buildbot timeout to avoid aborting this whole step when a
test timeouts.
> -    timeout = 1500

Weird that this was added if it has been ignored this whole time... maybe it
regressed at some point in buildbot...?