Closed
Bug 981654
Opened 10 years ago
Closed 10 years ago
Fix compression for non-compressed uploads
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: ted, Assigned: ted)
References
Details
Attachments
(1 file)
879 bytes,
patch
|
rail
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
Apparently this was broken from the start, so things like screenshots have been busted.
Assignee | ||
Comment 1•10 years ago
|
||
I fixed this on both the client and the server, so landing either fix in production will fix this: https://github.com/catlee/blobber/pull/10 https://github.com/MihaiTabara/blobuploader/pull/9
Comment 2•10 years ago
|
||
Merged the blobuploader changes too. Released 1.1.1 - available at https://pypi.python.org/pypi/blobuploader/1.1.1 It needs to get puppetized + mozharness requirement updated.
Comment 3•10 years ago
|
||
Attachment #8391865 -
Flags: review?(rail)
Comment 4•10 years ago
|
||
Comment on attachment 8391865 [details] [diff] [review] Bump blobuploader version to 1.1.1 I can deploy this on Tue/Wed.
Attachment #8391865 -
Flags: review?(rail) → review+
Comment 5•10 years ago
|
||
Comment on attachment 8391865 [details] [diff] [review] Bump blobuploader version to 1.1.1 https://hg.mozilla.org/build/mozharness/rev/ab87fc91c1f7
Attachment #8391865 -
Flags: checked-in+
Comment 6•10 years ago
|
||
Comment on attachment 8391865 [details] [diff] [review] Bump blobuploader version to 1.1.1 And merged to production: https://hg.mozilla.org/build/mozharness/rev/9876018e8c38
Comment 7•10 years ago
|
||
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #1) > I fixed this on both the client and the server, so landing either fix in > production will fix this: > https://github.com/catlee/blobber/pull/10 This one is live as well.
Comment 8•10 years ago
|
||
Rail: thanks for deplying. Ted: can we close this ?
Assignee | ||
Comment 9•10 years ago
|
||
Thanks! Yes, this should be fixed now. Needs to be verified, should be able to do that by looking at an uploaded screenshot from a test timeout.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•