Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

fix/enterpriseportal: forcibly run gorm-incompatible migration in local dev#63549

Merged
bobheadxi merged 1 commit into
mainfrom
ep-force-migrate-uuid
Jun 28, 2024
Merged

fix/enterpriseportal: forcibly run gorm-incompatible migration in local dev#63549
bobheadxi merged 1 commit into
mainfrom
ep-force-migrate-uuid

Conversation

@bobheadxi

@bobheadxi bobheadxi commented Jun 28, 2024

Copy link
Copy Markdown
Member

As titled - in deployments we ran this by hand already. We use || true to make the forced migration non-blocking if it fails in case it just works, though in local testing it seems that command is idempotent anyway

See https://github.com/sourcegraph/devx-support/issues/1070

Test plan

sg run enterprise-portal
sql -d sourcegraph -c 'ALTER TABLE enterprise_portal_subscriptions ALTER COLUMN id TYPE uuid USING id::uuid;'

@bobheadxi bobheadxi requested review from a team and burmudar June 28, 2024 16:02
@cla-bot cla-bot Bot added the cla-signed label Jun 28, 2024
@bobheadxi bobheadxi merged commit c70a5c4 into main Jun 28, 2024
@bobheadxi bobheadxi deleted the ep-force-migrate-uuid branch June 28, 2024 17:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants