Skip to content

Backport PR #9693 and #9700 to Release-1.18#9731

Merged
kaovilai merged 3 commits intovelero-io:release-1.18from
priyansh17:user/priyansh/backport-9693-1.18
Apr 16, 2026
Merged

Backport PR #9693 and #9700 to Release-1.18#9731
kaovilai merged 3 commits intovelero-io:release-1.18from
priyansh17:user/priyansh/backport-9693-1.18

Conversation

@priyansh17
Copy link
Copy Markdown
Collaborator

Thank you for contributing to Velero!

Please add a summary of your change

This is backporting of PRs #9693 and #9700 in main branch to release-1.16.

Please indicate you've done the following:

…elero-io#9693)

* Enhance backup deletion logic to handle tarball download failures and clean up associated CSI VolumeSnapshotContents
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>

* added changelog
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>

* Refactor error handling in backup deletion
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>

* Refactor backup deletion logic to skip CSI snapshot cleanup on tarball download failure
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>

* prevent backup deletion when errors occur
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>

* added logger
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>
…ion (velero-io#9700)

* Add delay to avoid race conditions during VolumeSnapshotContent deletion
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>

* updated changelog
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>

* Updated Changelog
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>
Signed-off-by: Priyansh Choudhary <im1706@gmail.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 16, 2026

Codecov Report

❌ Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.65%. Comparing base (77945d9) to head (3810d88).
⚠️ Report is 1 commits behind head on release-1.18.

Files with missing lines Patch % Lines
pkg/controller/backup_deletion_controller.go 85.71% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##           release-1.18    #9731   +/-   ##
=============================================
  Coverage         60.64%   60.65%           
=============================================
  Files               387      387           
  Lines             36896    36890    -6     
=============================================
  Hits              22376    22376           
+ Misses            12891    12888    -3     
+ Partials           1629     1626    -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kaovilai kaovilai merged commit 4c91959 into velero-io:release-1.18 Apr 16, 2026
60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants