Clean up sample conf files

Bug #1315390 reported by Greg Lucas
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Triaged
Wishlist
Unassigned

Bug Description

The trove sample conf files contain a number of settings that are specific to the test/trove-integration environment and should not be included when the product is installed. They are also inconsistent with the other OpenStack components in a number of ways.

Some specific issues to address:

 - remove the 20-char trove-integration passwords, e.g. rabbit_password = f7999d1955c5014aa32c
 - use SQLite (not MySQL) as the default database, like all the other services
 - don't set log dir/file, or default to /var/log/trove/xxx

Revision history for this message
Greg Lucas (glucas-q) wrote :

Looking at e.g. the keystone and cinder sample conf files: they leave all the options commented out. This makes the sample file more like documentation, showing what all the defaults are.

We could take a similar approach with trove where most/all settings are commented out and we use the cfg.py defaults.

Changed in trove:
assignee: nobody → Greg Lucas (glucas-q)
Revision history for this message
Nikhil Manchanda (slicknik) wrote :

Greg: Some of the other OpenStack projects actually generate the sample.conf using tooling, based on the default config values in the python code (cfg.py). It would be awesome if we could do something similar for Trove.

Changed in trove:
status: New → Triaged
importance: Undecided → Medium
milestone: none → juno-1
Revision history for this message
Greg Lucas (glucas-q) wrote :

Nikhil - Agreed, that would be preferable. I'll take a look at how nova is doing that.

Changed in trove:
milestone: juno-1 → juno-2
Changed in trove:
milestone: juno-2 → juno-3
Changed in trove:
milestone: juno-3 → ongoing
Revision history for this message
Greg Lucas (glucas-q) wrote :

Nikhil - Should we close this based on the changes that have already merged?

It may make sense to create a new bug for generating the sample conf files. That will require breaking up cfg.py or somehow annotating it to indicate which options go in which file.

Revision history for this message
Greg Lucas (glucas-q) wrote :

Ignore my previous comment, I see that a number of the proposed changes have not yet been made.

Revision history for this message
Amrith Kumar (amrith) wrote :

Updating importance to "Wishlist" (bugscrub, 2014-10-23)

Changed in trove:
importance: Medium → Wishlist
Amrith Kumar (amrith)
Changed in trove:
assignee: Greg Lucas (glucas-q) → Amrith (amrith)
Amrith Kumar (amrith)
Changed in trove:
assignee: Amrith (amrith) → nobody
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.