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

NC | lifecycle - add expire rule #8862

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

NC | lifecycle - add expire rule #8862

wants to merge 1 commit into from
+282 −60

Conversation

nadavMiz
Copy link
Contributor

@nadavMiz nadavMiz commented Mar 10, 2025 •
edited
Loading

Describe the Problem

part of the effort to support lifecycle in NC. add expire rule for lifecycle policy

Explain the Changes

  1. added filter for non versioned objects to be expired based on namespace_fs list_object function
  2. added tagging to get_object_info and made a common function to get tags from xattr
  3. added tests

continuation of #8853

Gaps

list object works in batches. need to support filtering and deleting in batches

Testing Instructions:

  1. run sudo npx jest test_nc_lifecycle_cli
  • Doc added/updated
  • Tests added

@nadavMiz nadavMiz requested a review from romayalon March 10, 2025 13:45
@nadavMiz nadavMiz force-pushed the lifecycle_expire branch 2 times, most recently from f636bef to 25ce209 Compare March 10, 2025 13:59
@nadavMiz nadavMiz self-assigned this Mar 10, 2025
@nadavMiz nadavMiz mentioned this pull request Mar 9, 2025
Open
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@romayalon romayalon Awaiting requested review from romayalon

At least 1 approving review is required to merge this pull request.

Assignees

@nadavMiz nadavMiz

Labels
size/L
Projects
None yet
Milestone
No milestone
Development

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