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

Managed Disk Download Failed #2588

Closed
vdubgeek opened this issue Feb 2, 2020 · 6 comments
Closed

Managed Disk Download Failed #2588

vdubgeek opened this issue Feb 2, 2020 · 6 comments
Assignees
Labels
⚙️ disks Related to managed disks ✅ merged A fix for this issue has been merged
Milestone

Comments

@vdubgeek
Copy link

vdubgeek commented Feb 2, 2020

Attempting to download managed disk with AzureStorageExplorer 1.12. Transfer fails after several hours. Following error is logged:
------Begin

AzCopy failed to complete the transfer: %s

AzCopy error message: Transfer of https://md-ssd-qdtft2bkpfmq.blob.core.windows.net/s0w1jpgphj2w/abcd to \?\D:\AzureDownload\epicor02-c.vhd failed: Status: Failed, Error code: 0

AzCopy log file can be found at C:\Users\administrator.ALTAKINC.azcopy\417f7757-c57d-014f-7079-70e5aa70d5e5.log

------End

417f7757-c57d-014f-7079-70e5aa70d5e5.log

@MRayermannMSFT
Copy link
Member

Hi @vdubgeek , it looks like the MD5 check failed:

2020/02/02 13:43:48 ERR: [P#0-T#0] DOWNLOADFAILED: https://md-ssd-qdtft2bkpfmq.blob.core.windows.net/s0w1jpgphj2w/abcd?si=cbc72cbf-9ef6-44ed-993d-50723914e18f&sig=-REDACTED-&sr=b&sv=2017-04-17 : 000 : the MD5 hash of the data, as we received it, did not match the expected value, as found in the Blob/File Service. This means that either there is a data integrity error OR another tool has failed to keep the stored hash up to date. When Checking MD5 hash. X-Ms-Request-Id: 

You can disable MD5 checking in Settings -> Transfers -> AzCopy -> Check MD5

Please let us know if that solves your problem.

@vdubgeek
Copy link
Author

vdubgeek commented Feb 5, 2020

It seems like it would be a bad idea to disable the MD5 check, no? I tried downloading several different VHD files using the explorer, and each one failed with a similar error. What might be causing the MD5 checksum to fail?

@MRayermannMSFT
Copy link
Member

MRayermannMSFT commented Feb 10, 2020

@vdubgeek we just learned today that we (Storage Explorer) should not be putting an MD5 hash on disks when uploading them and that we shouldn't check it when downloading. VMs/the Managed Disks backend don't keep that MD5 up-to-date. So it should be safe to disable the MD5 check while downloading disks. We'll also disable checking/putting MD5 for disks in 1.13.0.

Did you originally upload your disk using Storage Explorer?

@MRayermannMSFT MRayermannMSFT self-assigned this Feb 10, 2020
@MRayermannMSFT MRayermannMSFT added this to the 1.13.0 milestone Feb 10, 2020
@MRayermannMSFT MRayermannMSFT added the ⚙️ disks Related to managed disks label Feb 10, 2020
@MRayermannMSFT MRayermannMSFT added this to Committed in Storage Explorer via automation Feb 10, 2020
@vdubgeek
Copy link
Author

Interesting, good to know that the managed disks md5 is not kept current. I feel much better disabling the MD5 check knowing that. Thank you for the update

Storage Explorer automation moved this from Committed to Done Feb 11, 2020
@MRayermannMSFT
Copy link
Member

MRayermannMSFT commented Feb 11, 2020

Hey @vdubgeek, how did you originally create the disk? I just checked our code and we don't tell azcopy to set MD5 when uploading a disk. So I think it is important that we understand who originally put the MD5 on the disk's page blob.

Also, if you don't mind I need to leave this open for now as I do need to disable checking the md5 for disk download. I'll close this once I finish that work.

Storage Explorer automation moved this from Done to In Progress Feb 11, 2020
@MRayermannMSFT
Copy link
Member

@vdubgeek , I need to move on from this issue. If you need any additional help please open a new issue. Thanks.

Storage Explorer automation moved this from In Progress to Done Feb 25, 2020
@MRayermannMSFT MRayermannMSFT added the ✅ merged A fix for this issue has been merged label Feb 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ disks Related to managed disks ✅ merged A fix for this issue has been merged
Projects
Development

No branches or pull requests

2 participants