-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
objects: fix gdrive CI issue #6338
Conversation
dvc/fs/base.py
Outdated
size = self.info(path_info).get("size") | ||
if size is not None: | ||
size = int(size) | ||
return size |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was gdrive returning string size? fs.info
should always return size as int, so prob worth fixing in gdrive.py
itself.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, there were 2 issues
- after the transfer change we were verifying the destination ODB on both push and pull, when previously we only verified on pull
- the verify on push caused us to read sizes from the gdrive fs, which made the issue w/gdrive returning string sizes show up
I'll move the conversion into our gdrive fs, still waiting for the branch build to finish to make sure there's nothing else
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pmrowla Nice catch! π₯
0db5b1d
to
a36750a
Compare
Thank you, @pmrowla ! π |
Hm, looks like |
β I have followed the Contributing to DVC checklist.
π If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.
Thank you for the contribution - we'll try to review it as soon as possible. π