Bug 141509 - run-jsc-stress-tests should accept remote host config from JSON file
Summary: run-jsc-stress-tests should accept remote host config from JSON file
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Csaba Osztrogonác
URL:
Keywords:
: 141082 (view as bug list)
Depends on:
Blocks: 141165
  Show dependency tree / graph
 
Reported: 2015-02-12 07:06 PST by Csaba Osztrogonác
Modified: 2015-02-17 04:04 PST (History)
1 user (show)

See Also:

Attachments
Patch (5.72 KB, patch)
2015-02-12 07:11 PST, Csaba Osztrogonác 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2015-02-12 07:06:22 PST 
Now we have to pass --remote=user@host[:port] option directly in command 
line, which isn't so good to configure a public buildbot (bug141165)

Additionally the temp directory on the remote host can be configured 
with a JSON file lies on the remote host in ~/.bencher . It would be
good if we can specify this without modifying anything on the remote.

I'd like to add a new option to be able pass a JSON file with remote 
and remoteDirectory values and don't want to break the iOS(?) testing
which still(?) relies the ~/.bencher file on the remote side.
Comment 1 Csaba Osztrogonác 2015-02-12 07:11:15 PST 
Created attachment 246441 [details]
Patch
Comment 2 Michael Saboff 2015-02-12 07:23:50 PST 
Comment on attachment 246441 [details]
Patch

r=me
Comment 3 WebKit Commit Bot 2015-02-12 08:08:11 PST 
Comment on attachment 246441 [details]
Patch

Clearing flags on attachment: 246441

Committed r179986: <http://trac.webkit.org/changeset/179986>
Comment 4 WebKit Commit Bot 2015-02-12 08:08:15 PST 
All reviewed patches have been landed.  Closing bug.
Comment 5 Csaba Osztrogonác 2015-02-17 04:04:05 PST 
*** Bug 141082 has been marked as a duplicate of this bug. ***