Bug 186923 – Security EWS: bots fails with exception 'NoneType' object has no attribute 'is_closed'

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Aakash Jain]

Some of the EWS bots are hitting following exception while processing security sensitive bugs:

2018-06-21 18:41:22,834 - You don't have permission to view this attachment.
Traceback (most recent call last):
  File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/tool/bot/queueengine.py", line 103, in run
    if not self._delegate.process_work_item(work_item):
  File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/tool/commands/queues.py", line 465, in process_work_item
    passed = self.review_patch(patch)
  File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/tool/commands/earlywarningsystem.py", line 105, in review_patch
    succeeded = task.run()
  File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/tool/bot/earlywarningsystemtask.py", line 63, in run
    if not self._clean():
  File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/tool/bot/patchanalysistask.py", line 113, in _clean
    "Unable to clean working directory")
  File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/tool/bot/patchanalysistask.py", line 97, in _run_command
    if not self.validate():
  File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/tool/bot/earlywarningsystemtask.py", line 48, in validate
    if self._patch.bug().is_closed():
AttributeError: 'NoneType' object has no attribute 'is_closed'
2018-06-21 18:41:23,296 - Exception while preparing queue Sleeping until 2018-06-21 18:43:23 (120 seconds).

[Daniel Bates]

This cause of this bug is due to the fact that attachments downloaded from the status server do not have an associated Bug object (i.e. Attachment.bug() returns None). They do not have an associated Bug object because these attachments were not fetched from Bugzilla (since the EWS bot does not have access to the security-sensitive bug the attachment is actually hosted on). We need to fix bug #186817 and provide a way for the EWS bots to fetch up-to-date state of both the attachment and the associated bug.

[Daniel Bates]

Created attachment 343382 [details]
Patch

[Daniel Bates]

Comment on attachment 343382 [details]
Patch

Clearing flags on attachment: 343382

Committed r233107: <https://trac.webkit.org/changeset/233107>

[Daniel Bates]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/41385843>