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

Docker: Improve postgres setup #3221

Merged
merged 2 commits into from
Mar 5, 2021
Merged

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Jan 31, 2021

This PR allows us to use the docker-compose setup to run cargo test with the Dockerized Postgres server. It also adjusts the docker-compose setup a little bit to not expose the postgres container on all host interfaces, but instead only makes it available on 127.0.0.1.

To run just the database container you can use:

docker-compose up -d postgres

(-d runs it in daemon mode)

/cc @plaets

r? @pietroalbini

@plaets
Copy link
Contributor

plaets commented Feb 3, 2021

Why was I mentioned here? Not that I have a problem with that, I'm just not sure if I wasn't supposed to do something 😅

@Turbo87
Copy link
Member Author

Turbo87 commented Feb 3, 2021

@plaets only for awareness because you were working on the other Docker topics :)

@Turbo87 Turbo87 added the C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear label Feb 11, 2021
@Turbo87 Turbo87 force-pushed the docker-test-db branch 2 times, most recently from dca0c59 to 99a80e3 Compare February 14, 2021 20:25
@Turbo87 Turbo87 force-pushed the docker-test-db branch 3 times, most recently from 409bfb3 to 30a73ce Compare February 26, 2021 23:13
@Turbo87 Turbo87 force-pushed the docker-test-db branch 4 times, most recently from 636a797 to d6743a2 Compare March 1, 2021 21:52
@jtgeibel
Copy link
Member

jtgeibel commented Mar 5, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Mar 5, 2021

📌 Commit 1daff49 has been approved by jtgeibel

@bors
Copy link
Contributor

bors commented Mar 5, 2021

⌛ Testing commit 1daff49 with merge 47ef3e0...

@bors
Copy link
Contributor

bors commented Mar 5, 2021

☀️ Test successful - checks-actions
Approved by: jtgeibel
Pushing 47ef3e0 to master...

@bors bors merged commit 47ef3e0 into rust-lang:master Mar 5, 2021
@Turbo87 Turbo87 deleted the docker-test-db branch March 5, 2021 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants