storj / edge

Storj edge services (including multi-tenant, S3-compatible server to interact with the Storj network)
GNU Affero General Public License v3.0
48 stars 18 forks source link

Adjust DeleteBucket (forceful deletion) satellite endpoint (object retention) #402

Closed amwolff closed 2 weeks ago

amwolff commented 4 months ago

Goal

This issue is to make meaningful progress on Object Lock implementation within core and edge services. Specifically, to support immutable backups in backup software products like Veeam.

What needs to be done? • Acceptance Criteria

Modify the corresponding to DeleteBucket endpoint on the satellite (the endpoint might be named differently):

Summary

Implementation reference.

Depends on…

Links

kaloyan-raev commented 4 months ago

The S3 API allows deleting only empty buckets. The forceful deletion of buckets is a Minio extension of the S3 protocol.

For the MVP, we could support forceful deletion only for buckets not configured with object lock. For those with object lock, we could just return an error that the operation is unsupported. The check should be on bucket level, and not checking individual object versions for their retention period.

amwolff commented 4 months ago

The check should be on bucket level, and not checking individual object versions for their retention period.

That's a good idea. I concur.

storj-gerrit[bot] commented 2 months ago

Change satellite/{metabase,metainfo}: make DeleteBucket respect retention mentions this issue.