kubevirt / hyperconverged-cluster-operator / 1451

72%

main: 76%

push

travis-ci-com

[release-2.2] Don't create problematic ownerReferences for CDI and NetworkAddonsConfig (#401)

* Avoid creating problematic ownerReferences.

It's not legal to have cross-namespaced owners, that is, a
cluster scoped object owned by a namespaced object.

To ensure we preserve the same behaviour upon deletion,
create a finalizer and have it clean up the objects.
(Inspired by #122, which was accidentally lost in a refactor.)

To ensure we are still able to watch for events on those
children, switch to EnqueueRequestsFromMapFunc, which always
maps to HCO.

Signed-off-by: Maya Rashish <mrashish@redhat.com>

* Don't create problematic ownership relationship for KubeVirtCommonTemplateBundles

Handle deletion manually via a finalizer, as in the CDI case.
If ownerReferences already exist, delete them.

Signed-off-by: Maya Rashish <mrashish@redhat.com>

Co-authored-by: maya-r <57391580+maya-r@users.noreply.github.com>

171 of 171 new or added lines in 1 file covered. (100.0%)

751 of 1043 relevant lines covered (72.0%)

3.3 hits per line