Skip to content

Init EB DB using migrations instead of schema:create#77

Merged
johanib merged 4 commits intomainfrom
feature/use-engine-migrations
Feb 24, 2026
Merged

Init EB DB using migrations instead of schema:create#77
johanib merged 4 commits intomainfrom
feature/use-engine-migrations

Conversation

@johanib
Copy link
Contributor

@johanib johanib commented Feb 23, 2026

Prior to this change, the EB DB schema in devconf would deviate from the migrations (prod) database, because new changes would be applied automatically via schema:create instead of applying the migrations.

This change ensures the EB DB is bootstrapped using the database migrations to match prod.

Resolves: OpenConext/OpenConext-engineblock#1861
See: OpenConext/OpenConext-engineblock#1914

Prior to this change, the EB DB schema in devconf would deviate from the migrations (prod) database, because new changes would be applied automatically via schema:create instead of applying the migrations.

This change ensures the EB DB is bootstrapped using the database migrations to match prod.

Resolves: OpenConext/OpenConext-engineblock#1861
@johanib johanib requested a review from kayjoosten February 23, 2026 14:29
johanib and others added 2 commits February 24, 2026 14:05
Prior to this change, the script always attempted to bring up the engine and mariadb containers, regardless of their current status.

This change checks if the engine and mariadb containers are already running before attempting to start them, providing clearer messaging based on the situation.
Prior to this change, the init script would re-create some containers, but with an unexpected APP_ENV variable.
To reduce complexity, this change requires the user to run the start script, and then run the init script.

Co-authored-by: Kay Joosten <kay.joosten@dawn.tech>
@johanib johanib requested a review from kayjoosten February 24, 2026 13:46
Pulling images is slow, taking > 3 minutes. This change ensures pipeline has enough time to run, before being killed.
@johanib johanib force-pushed the feature/use-engine-migrations branch from 0877950 to 0e549a9 Compare February 24, 2026 14:09
@johanib johanib merged commit 098dcd1 into main Feb 24, 2026
1 check passed
@johanib johanib deleted the feature/use-engine-migrations branch February 24, 2026 14:17
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.

Ensure consistent database schema for migrations

2 participants