opendatahub-io / model-registry-operator

Apache License 2.0
3 stars 17 forks source link

feat: Add support for MySQL database, fixes RHOAIENG-3246, RHOAIENG-3247 #60

Closed dhirajsb closed 7 months ago

dhirajsb commented 7 months ago

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:

dhirajsb commented 7 months ago

MySQL test is timing out. Will take a look in the morning.

dhirajsb commented 7 months ago

For some reason the mysql DB in the ci Kind cluster fails to start with the error:

mysqladmin: connect to server at 'localhost' failed
error: 'Can't connect to local MySQL server through socket '/var/run/mysqld/mysqld.sock' (2)'
Check that mysqld is running and that the socket: '/var/run/mysqld/mysqld.sock' exists!

But the same mysql 8.3.0 image runs fine in a local Kind cluster and a ROSA OpenShift cluster. I've switched the test back to using postgres so that this Kind test for CI can pass. It still exercises the operator template changes so it's a valid test.