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

fix: Handle fs error when deleting vfolder #2741

Merged

Conversation

fregataa
Copy link
Member

@fregataa fregataa commented Aug 20, 2024

Let handle_fs_errors() decorator handles OSError when deleting vfolder.

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • API server-client counterparts (e.g., manager API -> client SDK)

Copy link
Member Author

fregataa commented Aug 20, 2024

@fregataa fregataa force-pushed the topic/08-20-fix_handle_fs_error_when_deleting_vfolder branch from 0409604 to 5b5e8f4 Compare August 20, 2024 08:48
@fregataa fregataa mentioned this pull request Aug 20, 2024
2 tasks
@fregataa fregataa marked this pull request as ready for review August 20, 2024 08:49
@fregataa fregataa added this pull request to the merge queue Aug 20, 2024
Merged via the queue into main with commit b15a210 Aug 20, 2024
18 checks passed
@fregataa fregataa deleted the topic/08-20-fix_handle_fs_error_when_deleting_vfolder branch August 20, 2024 08:53
lablup-octodog pushed a commit that referenced this pull request Aug 20, 2024
Backported-from: main (24.09)
Backported-to: 24.03
Backport-of: 2741
lablup-octodog pushed a commit that referenced this pull request Aug 20, 2024
Backported-from: main (24.09)
Backported-to: 23.09
Backport-of: 2741
github-merge-queue bot pushed a commit that referenced this pull request Aug 20, 2024
github-merge-queue bot pushed a commit that referenced this pull request Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:storage-proxy Related to Storage proxy component size:XS ~10 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant