-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add support for MySQL database, fixes RHOAIENG-3246, RHOAIENG-3247 #60
Conversation
MySQL test is timing out. Will take a look in the morning. |
a2424de
to
49ac0e8
Compare
…se config, fixed handling errors when deleting
For some reason the mysql DB in the ci Kind cluster fails to start with the error:
But the same mysql 8.3.0 image runs fine in a local Kind cluster and a ROSA OpenShift cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Description
How Has This Been Tested?
Unit test updated to test both DB configs.
Tested locally by using the updated mysql and postgres samples.
Merge criteria: