gluster / gluster-block

A framework for gluster block storage
GNU General Public License v2.0
74 stars 32 forks source link

delete: allow when backend file is absent #279

Closed pkalever closed 3 years ago

pkalever commented 4 years ago

What does this PR achieve? Why do we need it?

To defend a rare case like when the gluster bricks goes down/up randomly, there is a chance that, unlink of backend file return success but the writev to metadata file about 'ENTRYDELETE: SUCCESS' failed.

In such cases this patch allow delete to progress.

Does this PR fix issues?

Fixes: BZ#1847425

Notes for the reviewer

Signed-off-by: Prasanna Kumar Kalever \prasanna.kalever@redhat.com\

pkalever commented 3 years ago

Thanks @lxbsz, Merged now