oracle-database-operator icon indicating copy to clipboard operation
oracle-database-operator copied to clipboard

fix: panic on error

Open tenstad opened this issue 2 years ago • 2 comments

Controller currently panics on error :/ due to invalid log format; specifically due to an odd number of key-value parameters in log.Error.

Copied the log format from log.Error(err, "Failed to update status for :"+pdb.Name, "err", err.Error()) above, although I don't really see why "err", err.Error() is needed in log.Error.

tenstad avatar Dec 14 '23 11:12 tenstad

Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA). The following contributors of this PR have not signed the OCA:

To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application.

When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated.

If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public.

Thank you for signing the OCA.