Bug 190191 - [JSCOnly Add an armv7 JSCOnly EWS that runs tests
Summary: [JSCOnly Add an armv7 JSCOnly EWS that runs tests
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
: 190189 (view as bug list)
Depends on: 190190
Blocks:
  Show dependency treegraph
 
Reported: 2018-10-02 01:05 PDT by Guillaume Emont
Modified: 2018-10-17 08:52 PDT (History)
10 users (show)

See Also:


Attachments
Patch (3.54 KB, patch)
2018-10-04 09:46 PDT, Guillaume Emont
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Guillaume Emont 2018-10-02 01:05:57 PDT
We need to have an EWS for armv7 that runs the jsc tests.

For this to happen, we will need to solve these two issues first:
 - #190189 [JSCOnly] Add an armv7 JSCOnly build-only EWS
 - #190190 Allow run-javascriptcore-tests to distribute tests between devices
Comment 1 Guillaume Emont 2018-10-04 09:46:12 PDT
Created attachment 351597 [details]
Patch

WIP patch for early review, not to be committed as I am still working on preparing the bot.
Comment 2 Guillaume Emont 2018-10-04 09:46:38 PDT
(In reply to Guillaume Emont from comment #1)
> Created attachment 351597 [details]
> Patch
> 
> WIP patch for early review, not to be committed as I am still working on
> preparing the bot.

Also, this will need #190190 to be fixed to have a chance to work.
Comment 3 Guillaume Emont 2018-10-10 06:19:05 PDT
*** Bug 190189 has been marked as a duplicate of this bug. ***
Comment 4 WebKit Commit Bot 2018-10-10 06:33:10 PDT
Comment on attachment 351597 [details]
Patch

Clearing flags on attachment: 351597

Committed r237001: <https://trac.webkit.org/changeset/237001>
Comment 5 WebKit Commit Bot 2018-10-10 06:33:12 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2018-10-10 06:34:26 PDT
<rdar://problem/45158858>
Comment 7 Guillaume Emont 2018-10-10 07:27:08 PDT
Reading a little late that I was supposed[1] to add people to the bug ...oops! Does either of you need to do some manual operation to enable the bot, or did I forget something in the patch? The ews seems to be running on the builder, but never gets patches to process:

2018-10-10 07:23:28,109 - Fetching next work item for jsc-armv7-ews
2018-10-10 07:23:28,838 - No work item. Sleeping until 2018-10-10 07:25:28 (120 seconds).


Meanwhile this EWS does not appear on https://webkit-queues.webkit.org/ or on test bubbles (tried with Bug 190428 ).

[1] https://trac.webkit.org/wiki/EWS
Comment 8 Lucas Forschler 2018-10-11 21:40:30 PDT
Aakash, do you need to restart the ews server?
Comment 9 Aakash Jain 2018-10-12 09:27:43 PDT
(In reply to Lucas Forschler from comment #8)
> Aakash, do you need to restart the ews server?

Yes, Restarted it. The change is now live: https://webkit-queues.webkit.org/queue-status/jsc-armv7-ews
Comment 10 Guillaume Emont 2018-10-12 09:51:21 PDT
New issue (log from bot):

2018-10-12 09:39:26,014 - Running: webkit-patch --status-host=webkit-queues.webkit.org --bot-id=igalia-jsc-armv7-ews apply-attachment --no-update --non-interac
tive 352173 --port=jsc-only
2018-10-12 09:47:42,032 - Failed to run "['/home/buildbot/ews/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=igalia-js
c-armv7-ews', 'apply-attachment', '--no-update', '--non-interactive', 352173, '--port=jsc-only']" exit_code: 1 cwd: /home/buildbot/ews/WebKit

No handlers could be found for logger "webkitpy.common.system.user"
Logging in as guijemont+jsc-armv7-ews@igalia.com...
Received URLError: "unknown url type: https" while loading https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1. Retrying in 10 seconds...
Received URLError: "unknown url type: https" while loading https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1. Retrying in 15.0 seconds...
Received URLError: "unknown url type: https" while loading https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1. Retrying in 22.5 seconds...
Received URLError: "unknown url type: https" while loading https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1. Retrying in 33.75 seconds...
Received URLError: "unknown url type: https" while loading https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1. Retrying in 50.625 seconds...
Received URLError: "unknown url type: https" while loading https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1. Retrying in 75.9375 seconds...
Received URLError: "unknown url type: https" while loading https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1. Retrying in 113.90625 seconds...
Received URLError: "unknown url type: https" while loading https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1. Retrying in 170.859375 seconds...
Traceback (most recent call last):
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkit-patch", line 84, in <module>
    main()
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkit-patch", line 79, in main
    WebKitPatch(os.path.abspath(__file__)).main()
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 305, in main
    result = command.check_arguments_and_execute(options, args, self)
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 123, in check_arguments_and_execute
    return self.execute(options, args, tool) or 0
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/tool/commands/download.py", line 180, in execute
    patches = self._fetch_list_of_patches_to_process(options, args, tool)
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/tool/commands/download.py", line 224, in _fetch_list_of_patches_to_process
    patch = tool.bugs.fetch_attachment(patch_id, throw_on_access_error=True)
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla.py", line 552, in fetch_attachment
    bug_id = self.bug_id_for_attachment_id(attachment_id, throw_on_access_error)
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla.py", line 530, in bug_id_for_attachment_id
    return NetworkTransaction().run(lambda: self.get_bug_id_for_attachment_id(attachment_id, throw_on_access_error))
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/common/net/networktransaction.py", line 53, in run
    return request()
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla.py", line 530, in <lambda>
    return NetworkTransaction().run(lambda: self.get_bug_id_for_attachment_id(attachment_id, throw_on_access_error))
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla.py", line 533, in get_bug_id_for_attachment_id
    self.authenticate()
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla.py", line 576, in authenticate
    "index.cgi?GoAheadAndLogIn=1")
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla.py", line 335, in open_url
    return NetworkTransaction().run(lambda: self.browser.open(url), url)
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/common/net/networktransaction.py", line 61, in run
    self._check_for_timeout()
  File "/home/buildbot/ews/WebKit/Tools/Scripts/webkitpy/common/net/networktransaction.py", line 67, in _check_for_timeout
    raise NetworkTimeout()
webkitpy.common.net.networktransaction.NetworkTimeout: NetworkTimeout

2018-10-12 09:47:42,033 - Patch does not apply
2018-10-12 09:47:43,757 - Error: jsc-armv7-ews unable to apply patch.
2018-10-12 09:47:45,175 - Releasing work item 352173 from jsc-armv7-ews
Comment 11 Guillaume Emont 2018-10-12 09:52:18 PDT
(In reply to Guillaume Emont from comment #10)
> New issue (log from bot):

Actually visible in https://webkit-queues.webkit.org/results/9552471
Comment 12 jsc-armv7 EWS 2018-10-12 10:03:38 PDT
Double checked, and the credentials in .git/config seem correct for the account guijemont+jsc-armv7-ews@igalia.com
Comment 13 Aakash Jain 2018-10-12 10:07:28 PDT
> Actually visible in https://webkit-queues.webkit.org/results/9552471

Need to debug. Might be related to: https://bugs.webkit.org/show_bug.cgi?id=187061 

Also, please make sure that this bot(igalia-jsc-armv7-ews) can actually communicate with Bugzilla? (ping, curl etc.)
Comment 14 jsc-armv7 EWS 2018-10-12 10:13:19 PDT
(In reply to Aakash Jain from comment #13)
> > Actually visible in https://webkit-queues.webkit.org/results/9552471
> 
> Need to debug. Might be related to:
> https://bugs.webkit.org/show_bug.cgi?id=187061 
> 
> Also, please make sure that this bot(igalia-jsc-armv7-ews) can actually
> communicate with Bugzilla? (ping, curl etc.)

Yes, I can wget https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1 from the host no problem.
Looking at this bug and if there is any issue with our python install.
Comment 15 jsc-armv7 EWS 2018-10-12 10:38:10 PDT
(In reply to Guillaume Emont (jsc-armv7-ews) from comment #14)
Manually running this works fine on the host FWIW:

urllib2.urlopen("https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1").read()
Comment 16 jsc-armv7 EWS 2018-10-12 10:42:27 PDT
(In reply to Guillaume Emont (jsc-armv7-ews) from comment #15)
> (In reply to Guillaume Emont (jsc-armv7-ews) from comment #14)
> Manually running this works fine on the host FWIW:
> 
> urllib2.urlopen("https://bugs.webkit.org/index.cgi?GoAheadAndLogIn=1").read()

Oops. We have another python in the PATH, that is part of the buildroot "host" image used to generate the toolchain. This python indeed fails when trying the above, so I guess the script ends up using the wrong python (it is before in the path). I'll see what I can do about that.
Comment 17 jsc-armv7 EWS 2018-10-12 11:43:44 PDT
Changing the order of things in PATH seems to have fixed the python issue. Now to the next issue: https://webkit-queues.webkit.org/results/9553802
Comment 18 Aakash Jain 2018-10-12 11:58:58 PDT
(In reply to Guillaume Emont (jsc-armv7-ews) from comment #17)
> Changing the order of things in PATH seems to have fixed the python issue.
> Now to the next issue: https://webkit-queues.webkit.org/results/9553802
Can you manually run this command on the bot and see if that works?

['/home/buildbot/ews/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=igalia-jsc-armv7-ews', 'build-and-test', '--no-clean', '--no-update', '--test', '--non-interactive', '--build-style=release', u'--group=jsc', '--port=jsc-only']


Also please run without 'u' to check if the unicode in u'--group=jsc' is causing any issues.
Comment 19 jsc-armv7 EWS 2018-10-12 12:00:52 PDT
manually added --jsc-only as a parameter in the run script, though I suspect I shouldn't have to. It is now building its first patch.
Comment 20 jsc-armv7 EWS 2018-10-12 14:20:13 PDT
(In reply to Guillaume Emont (jsc-armv7-ews) from comment #19)
> manually added --jsc-only as a parameter in the run script, though I suspect
> I shouldn't have to. It is now building its first patch.

And the bug we have now is that my dealing of JSCTESTS_OPTIONS is not correct, I should split the arguments into a list. Patch upcoming
Comment 21 Guillaume Emont 2018-10-12 17:50:02 PDT
(In reply to Guillaume Emont (jsc-armv7-ews) from comment #20)
> (In reply to Guillaume Emont (jsc-armv7-ews) from comment #19)
> > manually added --jsc-only as a parameter in the run script, though I suspect
> > I shouldn't have to. It is now building its first patch.
> 
> And the bug we have now is that my dealing of JSCTESTS_OPTIONS is not
> correct, I should split the arguments into a list. Patch upcoming

Patch went to Bug 190537. Now I need to understand the weird build errors that I can't reproduce outside of the EWS, such as https://webkit-queues.webkit.org/results/9559484
Comment 22 Guillaume Emont 2018-10-17 08:52:28 PDT
EWS seems to be running fine now. I might have found an issue (Bug 190662) while debugging it though.