Bug 136791 - Mac queue slaves should reboot every so often
Summary: Mac queue slaves should reboot every so often
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-09-12 15:29 PDT by Ryosuke Niwa
Modified: 2014-09-25 23:15 PDT (History)
3 users (show)

See Also:


Attachments
Fixes the bug (1.53 KB, patch)
2014-09-12 15:31 PDT, Ryosuke Niwa
no flags Details | Formatted Diff | Diff
Reboot between 1am and 6am (1.62 KB, patch)
2014-09-16 14:23 PDT, Ryosuke Niwa
ap: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2014-09-12 15:29:33 PDT
Right now, Mac EWS, commit queue, etc... don't reboot ever.

Reboot them every 24 hours to keep the sanity.
Comment 1 Ryosuke Niwa 2014-09-12 15:31:34 PDT
Created attachment 238067 [details]
Fixes the bug
Comment 2 Alexey Proskuryakov 2014-09-12 15:56:57 PDT
Comment on attachment 238067 [details]
Fixes the bug

Ideally, it would only reboot at night, so that this doesn't happen when someone is logged into the machine, investigating something.

I don't know how to even formally express the wish though.
Comment 3 Lucas Forschler 2014-09-12 16:00:38 PDT
Maybe we could target a range of time to update... say between 3am and 6am. Then, select our 'time to reboot' to fall into that range.
Comment 4 Ryosuke Niwa 2014-09-12 16:27:39 PDT
Yeah, or the first time we finished past midnight?
Comment 5 Ryosuke Niwa 2014-09-16 14:23:43 PDT
Created attachment 238210 [details]
Reboot between 1am and 6am
Comment 6 Alexey Proskuryakov 2014-09-16 14:51:09 PDT
Comment on attachment 238210 [details]
Reboot between 1am and 6am

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

> Tools/ChangeLog:8
> +        Reboot Mac queue slaves every 12 hours between 1 A.M. and 6 A.M.

The end result would be closer to every 24 hours than every 12 hours, wouldn't it? I'd just say "every night between 1am and 6am".

> Tools/EWSTools/start-queue-mac.sh:41
> +TIME_TO_REBOOT=$(( $(date +%s) + 3600 * 12)) # Reboot every 12 hours between 1am and 6am.

Perhaps a "why" comment would help here (we only want to reboot at night to reduce the likelihood of confusing unexpected things happening while people are looking).
Comment 7 Ryosuke Niwa 2014-09-16 15:15:08 PDT
Committed r173671: <http://trac.webkit.org/changeset/173671>
Comment 8 Alexey Proskuryakov 2014-09-25 23:15:05 PDT
So the bot will only reboot if it happens to finish during this time period - it will not if it was idle all night, correct?