Skip to content

ADR: DBMigrationStrategy: Job-Based Schema Migration Design - #31

Open
mcruzdev wants to merge 1 commit into
kubesmarts:mainfrom
mcruzdev:migrator-app-dedicated-repo-adr
Open

mcruzdev wants to merge 1 commit into
kubesmarts:mainfrom
mcruzdev:migrator-app-dedicated-repo-adr

Conversation

@mcruzdev

@mcruzdev mcruzdev commented Aug 26, 2026

Copy link
Copy Markdown
Member

This pull request aims to proposal a solution to solve the #25.

@mcruzdev
mcruzdev marked this pull request as draft August 26, 2026 00:41
@mcruzdev
mcruzdev marked this pull request as ready for review August 26, 2026 14:41

@ricardozanini ricardozanini 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.

Additionally, we should:

  1. Create a new LogicFlowDBMigration type that will reconcile the objects to deploy the job
  2. The status field of this new CR will reflect the actual DB state (version, last modified time, etc)
  3. It should be one CR per platform/runtime since it will target specific schemas
  4. Think on how the matrix of schemas/components will work. For example, we can have a db migration per runtime, per platform, distributed accross 3 components (runtime, DI, and Quartz). Perhaps the migration strategy should be tied only to the platform, not runtime to not overengineer the solution.

Comment thread adr/implementation/db-migration-strategy.md Outdated
@mcruzdev
mcruzdev force-pushed the migrator-app-dedicated-repo-adr branch from d726822 to f812aa1 Compare August 27, 2026 21:02
@mcruzdev mcruzdev changed the title ADR: DB migration strategy ADR; DBMigrationStrategy: Job-Based Schema Migration Design Sep 1, 2026
@mcruzdev mcruzdev changed the title ADR; DBMigrationStrategy: Job-Based Schema Migration Design ADR: DBMigrationStrategy: Job-Based Schema Migration Design Sep 1, 2026
@ricardozanini

Copy link
Copy Markdown
Contributor

@mcruzdev any updates?

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