-
Notifications
You must be signed in to change notification settings - Fork 490
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
Inconsistency between Azure Blob Storage and other storage types #1815
Comments
@Eitan03 one could argue that the internal representation of the storage is not of concern to the user, so I wonder what exactly your use case is. Transferring storage is not in focus of Athens I would say and when it's needed, it is a one-time task that can be easily covered with custom scripting. Or am I wrong? However, does this issue prevent pre-filling with |
linking issues #1583 |
for documentation purposes (when we discuss this), here is a rough overview of code parts:
so currently, if I'm not mistaken, The argument to leave it as is would be that the internal storage representation of a cloud storage should not be of concern to the user and it could change with future releases. Also, a different storage provider may require a different representation. |
We should look at this inconsistency and find a reason for it (and document it) or correct it for version 1.0.0 |
When storing packages in Azure Blob Storage, the package files are stored under
/path/to/module/@v/
but when using other storage types (i.e disk storage), the packages are stored in
/path/to/module/$VERSION/
this difference creates problems when trying to change the storage type by uploading an existing folders from the disk to azure
this difference is not documented anywhere (from what Ive seen),
when trying to access a package that is stored in blob storage with a disk hierarchy format you recieve 404 without any additional information, and since the logs doesnt describe which path was not found, there isnt any clear way to identify the problem.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Consistent path in all storage platforms, a change is required either to the azure blob storage code to store the packages in
/path/to/module/$VERSION/
or to all the other storage types to match azure pathingEnvironment (please complete the following information):
Additional Information
In addition to the difference in the path hierarchy, the file names differ as well, where in disk storage the files are stored as
$VERSION.info
,source.zip
andgo.mod
, in azure theyre stored as$VERSION.info
$VERSION.mod
and$VERSION.zip
.I assume this difference comes from all of the versions being stored in the same folder in azure and on different folders in other storage types
The text was updated successfully, but these errors were encountered: