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

Storage cannot handle file links in when saving study to template #604

Closed
elisabettai opened this issue Mar 4, 2022 · 4 comments
Closed
Assignees
Labels
Feedback Feedback through frontend type:bug Issue that prevents to perform a certain task, features that don't work as t
Milestone

Comments

@elisabettai
Copy link
Contributor

Long Story Short

I tried to save a Study to Template. With the option "Save with data" saving doesn't work

Steps to reproduce

Showed the steps to @mrnicegyu11

Environment

Firefox, osparc.speag.com

@elisabettai elisabettai added type:bug Issue that prevents to perform a certain task, features that don't work as t Feedback Feedback through frontend labels Mar 4, 2022
@mrnicegyu11
Copy link
Member

More Information: This fails because the study uses a filelink. The storage service throws the following error when creating the template:

ERROR:servicelib.aiohttp.rest_middlewares:Unexpected server error "<class 'KeyError'>" from access: 10.11.3.87 "POST /v0/simcore-s3/folders". Responding with status 500
Traceback (most recent call last):
  File "/home/scu/.venv/lib/python3.8/site-packages/servicelib/aiohttp/rest_middlewares.py", line 75, in _middleware_handler
    response = await handler(request)
  File "/home/scu/.venv/lib/python3.8/site-packages/servicelib/aiohttp/rest_middlewares.py", line 203, in _middleware_handler
    resp: _ResponseOrBodyData = await handler(request)
  File "/home/scu/.venv/lib/python3.8/site-packages/simcore_service_storage/handlers.py", line 401, in create_folders_from_project
    await dsm.deep_copy_project_simcore_s3(
  File "/home/scu/.venv/lib/python3.8/site-packages/simcore_service_storage/dsm.py", line 831, in deep_copy_project_simcore_s3
    source = output["path"]
KeyError: 'path'

The filelink should be copied as-is into the template.

@mrnicegyu11 mrnicegyu11 changed the title Saving study to template with File Picker input as link doesn't work Storage cannot handle file links in when saving study to template Sep 30, 2022
@mrnicegyu11
Copy link
Member

Formally known as: Saving study to template with File Picker input as link doesn't work

@mrnicegyu11 mrnicegyu11 removed their assignment Sep 30, 2022
@sanderegg
Copy link
Member

@elisabettai , @mrnicegyu11 this is not reproducible on master deployment. I would wander in saying this was fixed with the storage refactoring (or the duplicate copy fixes). Please confirm and I think you could close it.

@elisabettai
Copy link
Contributor Author

@sanderegg, you're right. It now works in master, I'll close the issue.

Just noticed a very small thing: the "Download" button in the FilePicker doesn't work with a file link (which I suppose is valid). But this more for @odeimaiz, I guess.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feedback Feedback through frontend type:bug Issue that prevents to perform a certain task, features that don't work as t
Projects
None yet
Development

No branches or pull requests

3 participants