feat: support backup hooks on native sidecars

Open GabriFedi97 opened this issue 5 months ago • 2 comments

Include Sidecar containers when validating the existence of the target containers configured for the Backup Hooks.

Does your change fix a particular issue?

Fixes #9393

Please indicate you've done the following:

[x] Accepted the DCO. Commits without the DCO will delay acceptance.
[x] Created a changelog file (make new-changelog) or comment /kind changelog-not-required on this PR.
[ ] Updated the corresponding documentation in site/content/docs/main.

Nov 10 '25 14:11 GabriFedi97

Do we need to update the documentation for this patch? The Backup Hooks documentation section currently refers to containers in a general manner. Should we explicitly distinguish between main and sidecar containers?

Nov 10 '25 14:11 GabriFedi97

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 60.20%. Comparing base (c594026) to head (0fc8460).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #9403   +/-   ##
=======================================
  Coverage   60.19%   60.20%           
=======================================
  Files         386      386           
  Lines       35925    35934    +9     
=======================================
+ Hits        21624    21633    +9     
  Misses      12720    12720           
  Partials     1581     1581

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:

:snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
:package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Dec 10 '25 06:12 codecov[bot]