refactor finalize package

[endesapt]

Related to #1773

Removed reduntant SafeDelete calls where possible (except of ClusterRoles, orphaned resources)
Removed SafeDeleteForSelectorsWithFinalizer because it is repeats SafeDeleteWithFinalizer logic but with reduntant selector logic and only used in deleteSA.

---

Summary by cubic

Refactored finalizer logic to remove redundant delete calls, simplify ServiceAccount cleanup, and streamline the crObject interface, reducing API requests and avoiding label-mismatch issues. Fixes #1773.

• Refactors
  • Removed SafeDeleteForSelectorsWithFinalizer and the label-matching helper.
  • Simplified crObject: dropped GetLabels and SelectorLabels; use owner references for deletion.
  • Replaced duplicate SafeDelete calls for Role/RoleBinding and HTTPRoute with removeFinalizeObjByName; split e2e update checks for stability.

^{Written for commit 12c31ba. Summary will update on new commits.}

[cubic-dev-ai]

No issues found across 3 files

[AndrewChubatiuk]

thanks for a PR
could also simplify crObject interface, some methods are not needed anymore?

[vrutkovs]

Thank you for the contribution, that looks really good!

I updated the description as it doesn't fix all the points in the linked issue, and we don't want Github to auto close it

[vrutkovs]

Could you sign the commit with GPG as its a requirement for contribution? If not we could push the commit ourselves

[endesapt]

Thanks for filling the gaps for me. As i understand you already resigned commits with your gpg, so i don't need to do it?

[AndrewChubatiuk]

I've already signed them, thank you!

[vrutkovs]

Thanks for your contribution!