Make Gemini create its own DB tables

[jonathangreen]

For the most part when I'm installing a web app, I expect to be able to point it at an empty DB, and it till do what it needs and create its own tables. This is not the case with Gemini, and it makes supporting Postgres and MySQL difficult, and makes the playbook to install longer then I think it needs to be.

I'd like to see if there is a good way to have Gemini handle the table creation internally.

Thoughts anyone?

[ajs6f]

Both MySQL and Postgres have CREATE IF NOT EXISTS, but I dunno about other dbs.

[dannylamb]

@jonathangreen Composer script to run after deployment?

[jonathangreen]

The biggest issue I had was the ID column. Postgres wants SERIAL and MySQL wants AUTOINCREMENT. I was going to see what our database layer has for DB schema support. Maybe we can do something clever within the app.

Eventually we probably want something like schema versions right? If we ever expect the tables to change at all.

[dannylamb]

@jonathangreen DBAL looks like it has cross-db tools for checking and creating tables: http://docs.doctrine-project.org/projects/doctrine-dbal/en/latest/reference/schema-manager.html

[jonathangreen]

@dannylamb yes! something like that was exactly what I was thinking. How do you feel about gemini managing its schema using schema-manager?

[dannylamb]

👍 Sounds good to me.

[DiegoPino]

@seth-shaw-unlvI will make https://github.com/Islandora-CLAW/Crayfish/blob/master/Gemini/src/Migrations/Version20180530031926.php#L13 aware of postgres instead of having 2x migration files. Too complicated without access to modify app['db'] directly from Crayfish (part of commons).

Again, something like #828 (comment)

And will integrate into composer.json script section. Wonder if you want this to run on each install or on each update?

Also: I think we need to have an extra "migrate your data if you already have stuff there" option. Probably worth another ticket.

[seth-shaw-unlv]

I've created a new Crayfish branch (https://github.com/Islandora-CLAW/Crayfish/tree/issue-828) with the postgresql changes (took a while to get that right) but I'll hold off on the PR until we have the updated playbook. I'm working on that right now with a local ansible-role-crayfish branch.

I'm trying to get ansible to run composer install and php bin/console migrations:migrate. Hopefully I'll get it working soon.

[seth-shaw-unlv]

Resolved by islandora-deprecated/ansible-role-crayfish#9