Skip to content

Conversation

migmartri
Copy link
Member

@migmartri migmartri commented Jul 6, 2023

Adds support for versioned migrations

The reason for using versioned migrations is because it enables advanced DB changes which will be required during #201. For example db table rename oci_repositories => cas_backends or non-null DB column with values set for current entries, i.e provider = OCI.

It makes it look closer to how rails migrations works although it still relies on the declarative schema from ent.

It adds an initial migration with the current DB schema

  • app/controlplane/internal/data/ent/migrate/migrations/20230706165452_init-schema.sql

Note:

We'll need to add support for automatic migrations in our Helm Chart via an init container. There is a brand new container image that will be built during release that contains the migrations.

Refs #221

migmartri added 3 commits July 6, 2023 19:00
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
migmartri added 5 commits July 7, 2023 01:20
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
@migmartri migmartri changed the title feat(controlplane): versioned migrations support enhancement(controlplane): versioned migrations support Jul 7, 2023
@migmartri migmartri merged commit bfd154b into chainloop-dev:main Jul 7, 2023
@migmartri migmartri deleted the atlas-migrations branch July 7, 2023 09:46
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