-
Epic
-
Resolution: Done
-
Major
-
None
-
None
-
Do not persist Status if Reconcile exits due to panic
-
False
-
-
False
-
Not Selected
-
Proposed
-
No Docs Impact
-
Done
-
Proposed
-
rhos-conplat-core-operators
-
Proposed
-
0% To Do, 0% In Progress, 100% Done
-
-
-
Moderate
Today we use a deferred call to always update the Status field when the Reconcile() call exits. We should consider not doing that if the Reconcile() exited due to panic, to avoid persisting potentially incorrect data.
Same is fixed for nova-operator and placement-operator through below Epic.
https://issues.redhat.com/browse/OSPRH-16687
This looks like something that we should replicate across all our operators, as we use the same defer() pattern in all of them.