Skip to content

Conversation

@Endevir
Copy link
Member

@Endevir Endevir commented Oct 24, 2025

Description

Implement .spec.retention.ignoreForBackupConfigDeletion and .spec.retention.ignoreForBackupDeletion for BackupConfig.

This will allow to manage storage retention by S3 lifecycle policies and do not use operator for deleting anything from storage.

This will also protect from cases when accidential deletion for BackupConfig resources leads to removed all backups data in storage.

Fixes #40

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Manual tests

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 added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

…ention.ignoreForBackupDeletion for BackupConfig
@Endevir Endevir requested a review from a team as a code owner October 24, 2025 08:43
@Endevir Endevir merged commit 3181a60 into main Oct 24, 2025
4 checks passed
@Endevir Endevir deleted the feature/allow_skipping_storage_cleanup_on_backup_deletion branch October 28, 2025 09:40
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.

[FEATURE] Skip deletion of backups

3 participants