Bug 147359 – webkitbot and WKR unnecessarily rely on webkit-queues.appspot.com

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Alexey Proskuryakov]

appspot is currently down, and so are webkitbot and WKR.

It doesn't appear that their dependency on appspot is needed though - we do get log entries in webkit-queues.appspot.com datastore when the bots start, but these seem pretty useless.

[Alexey Proskuryakov]

Created attachment 257650 [details]
proposed fix

    def begin_work_queue(self):
        _log.info("CAUTION: %s will discard all local changes in \"%s\"" % (self.name, self._tool.scm().checkout_root))

# This is obviously not enabled on bots. Not sure how much we care about the case when someone runs WKR or webkitbot locally.
        if self._options.confirm:
            response = self._tool.user.prompt("Are you sure?  Type \"yes\" to continue: ")
            if (response != "yes"):
                _log.error("User declined.")
                sys.exit(1)
        _log.info("Running WebKit %s." % self.name)
# This raises an exception after a long timeout, and prevents the queue from running.
        self._tool.status_server.update_status(self.name, "Starting Queue")

[WebKit Commit Bot]

Comment on attachment 257650 [details]
proposed fix

Clearing flags on attachment: 257650

Committed r187501: <http://trac.webkit.org/changeset/187501>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.