Skip to content

[Don't Merge] Reference PR: Fix broken postgres init in docker compose#10773

Draft
cslzchen wants to merge 1 commit intoCenterForOpenScience:developfrom
cslzchen:feature/fix-postgres-config
Draft

[Don't Merge] Reference PR: Fix broken postgres init in docker compose#10773
cslzchen wants to merge 1 commit intoCenterForOpenScience:developfrom
cslzchen:feature/fix-postgres-config

Conversation

@cslzchen
Copy link
Collaborator

@cslzchen cslzchen commented Oct 9, 2024

Purpose

  • Note: don't merge since this breaks GV
  • Note: only need this two envs to init the OSF DB

postgres container fails to start

(osf-3.12) ➜  osf git:(develop) ✗ docker compose up postgres
[+] Running 2/2
 ✔ Network osf_default       Created
 ✔ Container osf-postgres-1  Created
Attaching to postgres-1
postgres-1  | Error: Database is uninitialized and superuser password is not specified.
postgres-1  |        You must specify POSTGRES_PASSWORD to a non-empty value for the
postgres-1  |        superuser. For example, "-e POSTGRES_PASSWORD=password" on "docker run".
postgres-1  |
postgres-1  |        You may also use "POSTGRES_HOST_AUTH_METHOD=trust" to allow all
postgres-1  |        connections without a password. This is *not* recommended.
postgres-1  |
postgres-1  |        See PostgreSQL documentation about "trust":
postgres-1  |        https://www.postgresql.org/docs/current/auth-trust.html
postgres-1 exited with code 1

Changes

Add back the following configuration

POSTGRES_DB: osf
POSTGRES_HOST_AUTH_METHOD: trust

QA Notes

N/A

Documentation

N/A

Side Effects

Breaks GravyValet

Ticket

Discovered during ENG-6297

@Johnetordoff
Copy link
Contributor

@cslzchen Did this get resolved, are we going to merge or close this?

@cslzchen
Copy link
Collaborator Author

cslzchen commented Mar 14, 2025

@Johnetordoff and Cc @opaduchak , here is the original discussion and I don't think it has been resolved

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.

2 participants