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

MD5 is missing from uploaded files #1223

Closed
da2r-20 opened this issue Mar 10, 2019 · 8 comments
Closed

MD5 is missing from uploaded files #1223

da2r-20 opened this issue Mar 10, 2019 · 8 comments
Assignees
Labels
⚙️ azcopy Related to AzCopy integration ❔ more info More information is needed from issue author to proceed ✅ merged A fix for this issue has been merged
Milestone

Comments

@da2r-20
Copy link

da2r-20 commented Mar 10, 2019

Storage Explorer Version: 1.7.0
Platform/OS: macOS Mojave
Architecture: x64
Regression From: yes, 1.6.*

Bug description
MD5 is missing from uploaded files

Steps to Reproduce

  1. upload a file
  2. check the file's properties for md5

Expected Experience
md5 should be present

Actual Experience
md5 is missing

@craxal craxal added the ❔ more info More information is needed from issue author to proceed label Mar 11, 2019
@craxal craxal added this to the 1.8.0 milestone Mar 11, 2019
@craxal craxal added this to Committed in Storage Explorer via automation Mar 11, 2019
@MRayermannMSFT
Copy link
Member

Hi @doronAtuar , quick clarification question. Are you uploading to a File share, an ADLS Gen2 Blob container, or a normal Blob container? For a normal Blob container, I cannot repro your issue. For a File Share and an ADLS Gen2 Blob container I can.

For File share, I don't think this is a regression, but I'm pretty sure we can add it.

For ADLS Gen2 Blob container, it doesn't look that property currently exists for those blobs, so I don't think it is something we can support at the moment...

@MRayermannMSFT MRayermannMSFT moved this from Committed to Investigating in Storage Explorer Mar 14, 2019
@MRayermannMSFT MRayermannMSFT moved this from Investigating to Blocked in Storage Explorer Mar 15, 2019
@MRayermannMSFT MRayermannMSFT moved this from Blocked to Investigating in Storage Explorer Mar 15, 2019
@da2r-20
Copy link
Author

da2r-20 commented Mar 25, 2019

Hi @doronAtuar , quick clarification question. Are you uploading to a File share, an ADLS Gen2 Blob container, or a normal Blob container? For a normal Blob container, I cannot repro your issue. For a File Share and an ADLS Gen2 Blob container I can.

For File share, I don't think this is a regression, but I'm pretty sure we can add it.

For ADLS Gen2 Blob container, it doesn't look that property currently exists for those blobs, so I don't think it is something we can support at the moment...

Hi @MRayermannMSFT
I'm using a regular Blob container

Let me know how I can provide you with more helpful data.
It reproduces everytime.
Maybe debug prints?

@da2r-20
Copy link
Author

da2r-20 commented Apr 3, 2019

I managed to narrow it down:
It reproduces only for large files
In my case > 500mb

@MRayermannMSFT
Copy link
Member

Thanks for narrowing it down! Will look into seeing if I see that as well.

@MRayermannMSFT
Copy link
Member

Hey @doronAtuar , I'm only repro'ing it with standard blob containers if I have AzCopy turned on. Do you also have it enabled?

@MRayermannMSFT MRayermannMSFT moved this from Investigating to Under Review in Storage Explorer Apr 12, 2019
@MRayermannMSFT MRayermannMSFT added 🔍 pending ✅ merged A fix for this issue has been merged and removed 🔍 pending labels Apr 16, 2019
@MRayermannMSFT
Copy link
Member

AzCopy 10.0.9 has been inserted and the calculate MD5 flag is definitely being set now.

Storage Explorer automation moved this from Under Review to Done Apr 17, 2019
@MRayermannMSFT MRayermannMSFT added the ⚙️ azcopy Related to AzCopy integration label Apr 19, 2019
@plivius
Copy link

plivius commented Apr 24, 2019

Hi @MRayermannMSFT , when can we expect a new SE release containing mentioned version of AzCopy >= 10.0.9?
Some of our clients (dependent on using SE with AzCopy) have reported the same issue under the same conditions, however for all file sizes (20Mb - 4Gb).

@MRayermannMSFT
Copy link
Member

Hey @plivius , 1.8.0 was released last week. Included in it is AzCopy 10.1.0!
image
Taken from: https://docs.microsoft.com/en-us/azure/vs-azure-tools-storage-explorer-relnotes#version-180

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ azcopy Related to AzCopy integration ❔ more info More information is needed from issue author to proceed ✅ merged A fix for this issue has been merged
Projects
Development

No branches or pull requests

4 participants