https://bugs.webkit.org/show_bug.cgi?id=197914 add support to transfer to S3 for EWS. We should add build step to start using that.
Created attachment 369983 [details] Patch
Attachment 369983 [details] did not pass style-queue: ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:772: [UploadBuiltProduct.getResultSummary] Use of super on an old style class [pylint/E1002] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:773: [UploadBuiltProduct.getResultSummary] Instance of 'UploadBuiltProduct' has no 'results' member [pylint/E1101] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1102: [TestTransferToS3.test_success] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1102: [TestTransferToS3.test_success] No value passed for parameter 'status_text' in function call [pylint/E1120] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1121: [TestTransferToS3.test_failure] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1121: [TestTransferToS3.test_failure] No value passed for parameter 'status_text' in function call [pylint/E1120] [5] Total errors found: 6 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 369983 [details] Patch Patch needs to be updated after changes to <https://bugs.webkit.org/show_bug.cgi?id=197914>
> Patch needs to be updated after changes to <https://bugs.webkit.org/show_bug.cgi?id=197914> This patch is still good. We still have the --patch_id parameter in https://bugs.webkit.org/show_bug.cgi?id=197914
actually need to remove --ews parameter.
Created attachment 369998 [details] Patch
Attachment 369998 [details] did not pass style-queue: ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:772: [UploadBuiltProduct.getResultSummary] Use of super on an old style class [pylint/E1002] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:773: [UploadBuiltProduct.getResultSummary] Instance of 'UploadBuiltProduct' has no 'results' member [pylint/E1101] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1101: [TestTransferToS3.test_success] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1101: [TestTransferToS3.test_success] No value passed for parameter 'status_text' in function call [pylint/E1120] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1119: [TestTransferToS3.test_failure] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1119: [TestTransferToS3.test_failure] No value passed for parameter 'status_text' in function call [pylint/E1120] [5] Total errors found: 6 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 370000 [details] Patch
Attachment 370000 [details] did not pass style-queue: ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:772: [UploadBuiltProduct.getResultSummary] Use of super on an old style class [pylint/E1002] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:773: [UploadBuiltProduct.getResultSummary] Instance of 'UploadBuiltProduct' has no 'results' member [pylint/E1101] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1101: [TestTransferToS3.test_success] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1101: [TestTransferToS3.test_success] No value passed for parameter 'status_text' in function call [pylint/E1120] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1119: [TestTransferToS3.test_failure] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:1119: [TestTransferToS3.test_failure] No value passed for parameter 'status_text' in function call [pylint/E1120] [5] Total errors found: 6 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 370000 [details] Patch Rejecting attachment 370000 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 370000, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Logging in as commit-queue@webkit.org... Fetching: https://bugs.webkit.org/attachment.cgi?id=370000&action=edit Fetching: https://bugs.webkit.org/show_bug.cgi?id=197922&ctype=xml&excludefield=attachmentdata Processing 1 patch from 1 bug. Updating working directory Processing patch 370000 from bug 197922. Fetching: https://bugs.webkit.org/attachment.cgi?id=370000 Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Committing to http://svn.webkit.org/repository/webkit/trunk ... M Tools/BuildSlaveSupport/ews-build/steps.py ERROR from SVN: Item is out of date: File '/trunk/Tools/BuildSlaveSupport/ews-build/steps.py' is out of date W: 7c96ca51f33c853306d8b8fb6b59e32aede4d6f8 and refs/remotes/origin/master differ, using rebase: :040000 040000 7b09d0ff72efd591203a5dfe5afd5c113d0be4f4 4c70e70207a9507ed8b9edbe23cc3573d89dd666 M Tools Current branch master is up to date. ERROR: Not all changes have been committed into SVN, however the committed ones (if any) seem to be successfully integrated into the working tree. Please see the above messages for details. Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Committing to http://svn.webkit.org/repository/webkit/trunk ... M Tools/BuildSlaveSupport/ews-build/steps.py ERROR from SVN: Item is out of date: File '/trunk/Tools/BuildSlaveSupport/ews-build/steps.py' is out of date W: 7c96ca51f33c853306d8b8fb6b59e32aede4d6f8 and refs/remotes/origin/master differ, using rebase: :040000 040000 7b09d0ff72efd591203a5dfe5afd5c113d0be4f4 4c70e70207a9507ed8b9edbe23cc3573d89dd666 M Tools Current branch master is up to date. ERROR: Not all changes have been committed into SVN, however the committed ones (if any) seem to be successfully integrated into the working tree. Please see the above messages for details. Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Updating OpenSource Current branch master is up to date. Full output: https://webkit-queues.webkit.org/results/12212519
Comment on attachment 370000 [details] Patch Clearing flags on attachment: 370000 Committed r245461: <https://trac.webkit.org/changeset/245461>
All reviewed patches have been landed. Closing bug.
<rdar://problem/50892140>