Differential D1222

docker-compose/vault: Fix database configuration
ClosedPublic
Actions

Authored by anlambert on Mar 6 2019, 11:39 AM.

Tags

None

Subscribers

Details

Reviewers

Group Reviewers

Commits

rDENV8706815679b7: vault: Fix database configuration
rCDFD8706815679b7: vault: Fix database configuration

Summary

This fixes the correct initialization of the vault services.

Diff Detail

Repository

rCDFD Dockerfiles for developers

Branch

fix-vault-db

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 4548
Build 6033: arc lint + arc unit

Event Timeline

anlambert created this revision.Mar 6 2019, 11:39 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptMar 6 2019, 11:39 AM

Harbormaster completed remote builds in B4548: Diff 3836.Mar 6 2019, 11:39 AM

ardumont requested changes to this revision.Mar 6 2019, 12:03 PM

ardumont added a subscriber: ardumont.

ardumont added inline comments.

docker-compose.yml
204	as you fixed env/vault.env, you should simply keep it here. What did i miss?

This revision now requires changes to proceed.Mar 6 2019, 12:03 PM

olasd added a subscriber: olasd.Mar 6 2019, 12:07 PM

olasd added inline comments.

docker-compose.yml
204	the PGHOST variable in the environment file breaks the initialization of that container initialization of postgres containers is inconsistent (we seem to have three different ways of doing it, this diff scales it back down to two)

the PGHOST variable in the environment file breaks the initialization of that container
initialization of postgres containers is inconsistent (we seem to have three different ways of doing it, this diff scales it back down to two)

Damn!
Thanks for the heads up!

This revision is now accepted and ready to land.Mar 6 2019, 12:11 PM

Closed by commit rCDFD8706815679b7: vault: Fix database configuration (authored by anlambert). · Explain WhyMar 6 2019, 1:16 PM

This revision was automatically updated to reflect the committed changes.

anlambert added a commit: rDENV8706815679b7: vault: Fix database configuration.Nov 26 2019, 1:35 PM

Revision Contents
Changeset List

Path

Size

docker-compose.yml

4 lines

env/

2 lines

Diff 3836

docker-compose.yml

Loading...

env/vault.env

Loading...