Bug 69089 - EWS bots shouldn't die if a buggy patch with r? modifies the source code of the EWS
Summary: EWS bots shouldn't die if a buggy patch with r? modifies the source code of t...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2011-09-29 09:40 PDT by Csaba Osztrogonác
Modified: 2019-09-03 12:39 PDT (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2011-09-29 09:40:11 PDT
https://bugs.webkit.org/attachment.cgi?id=109089 in https://bugs.webkit.org/show_bug.cgi?id=68973
killed all EWS bots, because the proposed patch updated the code of the EWS, and it was incorrect.

I think we should find a way to protect EWS from buggy python patches in bugzilla with r? .


error log:

Failed to run "['/storage/WebKit-qt-ews/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-i..." exit_code: 1

Traceback (most recent call last):
  File "/storage/WebKit-qt-ews/Tools/Scripts/webkit-patch", line 74, in <module>
    main()
  File "/storage/WebKit-qt-ews/Tools/Scripts/webkit-patch", line 67, in main
    from webkitpy.tool.main import WebKitPatch
  File "/storage/WebKit-qt-ews/Tools/Scripts/webkitpy/tool/main.py", line 39, in <module>
    from webkitpy.tool import commands
  File "/storage/WebKit-qt-ews/Tools/Scripts/webkitpy/tool/commands/__init__.py", line 6, in <module>
    from webkitpy.tool.commands.download import *
  File "/storage/WebKit-qt-ews/Tools/Scripts/webkitpy/tool/commands/download.py", line 32, in <module>
    from webkitpy.tool import steps
  File "/storage/WebKit-qt-ews/Tools/Scripts/webkitpy/tool/steps/__init__.py", line 32, in <module>
    from webkitpy.tool.steps.applywatchlisttopatch import ApplyWatchListToPatch
ImportError: No module named applywatchlisttopatch
Comment 1 Adam Barth 2011-09-29 09:57:30 PDT
Any thoughts on how to accomplish this?
Comment 2 David Levin 2011-09-29 10:01:02 PDT
(In reply to comment #1)
> Any thoughts on how to accomplish this?

Perhaps the ews could run from a different enlistment that the enlistment where it applies the patch, but still sync its own enlistment every so often.

Or perhaps the ews could copy itself somewhere else and run from there (and update that location -- once in a while).
Comment 3 Adam Barth 2011-09-29 10:04:31 PDT
> Perhaps the ews could run from a different enlistment that the enlistment where it applies the patch, but still sync its own enlistment every so often.

^^^ That sounds like a good approach.
Comment 4 EWS Watchlist 2019-09-03 12:39:22 PDT
We have re-written EWS, and with buildbot-based EWS, this isn't an issue anymore.
Comment 5 Radar WebKit Bug Importer 2019-09-03 12:39:59 PDT
<rdar://problem/54986401>