Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Increase upload file size to 16G by default #5611

Open
szaimen opened this issue Nov 19, 2024 · 6 comments · May be fixed by #5613
Open

Increase upload file size to 16G by default #5611

szaimen opened this issue Nov 19, 2024 · 6 comments · May be fixed by #5613
Assignees
@szaimen
Labels
2. developing Work in progress enhancement New feature or request
Milestone
next

Comments

@szaimen
Copy link
Collaborator

szaimen commented Nov 19, 2024

See nextcloud/server#49372
@szaimen szaimen added 1. to develop Accepted and waiting to be taken care of blocked upstream labels Nov 19, 2024
@skjnldsv
Copy link
Member

skjnldsv commented Nov 19, 2024 •
edited
Loading

It's not a nc31 recommendation, it's been like that for years.

@szaimen
Copy link
Collaborator Author

szaimen commented Nov 19, 2024

It's not a nc31 recommendation, it's been like that for years.

The default in aio is currently set to 10G so need to adjust it for nc31...

@skjnldsv
Copy link
Member

skjnldsv commented Nov 19, 2024 •
edited
Loading

Then change the default to 16 for 28+ ? 😁

@szaimen
Copy link
Collaborator Author

szaimen commented Nov 19, 2024

Then change the default to 16 for 28+ ? 😁

Why that? Isnt the setupcheck a new feature and thud only for 31?

@skjnldsv
Copy link
Member

I'm planning to backport it.
But it doesn't matter, it's better to follow our own guidelines, the 16G was there before I added the setup check.

@szaimen szaimen changed the title Increase upload file size to 16G by default with nc31 Increase upload file size to 16G by default Nov 19, 2024
@szaimen szaimen linked a pull request Nov 19, 2024 that will close this issue
Draft
@szaimen
Copy link
Collaborator Author

szaimen commented Nov 19, 2024

I'm planning to backport it. But it doesn't matter, it's better to follow our own guidelines, the 16G was there before I added the setup check.

I didnt know that 16G is our recommended default until today. So fine by me to adjust it even before nc31 ;)
See #5613

@szaimen szaimen added 2. developing Work in progress enhancement New feature or request and removed 1. to develop Accepted and waiting to be taken care of upstream blocked labels Nov 19, 2024
@szaimen szaimen self-assigned this Nov 19, 2024
@szaimen szaimen added this to the next milestone Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@szaimen szaimen

Labels
2. developing Work in progress enhancement New feature or request
Projects
None yet
Milestone
next
Development

Successfully merging a pull request may close this issue.

change default upload limit to 16G as recommended in the docs nextcloud/all-in-one
2 participants
@skjnldsv @szaimen