Skip to content
This repository has been archived by the owner on Jul 29, 2022. It is now read-only.

can no longer download a published asset #112

Closed
yarikoptic opened this issue Aug 1, 2020 · 11 comments
Closed

can no longer download a published asset #112

yarikoptic opened this issue Aug 1, 2020 · 11 comments
Assignees

Comments

@yarikoptic
Copy link
Member

Try downloading e.g. via web ui or via request the single asset on
https://deploy-preview-341--gui-dandiarchive-org.netlify.app/#/dandiset/000027/0.200721.2222/files?location=%2Fsub-RAT123%2F

you would get smth like

<?xml version="1.0" encoding="UTF-8"?>
<Error><Code>AuthorizationQueryParametersError</Code><Message>Error parsing the X-Amz-Credential parameter; the region 'us-east-2' is wrong; expecting 'us-east-1'</Message><Region>us-east-1</Region><RequestId>59B164ED0B5A6C55</RequestId><HostId>GR3DZC8O8s2i5ZND81JD3joTSCSAxTgthXZdWkhlM1zyO536SdTREuEtZEP6LxQhSO/ebWWFh6o=</HostId></Error>%  

I would recommend to use this small dataset in a unit/smoke test on dandi-publish or wherever it is pertinent.

@brianhelba
Copy link
Contributor

This is likely caused by some infrastructure work I've been doing on the deployment. It's not the software itself. I should be able to get it fixed soon.

@yarikoptic
Copy link
Member Author

Any ETA so we could proceed with testing?

@mgrauer
Copy link
Contributor

mgrauer commented Aug 3, 2020

We hope to have this fixed by tomorrow.

@brianhelba
Copy link
Contributor

I've identified the issue. The fix requires copying the existing data across regions, so it should be complete in 1-2 hours.

I'll post again soon when this is fully resolved.

yarikoptic added a commit to dandi/dandi-cli that referenced this issue Aug 5, 2020
Download of released datasets is broken, independent issue:
dandi/dandi-publish#112
@yarikoptic
Copy link
Member Author

Thank you @brianhelba ! how did data migrate across regions? or it just was "published" originally into the wrong region?

@brianhelba
Copy link
Contributor

Our bucket for testing publish functionality was created in the wrong region(us-east-1, instead of us-east-2). Previously, it worked, because the rest of our infrastructure was also in the wrong region, but we recently re-provisioned most of the other infrastructure in the correct region.

In principle, it is possible to configure our infrastructure to talk to buckets in other regions, but it's less configuration burden and closer to the final setup if we just put everything in us-east-2.

@brianhelba
Copy link
Contributor

Also, the copy is still proceeding, about 65% complete.

@yarikoptic
Copy link
Member Author

yarikoptic commented Aug 6, 2020

Aren't we publishing into the same bucket as used for the drafts, just a different "folder" (prefix/)?

@brianhelba
Copy link
Contributor

Aren't we publishing into the same bucket as used for the drafts, just a different "folder" (prefix/)?

Not yet, but that change is part of the recent infrastructure updates.

@brianhelba
Copy link
Contributor

This is now resolved.

@yarikoptic
Copy link
Member Author

confirming - tests of dandi download (against main deployment) pass again:

(git)lena:~/proj/dandi/dandi-cli[enh-download-api]git
$> python -m pytest -v dandi/tests/test_download.py                                                                                           
======================================================================================================= test session starts =======================================================================================================
platform linux -- Python 3.8.5, pytest-4.6.9, py-1.8.1, pluggy-0.13.0 -- /home/yoh/proj/dandi/dandi-cli/venvs/dev3/bin/python
cachedir: .pytest_cache
hypothesis profile 'default' -> database=DirectoryBasedExampleDatabase('/home/yoh/proj/dandi/dandi-cli/.hypothesis/examples')
rootdir: /home/yoh/proj/dandi/dandi-cli, inifile: tox.ini
plugins: xdist-1.31.0, timeout-1.3.3, forked-1.1.3, hypothesis-4.36.2
collected 5 items                                                                                                                                                                                                                 

dandi/tests/test_download.py::test_parse_dandi_url PASSED                                                                                                                                                                   [ 20%]
dandi/tests/test_download.py::test_parse_dandi_url_redirect PASSED                                                                                                                                                          [ 40%]
dandi/tests/test_download.py::test_download_multiple_files PASSED                                                                                                                                                           [ 60%]
dandi/tests/test_download.py::test_download_000027[https://deploy-preview-341--gui-dandiarchive-org.netlify.app/#/dandiset/000027/0.200721.2222] PASSED                                                                     [ 80%]
dandi/tests/test_download.py::test_download_000027[https://gui.dandiarchive.org/#/dandiset/5f0640a2ab90ac46c4561e4f] PASSED                                                                                                 [100%]

==================================================================================================== 5 passed in 11.87 seconds ====================================================================================================

$> git describe
0.5.0-73-gc56d948

jwodder pushed a commit to dandi/dandi-schema that referenced this issue May 20, 2021
Download of released datasets is broken, independent issue:
dandi/dandi-publish#112
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants