Bump file size limit up to 250 Mb

RESOLVED FIXED in 2015-04-28

Status

P3
major
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Harald, Assigned: clouserw)

Tracking

2015-04-28
Points:
---

Details

(Whiteboard: [qa-])

(Reporter)

Description

4 years ago
We pitched Marketplace at GDC and our business development team is having several Unity games in the pipelines. The first submissions are coming in that hit the 100 Mb file size limit. As the limit was added with FxOS in mind and not desktop + games need to get this out of the way asap.

I would suggest bumping the filesize to at least 200 Mb.

For further discussion: Android solves this by allowing splitting APKs where in case of unity one shared APK contains the game code and other APKs are generated with game assets in various qualities/sizes.
(Reporter)

Comment 1

4 years ago
ni? on D&W
Flags: needinfo?(wclouser)
Flags: needinfo?(dbialer)
FWIW this might not be as straightforward as just raising the limit ; the validator might also choke and time out on big archives containing a lot of code to process (and the timeout is already pretty big).

(And yeah, we should figure out something to allow splitting the packages in order not to have enormous single file uploads/downloads)
This is set in https://github.com/mozilla/zamboni/blob/master/mkt/constants/submit.py.

I don't think it's going to be JS that is causing these files to be bigger than 100MB although I could be wrong. I'm assuming it will mostly be images and videos that put bundles over the limit.

100MB is already lots of room to put a ton of JS but it's probably worth putting Unity and Unreal lib files in our safe list as I'm guessing those files are probably pretty hefty.
(Reporter)

Updated

4 years ago
Summary: Bump file size limit up to 200 Mb → Bump file size limit up to 250 Mb
(Assignee)

Comment 4

4 years ago
If we already don't validate files over 1MB I wonder if this limit is really useful to us.  It could just as easily choke on something at 100MB as it can at 250MB.  Just thinking out loud - either way, let's increase this to 250MB for now and we can talk about how to scale it in the future.
Flags: needinfo?(wclouser)
(Assignee)

Comment 5

4 years ago
https://github.com/mozilla/zamboni/commit/ca7a708a4787ac551b14dd649ff53778a9577b57

I'm marking this QA- because Harald is going to try it as soon as it lands, I imagine.
Assignee: nobody → wclouser
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Flags: needinfo?(dbialer)
Priority: -- → P3
Resolution: --- → FIXED
Whiteboard: [qa-]
Target Milestone: --- → 2015-04-07
(Reporter)

Comment 6

4 years ago
Upload worked and validation didn't fail because of archive file size but some file size limit for files within the archive which isn't even mentioned or explained: https://cloudup.com/cLB7rNTH144
Flags: needinfo?(wclouser)
(Assignee)

Comment 7

4 years ago
Interesting.  It looks like that is set at https://github.com/mozilla/zamboni/blob/master/mkt/settings.py#L706

PR @ https://github.com/mozilla/zamboni/pull/3054
Status: RESOLVED → REOPENED
Flags: needinfo?(wclouser)
Resolution: FIXED → ---
(Assignee)

Comment 8

4 years ago
https://github.com/mozilla/zamboni/commit/580d2e83969b287f88a8f235cf572008df572f7a

This is before 11 so I'd expect it to get in the tag for next Tuesday
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago4 years ago
Resolution: --- → FIXED
(Assignee)

Updated

4 years ago
Target Milestone: 2015-04-07 → 2015-04-28
You need to log in before you can comment on or make changes to this bug.