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.

Sign up for GitHub

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

Jump to bottom

feat: Apply semaphore and file lock on folder deletion #2737

Draft
wants to merge 2 commits into
base: topic/08-20-feat_add_log_level_to_background_task_message
Choose a base branch
from
Draft

feat: Apply semaphore and file lock on folder deletion #2737

wants to merge 2 commits into from

Conversation

fregataa
Copy link
Member

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • Mention to the original issue
  • Installer updates including:
    • Fixtures for db schema changes
    • New mandatory config options
  • Update of end-to-end CLI integration tests in ai.backend.test
  • API server-client counterparts (e.g., manager API -> client SDK)
  • Test case(s) to:
    • Demonstrate the difference of before/after
    • Demonstrate the flow of abstract/conceptual models with a concrete implementation
  • Documentation
    • Contents in the docs directory
    • docstrings in public interfaces and type annotations

@github-actions github-actions bot added the comp:storage-proxy Related to Storage proxy component label Aug 20, 2024
@fregataa Graphite App
Copy link
Member Author

fregataa commented Aug 20, 2024 •
edited
Loading

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fregataa and the rest of your teammates on Graphite Graphite

@github-actions github-actions bot added the size:M 30~100 LoC label Aug 20, 2024
This was referenced Aug 20, 2024
Closed
Closed
Closed
@fregataa fregataa force-pushed the topic/08-19-feat_apply_semaphore_and_file-lock_on_folder_deletion branch from e638f83 to ee7792f Compare August 20, 2024 08:35
@fregataa fregataa changed the base branch from main to topic/08-20-fix_handle_fs_error_when_deleting_vfolder August 20, 2024 08:35
@fregataa fregataa mentioned this pull request Aug 20, 2024
Merged
2 tasks
@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 force-pushed the topic/08-19-feat_apply_semaphore_and_file-lock_on_folder_deletion branch from ee7792f to be917b0 Compare August 20, 2024 08:48
@fregataa fregataa mentioned this pull request Aug 20, 2024
Merged
2 tasks
@fregataa fregataa changed the base branch from topic/08-20-fix_handle_fs_error_when_deleting_vfolder to topic/08-20-fix_vfolder_status_update_error August 20, 2024 08:48
@fregataa fregataa force-pushed the topic/08-20-fix_vfolder_status_update_error branch from 07eaecb to 0a511fe Compare August 20, 2024 08:59
Base automatically changed from topic/08-20-fix_vfolder_status_update_error to main August 20, 2024 09:08
@fregataa fregataa force-pushed the topic/08-19-feat_apply_semaphore_and_file-lock_on_folder_deletion branch from be917b0 to b16285c Compare August 20, 2024 09:20
@fregataa fregataa changed the base branch from main to topic/08-19-feat_run_vfolder_deletion_on_bgtask August 20, 2024 09:20
@fregataa fregataa force-pushed the topic/08-19-feat_apply_semaphore_and_file-lock_on_folder_deletion branch from b16285c to 98071a9 Compare November 9, 2024 03:47
@fregataa fregataa changed the base branch from topic/08-19-feat_run_vfolder_deletion_on_bgtask to topic/08-20-feat_add_log_level_to_background_task_message November 9, 2024 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@fregataa fregataa

Labels
comp:storage-proxy Related to Storage proxy component size:M 30~100 LoC
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@fregataa