Skip to content

fix(scripts): derive seed PG* from DATABASE_URL - #3483

Open
Chessing234 wants to merge 1 commit into
block:mainfrom
Chessing234:fix/seed-pgport-from-database-url
Open

fix(scripts): derive seed PG* from DATABASE_URL#3483
Chessing234 wants to merge 1 commit into
block:mainfrom
Chessing234:fix/seed-pgport-from-database-url

Conversation

@Chessing234

Copy link
Copy Markdown
Contributor

Summary

  • scripts/seed-local-community.sh now prefers connection fields from DATABASE_URL when set, so a stale PGPORT in .env cannot send seeding at the host Postgres while migrations hit Buzz's container
  • Document the pairing in .env.example

Follow-up to the #2479 / #2511 port-override discussion (comment).

Test plan

  • With DATABASE_URL=…:55432/buzz and PGPORT=5432 in .env, run ./scripts/seed-local-community.sh and confirm it connects on 55432
  • With no DATABASE_URL, confirm defaults (localhost:5432 / buzz) still work
  • bash -n scripts/seed-local-community.sh

Made with Cursor

Keep community seeding on the same Postgres migrations use when
PGPORT in .env has drifted from DATABASE_URL.

Signed-off-by: Taksh <takshkothari09@gmail.com>
@Chessing234
Chessing234 requested a review from a team as a code owner July 29, 2026 07:24
@cameronhotchkies cameronhotchkies added the triage-ready Appropriate for agentic review label Jul 30, 2026
@Chessing234

Copy link
Copy Markdown
Contributor Author

@tlongwell-block @wesbillman @wpfleger96 mind taking a look when you get a chance?

@Chessing234
Chessing234 force-pushed the fix/seed-pgport-from-database-url branch from 8b4a853 to 4d8a3b6 Compare August 5, 2026 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

triage-ready Appropriate for agentic review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants