Skip to content
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

dev-tools should support option of using postgresql #13

Closed
eggmaster opened this issue Dec 4, 2012 · 2 comments
Closed

dev-tools should support option of using postgresql #13

eggmaster opened this issue Dec 4, 2012 · 2 comments
Milestone

Comments

@eggmaster
Copy link
Contributor

This should allow for specification of postgresql username and password via environment variables, as well as configuration of the database name conductor will be using.

@cwolferh
Copy link
Contributor

cwolferh commented Dec 6, 2012

Breaking it into two steps:

  1. The puppet scripts should be able to accept facter environment
    variables for an existing postgres database and configure conductor
    accordingly.

  2. If the user specifies certain env vars, bootstrap.sh should install
    postgres (if needed) and create a new database (if desired), then set
    the appropriate facter vars before puppet gets invoked.

Focusing on the first step, I propose that when in "postgres mode"
puppet configures conductor based on the following env vars:

FACTER_POSTGRES_USERNAME  # typically just the current system user
FACTER_POSTGRES_PASSWORD
FACTER_POSTGRES_DBNAME

I think it is safe for the puppet scripts to assume the db is empty
(at least for now).

If FACTER_POSTGRES_DBNAME is not defined, puppet should configure
sqlite as it does currently.

Once the above is done, we can start step 2.

The following scripts may be useful for reference on postgres configuration:
https://gist.github.com/3178181#file_conductor_dev_root_prep.sh
https://gist.github.com/3178181#file_conductor_dev_setup.sh

@cwolferh
Copy link
Contributor

cwolferh commented Jan 7, 2013

Closing since this was addressed with #15 .

@cwolferh cwolferh closed this as completed Jan 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants