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.

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

Ingress controller does not allow uploading files bigger than 5MB #35

Closed
miwurster opened this issue Sep 22, 2020 · 7 comments
Closed

Comments

@miwurster
Copy link
Contributor

Currently it seems that the configured Ingress controller of RADON's Kubernetes cluster does not allow to upload files bigger than X megabytes (tested it with a file of 8MB).

The Ingress layer returns a "413 Request Entity Too Large" error depicted in the following screenshot:
image

We may have to configure the underlying nginx configuration to solve this. See here: kubernetes/ingress-nginx#4825 (comment)

@miwurster
Copy link
Contributor Author

@sdagostini any update on this?
IMHO we should work on this till the final release.

@sdagostini
Copy link
Collaborator

@miwurster Hi, I don't have updates yet but yes, the idea is to solve it for the final release. I will update you on that issue as soon as possible.

@sdagostini
Copy link
Collaborator

@miwurster Hi, just to let you know that I have updated the underlying nginx configuration of the "che" ingress to allow uploading files up to 16MB but the issue is not been solved. This is the motivation: when a new RADON workspace is created a new ingress is created for the GMT component (e.g. servernjp9g1pa-winery-deployment-winery-server-8080.217.172.12.178.nip.io). In order to fix the issue the configuration of the created ingress should be updated (i.e. for each new GMT instance created for each workspace). Therefore, if there is no way to increse the file size when the GMT kubernetes container is created I think that we have to limit the size to 5MB

@miwurster
Copy link
Contributor Author

Have you tried to find some information if some default value can be set for the ingress. To my knowledge, the ingress is created behind the scene by che. So you cannot influence it from the devfile. Nevertheless, if you find some information on how to this from the devfile, please forward it.

@sdagostini
Copy link
Collaborator

@miwurster Hi, I have updated the configMap for the nginx ingress controller to allow uploading files up to 16MB. Now it works. Please, let me know if we need to increse more the size otherwise I will close the issue

@miwurster
Copy link
Contributor Author

Nice! I would suggest going for 100MB ;-) Then we can close this!

@sdagostini
Copy link
Collaborator

The size has been increased to 100MB.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants