Skip to content

fix testdb setup to ensure you can createrole#22973

Open
se3000 wants to merge 1 commit into
developfrom
se3000/setup-testdb-createrole
Open

fix testdb setup to ensure you can createrole#22973
se3000 wants to merge 1 commit into
developfrom
se3000/setup-testdb-createrole

Conversation

@se3000

@se3000 se3000 commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Ran into this issue when running through the test setup instructions in the readme. I believe all it takes to recreate the issue was to run make testdb-force and then go test ./core/store/migrate/.

@github-actions

Copy link
Copy Markdown
Contributor

✅ No conflicts with other open PRs targeting develop

@trunk-io

trunk-io Bot commented Jun 26, 2026

Copy link
Copy Markdown

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@se3000 se3000 force-pushed the se3000/setup-testdb-createrole branch 2 times, most recently from d5a5155 to 2c38cb8 Compare June 29, 2026 14:18
The heavyweight migration tests now use pgtestdb, which provisions an
isolated role (pgtdbuser) per test run. The dev user created by this
script only had CREATEDB, so those tests failed with:

  could not create pgtestdb user: failed to create role pgtdbuser:
  ERROR: permission denied to create role (SQLSTATE 42501)

Grant CREATEROLE alongside the existing CREATEDB so a fresh setup works.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

fix test db setup to ensure you can create roles
@se3000 se3000 force-pushed the se3000/setup-testdb-createrole branch from 2c38cb8 to 558ce04 Compare June 29, 2026 14:43
@cl-sonarqube-production

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@se3000 se3000 marked this pull request as ready for review June 30, 2026 02:32
@se3000 se3000 requested review from a team as code owners June 30, 2026 02:32
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.

1 participant