DVC does fails when the md5 hash contains only numbers

[charlesbaynham]

The description is in the title. When an md5 hash randomly contains only numbers DVC fails, reporting

ERROR: failed to update 'step_X.dvc'. - DVC-file 'step_X.dvc' format error: expected str for dictionary value @ data['md5']

This should happen on average once for every 3 million hashes. For a big project with a lot of files that get changed, that's not so implausible.

[efiop]

Hi @charlesbaynham ! Could you please show dvc version output?

[efiop]

For the record: can reproduce this myself as well, by modifying md5 in a dvc file by-hand. We've had a similar issue with pyyaml earlier, need to check this.

[efiop]

Ok, so both pyyaml and ruamel are affected, so we need to tweak our SCHEMA instead. Working on a fix...

[efiop]

@charlesbaynham We are releasing 0.85.0 right now. Please upgrade, give it a try and let us know if that fixed the issue for you or not. Thanks for the feedback! 🙂

[charlesbaynham]

Wow I think that's the fastest response I've ever seen on github! Thanks a lot, I'll give it a try