Bug 255340 - [ews] Add a script to upload a file to given url
Summary: [ews] Add a script to upload a file to given url
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Aakash Jain
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-04-12 07:32 PDT by Aakash Jain
Modified: 2023-05-24 09:35 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 2023-04-12 07:32:33 PDT
Write a script to upload a file to given url. 
This would be used by https://bugs.webkit.org/show_bug.cgi?id=255339 "EWS bots should upload directly to S3"
Comment 1 Aakash Jain 2023-04-12 07:36:10 PDT
Pull request: https://github.com/WebKit/WebKit/pull/12650
Comment 2 EWS 2023-04-12 14:56:55 PDT
Committed 262888@main (ab2b21fb6389): <https://commits.webkit.org/262888@main>

Reviewed commits have been landed. Closing PR #12650 and removing active labels.
Comment 3 Radar WebKit Bug Importer 2023-04-12 14:57:19 PDT
<rdar://problem/107961097>
Comment 4 EWS 2023-04-25 14:49:13 PDT
Committed 259548.688@safari-7615-branch (0500e7adcfa5): <https://commits.webkit.org/259548.688@safari-7615-branch>

Reviewed commits have been landed. Closing PR #559 and removing active labels.
Comment 5 Aakash Jain 2023-04-27 04:20:30 PDT
Re-opening for pull request https://github.com/apple/WebKit/pull/559
Comment 6 Claudio Saavedra 2023-05-24 09:02:25 PDT
From https://bugs.webkit.org/show_bug.cgi?id=255819, IIUC there's intention to remove usage of the requests module. Noticed some warnings in the GTK bots because requests is used (which doesn't seem to be installed) (https://ews-build.webkit.org/#/builders/2/builds/9500). Might be a good idea to update the script ot move away from it.
Comment 7 Aakash Jain 2023-05-24 09:35:46 PDT
We don't have concern with request module as such. The reason for the change in https://bugs.webkit.org/show_bug.cgi?id=255819 is for the code running on server side to convert from synchronous to async code.

If you notice error due to missing requests module on any of your bots, please install requests module.