Closed razo7 closed 10 months ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
/test 4.14-openshift-e2e /test 4.15-openshift-e2e
/test 4.14-openshift-e2e /test 4.15-openshift-e2e
/lgtm
/retest
/retest
I wonder if all the error messages after UpdateStatus failures could be put inside UpdateStatus function itself.
They can but this way we log an error in the function and Reconcile
I wonder if all the error messages after UpdateStatus failures could be put inside UpdateStatus function itself.
They can but this way we log an error in the function and Reconcile
Sorry, I meant inside utils.UpdateConditions
.
IIUC the additional information this new logs provide is the CR's name, which is already accessible inside utils.UpdateConditions
, so we could simply update the error logs that are inside utils.UpdateConditions
SGTM
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: clobrano, razo7
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest
The
UpdateConditions
function has an error only when an unknown agent has been found. Thus, there is no need to return and check it from reconcile, since it is not an error from trying to update it. FAR can simply log an informative error when it was called when unknown ConditionsChangeReason. Moreover, the lastTrinsitionTime print is more human friendly