Skip to content

feat: set up core postgres tables - #6308

Merged
michaelkedar merged 4 commits into
google:masterfrom
michaelkedar:💽📋
Nov 9, 2026

Hidden character warning

The head ref may contain hidden characters: "\ud83d\udcbd\ud83d\udccb"
Merged

michaelkedar merged 4 commits into
google:masterfrom
michaelkedar:💽📋

Conversation

@michaelkedar

Copy link
Copy Markdown
Member

Set up goose for the core vulnerability tables (relating to worker and API)

  • Delete references to experimental db from most places (not terraform just yet), and set API shadow rate to 0
  • Created the core PostgreSQL table files
  • Create Cloud Builder to run goose within the Cloud SQL proxy for both test and prod/private (since it's cumbersome to do the proxy stuff for our own release)
    • Also enabled the ipv4 for the private instance so we can actually run goose without jumping through GKE hoops
  • Added some documentation of new required things for development.

@another-rex another-rex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Learnt way too much about Postgresql in the last few hours.

Comment thread deployment/README.md Resolved
Comment thread deployment/deploy-prod.yaml Resolved
Comment thread go/internal/database/postgres/models.go Resolved
Comment thread docker/goose/entrypoint.bash Resolved
Comment thread docker/goose/entrypoint.bash Resolved

@another-rex another-rex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥🔥🔥🔥

@michaelkedar
michaelkedar merged commit 942b1c1 into google:master Nov 9, 2026
23 of 24 checks passed
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