ratify icon indicating copy to clipboard operation
ratify copied to clipboard

Save errors from reconciling and report it during artifact verification

Open binbin-li opened this issue 7 months ago • 0 comments

What would you like to be added?

CR controllers adds the error messages to the CR status when the reconciling failed. However, artifact validation workflow will not catch the error during the reconciliation and return a general or even confusing error message to users.

One potential improvement is to save the error messages during reconciliation, then artifact validation could check those errors before calling the resources.

Another important consideration is that if a misconfigured CR has the same name as an existing one, we should not save the error but keep the original one. This follows our previous discussion that misconfigured CR should not override existing one. Therefore, we should only retain errors for new CRs with unique names.

Anything else you would like to add?

No response

Are you willing to submit PRs to contribute to this feature?

  • [ ] Yes, I am willing to implement it.

binbin-li avatar Jul 23 '24 13:07 binbin-li