Skip to content

Fix cockroachdb failing on "Add Marketplace" migration#26467

Merged
AlexGaillard merged 10 commits intomainfrom
issue-25262
Jan 14, 2026
Merged

Fix cockroachdb failing on "Add Marketplace" migration#26467
AlexGaillard merged 10 commits intomainfrom
issue-25262

Conversation

@ComfortablyCoding
Copy link
Member

@ComfortablyCoding ComfortablyCoding commented Jan 8, 2026

Scope

What's changed:

  • Use a native approach for changing the primary key in cockroachdb
  • Refactor the changePrimaryKey function with the following changes
    • Rename columns parameter to to
    • Add input assertions
    • Add JSDoc description

Potential Risks / Drawbacks

  • Should be none

Tested Scenarios

  • Tested the migration succeeds on v25.2, v24.3, v24.1, v23.2

Review Notes / Questions

Checklist

  • Added or updated tests
  • Documentation PR created here or not required
  • OpenAPI package PR created here or not required

Fixes #25262
Fixes #23903

Copy link
Member

@br41nslug br41nslug left a comment

Choose a reason for hiding this comment

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

Done a quick test with the latest v25 of cockroach and the bootstrap is running without errors 🙌 want to run a few more tests before approving but looking good so far 🚀

Copy link
Member

@AlexGaillard AlexGaillard left a comment

Choose a reason for hiding this comment

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

🪳

@AlexGaillard AlexGaillard merged commit ea238b1 into main Jan 14, 2026
73 of 75 checks passed
@AlexGaillard AlexGaillard deleted the issue-25262 branch January 14, 2026 16:22
@github-actions github-actions bot added this to the Next Release milestone Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CockroachDB migrations stuck in infinite loop without SKIP_MIGRATIONS=true Cockroach DB 24.2.4 migrations failing with new install

3 participants