SUSE / caasp-services

Layered Services on the SUSE Container-as-a-Service Platform
Apache License 2.0
25 stars 30 forks source link

Adds Portus chart support for existing mariadb instance #18

Closed bonhamjp closed 7 years ago

bonhamjp commented 7 years ago

I removed the PORTUS_PRODUCTION_USERNAME value in the Portus and Crono deployments since the Portus app is configured to use the "root" db user by default :https://github.com/SUSE/Portus/blob/master/config/database.yml#L19

Plus the values file sets up a root Mariadb account by default, and I wanted all of the db configuration settings in the variables file to be in the "mariadb" section.

I'm wondering if we want to allow users to set a non "root" Mariadb account in the values file, because maybe somebody wants to incorporate an existing database they have, and they want to create a specific user for the Portus application to use?