WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
199422
[ews-build] Allow skipping uploading built product for few builders
https://bugs.webkit.org/show_bug.cgi?id=199422
Summary
[ews-build] Allow skipping uploading built product for few builders
Aakash Jain
Reported
2019-07-02 14:37:08 PDT
We don't need to upload built-product for all the builder-only queues (which don't trigger any tester). For e.g.: WPE/GTK folks don't need the uploaded built-product (as per email conversation). This process also adds extra time in the build (as we need to run archive-built-product, upload-built-product and transfer-to-s3 step), and the upload might take long time depending on the physical location of the bot and network conditions. We should allow the option to skip the upload.
Attachments
Patch
(2.03 KB, patch)
2019-07-02 14:39 PDT
,
Aakash Jain
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Aakash Jain
Comment 1
2019-07-02 14:39:08 PDT
Created
attachment 373362
[details]
Patch
Aakash Jain
Comment 2
2019-07-02 14:39:48 PDT
Sample run:
https://ews-build.webkit-uat.org/#/builders/7/builds/444
EWS Watchlist
Comment 3
2019-07-02 14:41:06 PDT
Comment hidden (obsolete)
Attachment 373362
[details]
did not pass style-queue: ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:622: [CompileWebKit.__init__] Use of super on an old style class [pylint/E1002] [5] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
WebKit Commit Bot
Comment 4
2019-07-02 16:03:17 PDT
Comment on
attachment 373362
[details]
Patch Clearing flags on attachment: 373362 Committed
r247075
: <
https://trac.webkit.org/changeset/247075
>
WebKit Commit Bot
Comment 5
2019-07-02 16:03:18 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6
2019-07-02 16:04:18 PDT
<
rdar://problem/52553282
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug