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

appliance/postgres: Add DATABASE_URL to postgres provider #2024

Merged
merged 1 commit into from Oct 25, 2015

Conversation

Projects
None yet
3 participants
@josephglanville
Copy link
Contributor

commented Oct 24, 2015

Closes #2020

@josephglanville

This comment has been minimized.

Copy link
Contributor Author

commented Oct 24, 2015

Docs will probably need updating to reflect that you no longer need to set DATABASE_URL manually.

@titanous

This comment has been minimized.

@josephglanville

This comment has been minimized.

Copy link
Contributor Author

commented Oct 24, 2015

Cool, I will fix up the docs too. :)

@josephglanville josephglanville force-pushed the database_url branch from 9307bb1 to 4e2f9e2 Oct 24, 2015

```text
flynn env set DATABASE_URL=$(export $(flynn env | xargs); echo "postgres://$PGUSER:$PGPASSWORD@$PGHOST:5432/$PGDATABASE")
```
Flynn will also create the `DATABASE_URL` environment which is utilized by some frameworks to configure database connections.

This comment has been minimized.

Copy link
@titanous

titanous Oct 24, 2015

Member

s/environment/environment variable/

appliance/postgres: Add DATABASE_URL to postgres provider
Signed-off-by: Joseph Glanville <jpg@jpg.id.au>

@josephglanville josephglanville force-pushed the database_url branch from 4e2f9e2 to 69659bd Oct 24, 2015

@titanous

This comment has been minimized.

Copy link
Member

commented Oct 24, 2015

LGTM

@kwando

This comment has been minimized.

Copy link

commented Oct 25, 2015

❤️

josephglanville added a commit that referenced this pull request Oct 25, 2015

Merge pull request #2024 from flynn/database_url
appliance/postgres: Add DATABASE_URL to postgres provider

@josephglanville josephglanville merged commit 4dce31e into master Oct 25, 2015

1 check failed

continuous-integration/flynn The Flynn CI build failed [1 failure(s)]
Details

@josephglanville josephglanville deleted the database_url branch Oct 25, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.