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
doronAtuar opened this issue Mar 10, 2019 · 8 comments

Comments

@doronAtuar
Copy link

@doronAtuar doronAtuar 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 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

This comment has been minimized.

Copy link
Member

@MRayermannMSFT MRayermannMSFT commented Mar 13, 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...

@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
@doronAtuar

This comment has been minimized.

Copy link
Author

@doronAtuar doronAtuar 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?

@doronAtuar

This comment has been minimized.

Copy link
Author

@doronAtuar doronAtuar commented Apr 3, 2019

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

@MRayermannMSFT

This comment has been minimized.

Copy link
Member

@MRayermannMSFT MRayermannMSFT commented Apr 11, 2019

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

@MRayermannMSFT

This comment has been minimized.

Copy link
Member

@MRayermannMSFT MRayermannMSFT commented Apr 12, 2019

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

This comment has been minimized.

Copy link
Member

@MRayermannMSFT MRayermannMSFT commented Apr 17, 2019

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
@plivius

This comment has been minimized.

Copy link

@plivius 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).

@wahyuen wahyuen referenced this issue May 6, 2019
0 of 6 tasks complete
@MRayermannMSFT

This comment has been minimized.

Copy link
Member

@MRayermannMSFT MRayermannMSFT commented May 8, 2019

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
Projects
4 participants
You can’t perform that action at this time.