-
Notifications
You must be signed in to change notification settings - Fork 105
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
Handle empty file request on dedup store #1398
base: main
Are you sure you want to change the base?
Handle empty file request on dedup store #1398
Conversation
f372e8d
to
c43cdbe
Compare
c43cdbe
to
ec9e403
Compare
ec9e403
to
64baa70
Compare
64baa70
to
7e6c796
Compare
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.
Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: 1 of 2 LGTMs obtained, and all files reviewed, and pending CI: Publish image
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.
@allada was there any reason why empty digests wasn't checked on dedup store? I don't recall hitting this issue
Reviewable status: 1 of 2 LGTMs obtained, and all files reviewed, and pending CI: Publish image
Description
Fixes #1397
Type of change
Please delete options that aren't relevant.
How Has This Been Tested?
Added new test case in unit tests
Checklist
This change is