Upgrade go.mod k8s.io/ go.mod to v0.31.3 and use server logger for client-go, controller-runtime

[kaovilai]

Signed-off-by: Tiger Kaovilai [email protected]

Thank you for contributing to Velero!

Please add a summary of your change

Upgrade k8s.io/ go.mod to v0.31.3 (minor version used by https://github.com/kubernetes-sigs/controller-tools/blob/v0.16.5/go.mod#L18) and set logger for client-go to use the same logger as the rest of the codebase for consistent logging.
Upgraded controller-gen to version that support the new deps.

cat go.mod | grep '\tk8s.io' | cut -f 2 | cut -d ' ' -f 1 | xargs -I {} -L 1 sh -c 'go get {}@v0.31.3'

Also bumped to support upgraded k8s.io/ deps.

• controller-gen to v0.16.5
• sigs.k8s.io/controller-runtime v0.19.2
• structs/funcs for eventhandlers are updated

Does your change fix a particular issue?

Fixes #3550

Please indicate you've done the following:

• Accepted the DCO. Commits without the DCO will delay acceptance.
• 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.

[kaovilai]

fyi I need to fix go sum here before merging :D

[kaovilai]

time="2024-11-25T20:27:46Z" level=fatal msg="controller with name backup already exists. Controller names must be unique to avoid multiple controllers reporting to the same metricunable to create controllercontrollerbackup-operations" logSource="pkg/cmd/server/server.go:661"

[kaovilai]

FYI zap is being removed in this PR.

It was meant to be removed here https://github.com/vmware-tanzu/velero/pull/2561/files#r432107463 and was kept here #8046 (comment)

This means all logs are going to be affected by the default logger flags, that means if user want all logs in json, they will get all logs in json.

[codecov]

Codecov Report

Attention: Patch coverage is 24.39024% with 31 lines in your changes missing coverage. Please review.

Project coverage is 58.96%. Comparing base (9f0026d) to head (c0f448e).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
pkg/util/kube/event_handler.go	0.00%	11 Missing ⚠️
internal/hook/wait_exec_hook_handler.go	64.28%	5 Missing ⚠️
pkg/cmd/cli/datamover/backup.go	0.00%	2 Missing ⚠️
pkg/cmd/cli/datamover/restore.go	0.00%	2 Missing ⚠️
pkg/cmd/cli/nodeagent/server.go	0.00%	2 Missing ⚠️
pkg/cmd/server/server.go	0.00%	1 Missing ⚠️
pkg/controller/backup_controller.go	0.00%	1 Missing ⚠️
pkg/controller/backup_finalizer_controller.go	0.00%	1 Missing ⚠️
pkg/controller/backup_operations_controller.go	0.00%	1 Missing ⚠️
...g/controller/backup_storage_location_controller.go	0.00%	1 Missing ⚠️
... and 4 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8450      +/-   ##
==========================================
- Coverage   58.98%   58.96%   -0.02%     
==========================================
  Files         368      368              
  Lines       39000    39017      +17     
==========================================
+ Hits        23004    23007       +3     
- Misses      14532    14546      +14     
  Partials     1464     1464              

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