We pass in quantum's sql connection string to the controller, but set the database up in the controller block

Bug #1090536 reported by Ian Wells
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cisco Openstack
New
Undecided
Unassigned
Folsom
Triaged
Medium
Unassigned

Bug Description

We pass this information in, but it's a value obviously derived from other arguments, because the actual creation of the database takes place in the same controller block. The controller block should, therefore, be constructing the DB connection string from the same information that quantum::db::mysql gets passed.

Revision history for this message
Mark T. Voelker (mvoelker) wrote :

I'm reading this as "there should be one source of truth but there isn't; therefore there's a chance that if someone changes one bit but not the other things could get broken". Correct?

Revision history for this message
Ian Wells (ijw-ubuntu) wrote :

Yes. It appears that you can change the SQL string and that's it; but the internal database setup sets up precisely that DB. If you change the SQL string that DB is still set up.

Revision history for this message
Daneyon Hansen (danehans) wrote :

The SQL connection information is exposed to the site/core manifest so we can leverage the controller_node_address variable. Separate SQL connections are used for services to give customers the option for providing unique user credentials. Here is an example from my core manifest:

ovs_sql_connection => "mysql://quantum:${quantum_db_password}@${controller_node_internal}/quantum",

Revision history for this message
Ian Wells (ijw-ubuntu) wrote :

The question is if we're passing the same information twice in different forms: once as the connection string, once as individual components to actually install the database. Bad if so.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.