-
Story
-
Resolution: Done
-
Undefined
-
None
-
None
-
False
-
None
-
False
-
-
The error handling in and around the updateStatus() function in all three controllers looks suspicious and is also hard to understand. We should refactor it and solve any bugs that become apparent during the refactor.
For example, the updateStatus function takes the reconcile error as an argument. Sometimes, it returns this error, and sometimes it doesn't. It should probably only return an error when a new error occurs within the function itself. It should not return the reconcile error that was passed in. Instead, the caller should handle the four possible combinations of (reconcileError, statusError).
- is related to
-
OSSM-6140 No indication of error when deploying an IstioCNI resource to non-existent namespace
- Closed