-
Notifications
You must be signed in to change notification settings - Fork 0
can no longer download a published asset #112
Comments
Download of released datasets is broken, independent issue: dandi/dandi-publish#112
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. |
Any ETA so we could proceed with testing? |
We hope to have this fixed by tomorrow. |
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. |
Download of released datasets is broken, independent issue: dandi/dandi-publish#112
Thank you @brianhelba ! how did data migrate across regions? or it just was "published" originally into the wrong region? |
Our bucket for testing publish functionality was created in the wrong 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 |
Also, the copy is still proceeding, about 65% complete. |
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. |
This is now resolved. |
confirming - tests of (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 |
Download of released datasets is broken, independent issue: dandi/dandi-publish#112
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
I would recommend to use this small dataset in a unit/smoke test on dandi-publish or wherever it is pertinent.
The text was updated successfully, but these errors were encountered: