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

Handle write-perm on LOCATE correctly #233

Merged
merged 1 commit into from
Dec 9, 2020
Merged

Handle write-perm on LOCATE correctly #233

merged 1 commit into from
Dec 9, 2020

Conversation

piste-jp
Copy link
Member

@piste-jp piste-jp commented Dec 9, 2020

Summary of changes

Override the error code to LTFS_LOCATE_ERROR when tape backend's locate() call returns write-perm error.

Description

On #232, LTFS could remove files automatically when tape backend's locate() call returns write-perm error even if It is not necessary. But I believe this is so rare like breaking H/W.

Fixes #232

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have confirmed my fix is effective or that my feature works

@piste-jp piste-jp self-assigned this Dec 9, 2020
@piste-jp piste-jp merged commit fffa125 into LinearTapeFileSystem:master Dec 9, 2020
@piste-jp piste-jp deleted the locate-error branch December 9, 2020 07:13
piste-jp pushed a commit that referenced this pull request Dec 9, 2020
@piste-jp
Copy link
Member Author

piste-jp commented Dec 9, 2020

Cherry picked to branch v2.4-stable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LOCATE error handles as write perm
1 participant