equip gc controller with configurable frequency

[allenxu404]

Signed-off-by: allenxu404 qix2@vmware.com

Thank you for contributing to Velero!

Please add a summary of your change

• add a field of frequency into gcReconciler

Does your change fix a particular issue?

Please indicate you've done the following:

• Accepted the DCO. Commits without the DCO will delay acceptance.
• Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
• Updated the corresponding documentation in site/content/docs/main.

[codecov-commenter]

Codecov Report

Merging #5248 (a8ba487) into main (e849441) will decrease coverage by 0.16%.
The diff coverage is 61.53%.

@@            Coverage Diff             @@
##             main    #5248      +/-   ##
==========================================
- Coverage   41.03%   40.87%   -0.17%     
==========================================
  Files         231      233       +2     
  Lines       19645    20043     +398     
==========================================
+ Hits         8062     8193     +131     
- Misses      10993    11253     +260     
- Partials      590      597       +7     

Impacted Files	Coverage Δ
pkg/cmd/server/server.go	6.60% <0.00%> (-0.04%)	⬇️
pkg/controller/gc_controller.go	55.55% <66.66%> (-0.76%)	⬇️
pkg/restic/common.go	26.86% <0.00%> (-12.92%)	⬇️
pkg/uploader/provider/kopia.go	57.02% <0.00%> (-4.14%)	⬇️
pkg/controller/backup_controller.go	48.19% <0.00%> (-0.29%)	⬇️
pkg/podvolume/util.go	100.00% <0.00%> (ø)
pkg/repository/manager.go	20.40% <0.00%> (ø)
pkg/repository/provider/restic.go	0.00% <0.00%> (ø)
pkg/restore/restic_restore_action.go	75.24% <0.00%> (ø)
... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[reasonerjt]

lgtm

[ywk253100]

@allenxu404 Could you remove this line or just log the needed information? This hits a bug of delve when debugging Velero on Mac as the DeletionTimestamp of the backup is nil.

[allenxu404]

@allenxu404 Could you remove this line or just log the needed information? This hits a bug of delve when debugging Velero on Mac as the DeletionTimestamp of the backup is nil.

Thanks for your information. Yes, I think I can just log the name of backup instead.