Bug 60345 - EWS can't reject patches
Summary: EWS can't reject patches
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: Adam Barth
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-06 00:00 PDT by Adam Barth
Modified: 2011-05-06 08:40 PDT (History)
1 user (show)

See Also:


Attachments
Patch (2.93 KB, patch)
2011-05-06 00:06 PDT, Adam Barth
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2011-05-06 00:00:09 PDT
Traceback (most recent call last):
  File "/mnt/git/webkit-chromium-ews/Tools/Scripts/webkitpy/tool/bot/queueengine.py", line 108, in run
    if not self._delegate.process_work_item(work_item):
  File "/mnt/git/webkit-chromium-ews/Tools/Scripts/webkitpy/tool/commands/queues.py", line 378, in process_work_item
    if not self.review_patch(patch):
  File "/mnt/git/webkit-chromium-ews/Tools/Scripts/webkitpy/tool/commands/earlywarningsystem.py", line 149, in review_patch
    self._post_reject_message_on_bug(self._tool, patch, task.failure_status_id, self._failing_tests_message(task, patch))
  File "/mnt/git/webkit-chromium-ews/Tools/Scripts/webkitpy/tool/commands/earlywarningsystem.py", line 109, in _post_reject_message_on_bug
    tool.bugs.set_flag_on_attachment(patch.bug_id(), "commit-queue", "-", message, extra_message_text)
  File "/mnt/git/webkit-chromium-ews/Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla.py", line 631, in set_flag_on_attachment
    self.browser.set_value(comment_text, name='comment', nr=0)
  File "/mnt/git/webkit-chromium-ews/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_form.py", line 2823, in set_value
    c = self.find_control(name, type, kind, id, label=label, nr=nr)
  File "/mnt/git/webkit-chromium-ews/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_form.py", line 3101, in find_control
    return self._find_control(name, type, kind, id, label, predicate, nr)
  File "/mnt/git/webkit-chromium-ews/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_form.py", line 3185, in _find_control
    raise ControlNotFoundError("no control matching "+description)
ControlNotFoundError: no control matching name 'comment'
Comment 1 Adam Barth 2011-05-06 00:06:29 PDT
Created attachment 92551 [details]
Patch
Comment 2 Adam Barth 2011-05-06 00:07:02 PDT
Committed r85922: <http://trac.webkit.org/changeset/85922>
Comment 3 Adam Barth 2011-05-06 00:12:53 PDT
Comment on attachment 92551 [details]
Patch

I landed this without review because I'm a problem child.
Comment 4 Eric Seidel (no email) 2011-05-06 08:40:24 PDT
Comment on attachment 92551 [details]
Patch

Lgtm