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

5Gb limit #617

Open
1 task done
Tracked by #950 ...
esraneufeld opened this issue Apr 7, 2022 · 13 comments · Fixed by ITISFoundation/osparc-simcore-clients#63
Open
1 task done
Tracked by #950 ...

5Gb limit #617

esraneufeld opened this issue Apr 7, 2022 · 13 comments · Fixed by ITISFoundation/osparc-simcore-clients#63
Assignees
Labels
Epic Zenhub label (Pleas do not modify) PO issue Created by Product owners

Comments

@esraneufeld
Copy link
Member

esraneufeld commented Apr 7, 2022

Problem:

AWS S3 has a 5 GB limit for a sinlge put operation. The single pre-signed link strategy fails. This affects:

  • Uploads through filepicker
  • Uploads through nodeports
  • Uploads from comp. backend

Remedy:

  • rclone instad of nodeports?
  • temporary S3 credentials for uplaods into specific subfolder of the bucket?
  • multiple upload links?

Baklava

  1. 1 of 1
    a:dask-service
    sanderegg
@esraneufeld esraneufeld added the PO issue Created by Product owners label Apr 7, 2022
@newton1985
Copy link

Does "uploads from comp. backend" also cover the case of saving work.zip to S3 when a study is closed? Afaik, this is also affected.

@sanderegg sanderegg added the Epic Zenhub label (Pleas do not modify) label Apr 22, 2022
@sanderegg
Copy link
Member

sanderegg commented Apr 29, 2022

Update on sprint Macarons

Done

Ongoing

Open

  • Creation of temporary S3 credentials per user (limited to 1000 concurrent temp credentials by AWS)
  • Investigate multipart upload with presigned links (for frontend, and potentially for computational sidecars in external clusters)
  • Implement solution for frontend/external computational sidecars

@sanderegg
Copy link
Member

sanderegg commented Jun 1, 2022

Update on sprint Croissant

Done

Ongoing

Implement multipart upload links to break 5Gb limit:

Open

  • Implement solution for frontend/external computational sidecars
  • Implement solution for public API

@drniiken drniiken closed this as completed Jul 7, 2022
@pcrespov pcrespov reopened this Jul 12, 2022
@esraneufeld
Copy link
Member Author

todo

  • fix bug in API that limits to 2GB

@mrnicegyu11 mrnicegyu11 assigned pcrespov and unassigned GitHK Aug 5, 2022
@pcrespov
Copy link
Member

pcrespov commented Aug 29, 2022

Update on sprint Brutalism

No progress this sprint on this issue

Open

@esraneufeld
Copy link
Member Author

@sanderegg
dear sylvain. is there an update on this one?

@sanderegg
Copy link
Member

@sanderegg dear sylvain. is there an update on this one?

@esraneufeld no there isn't we decided in PM1 to concentrate on other priorities, since this one has a workaround and that we concentrate on dynamic services at the moment.

@newton1985
Copy link

@esraneufeld - just to add some detail, the workaround is to use the "curl" command's "put" feature. It's a substitution of 1 or 2 lines of code, and seems to work just fine without any drawbacks. Though at some point it would be nice to fix it, since the average user won't know the trick.

@matusdrobuliak66
Copy link
Contributor

@matusdrobuliak66 Investigate options AWS temporary credientials for S3

@sanderegg
Copy link
Member

Current status is that presigned link are created with a very long valid time. this is not optimal. Best would be to use the osparc API directly, therefore we would not rely on a hard-coded time value. it would also be more secure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic Zenhub label (Pleas do not modify) PO issue Created by Product owners
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants