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

PostgreSQL integration #155

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

PostgreSQL integration #155

wants to merge 4 commits into from

Conversation

kleinmp
Copy link
Contributor

@kleinmp kleinmp commented Nov 13, 2018

No description provided.

@ElusiveMind
Copy link
Contributor

This looks good. I'm curious about the way postgresql is started. In my tests I used service to start postgresql (in my defaults.yaml file)

postgresql:
command: 'service postgresql start'

Is there any functional difference between that and the way it is started here?

@kleinmp
Copy link
Contributor Author

kleinmp commented Nov 19, 2018

When I started up a container and used the postgres service command the postgres service wasn't starting up for some reason. I didn't investigate too hard because this command worked fine and did the same thing.

Is that working for you? I could try it again.

@ElusiveMind
Copy link
Contributor

It did work, but it seems to take a while to start up. If your command resolves this issue, then I am a BIG BIG fan. I will test it in the next couple of days and report back on this PR

@ElusiveMind
Copy link
Contributor

It looks like what you're doing and what I am doing are the same on the 16.04 images. I have not at all tested the 14.04 images because of some other things I have running in 14.04 (such as Ruby 2) that aren't part of that distribution. The biggest issue I have is that Postgresql takes a while to start up. Not a dealbreaker though. I'll adopt your way.

@ElusiveMind ElusiveMind self-assigned this Nov 5, 2022
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

Successfully merging this pull request may close these issues.

None yet

2 participants