981654 - Fix compression for non-compressed uploads

Status: VERIFIED FIXED

(Release Engineering :: General, defect)

Description

[(not currently active) Ted Mielczarek]

Apparently this was broken from the start, so things like screenshots have been busted.

Comment 1

[(not currently active) Ted Mielczarek]

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

[Mihai Tabara [:mtabara]⌚️GMT]

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

[Mihai Tabara [:mtabara]⌚️GMT]

Attachment: Bump blobuploader version to 1.1.1

Comment 4

[Rail Aliiev [:rail]]

Comment on attachment 8391865 [details] [diff] [review]
Bump blobuploader version to 1.1.1

I can deploy this on Tue/Wed.

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 8391865 [details] [diff] [review]
Bump blobuploader version to 1.1.1

https://hg.mozilla.org/build/mozharness/rev/ab87fc91c1f7

Comment 6

[Rail Aliiev [:rail]]

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

[Rail Aliiev [:rail]]

(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

[Mihai Tabara [:mtabara]⌚️GMT]

Rail: thanks for deplying.
Ted: can we close this ?

Comment 9

[(not currently active) Ted Mielczarek]

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.

Comment 10

[Phil Ringnalda (:philor)]

A thing which is surprisingly easy to find.