kubevirt / containerized-data-importer / 4802

62%

main: 49%

push

travis-ci

Avoid failing reconciliation on new CR (#1142)

If the user explicitly delete CDI cr,
HCO will quickly try to create a new one.
If HCO is quick enough, CDI operator can
enter the reconciliation loop when an older
cdi-config config-map is still there although
marked for deletion.
In that case CDI operator was not going to create
a new config-map but was then marking the new CDI CR
in error phase just because it's still not controlled
by the config map pending for deletion.
On the next run, CDI operator was not going to create a
new config map just because the new CR is already marked
with phase=Errror.
Skip the controlled-by check on config maps marked for
deletion to avoid this bad loop.

Fixes: https://bugzilla.redhat.com/1809872

Signed-off-by: Simone Tiraboschi <stirabos@redhat.com>

Co-authored-by: Simone Tiraboschi <stirabos@redhat.com>

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

5082 of 8199 relevant lines covered (61.98%)

0.68 hits per line