Bug 160422 - Make wpt server generate its certificates in layout test results folder
Summary: Make wpt server generate its certificates in layout test results folder
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-01 13:36 PDT by youenn fablet
Modified: 2016-08-01 14:48 PDT (History)
4 users (show)

See Also:


Attachments
Patch (10.97 KB, patch)
2016-08-01 13:38 PDT, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2016-08-01 13:36:36 PDT
This is reverting http://trac.webkit.org/changeset/202681 as bug causing that patch is now fixed upstream in wpt tools.
And it now overwrites checked-in certificates.
See bug 159166
Comment 1 youenn fablet 2016-08-01 13:38:39 PDT
Created attachment 285038 [details]
Patch
Comment 2 Chris Dumez 2016-08-01 13:41:01 PDT
Comment on attachment 285038 [details]
Patch

rs=me
Comment 3 youenn fablet 2016-08-01 13:42:39 PDT
Comment on attachment 285038 [details]
Patch

When the bots are green, feel free to cq+ the patch.
Otherwise, I'll do it tomorrow morning.
Comment 4 Chris Dumez 2016-08-01 14:07:54 PDT
Comment on attachment 285038 [details]
Patch

Clearing flags on attachment: 285038

Committed r203987: <http://trac.webkit.org/changeset/203987>
Comment 5 Chris Dumez 2016-08-01 14:07:59 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Alexey Proskuryakov 2016-08-01 14:48:34 PDT
Now that it's in the results folder, the certificates will be stored with the results on the server, which is not ideal either.

Shouldn't it just be /tmp/WebKit?