YunoHost-Apps / n8n_ynh

n8n package for YunoHost
https://n8n.io/
GNU General Public License v3.0
14 stars 19 forks source link

Update .env #21

Closed ericgaspar closed 2 years ago

ericgaspar commented 2 years ago

Problem

Solution

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

ericgaspar commented 2 years ago

@fflorent an env variable was missing... to be tested

fflorent commented 2 years ago

I test that.

I wonder if there are ways to migrate data from sqlite to mariadb

fflorent commented 2 years ago

Might be a good start (in french and not tested at this point): https://blog.bandinelli.net/index.php?post/2014/03/27/sqlite3-to-mysql

fflorent commented 2 years ago

I get this error:

mars 29 14:45:26 zeteo.me systemd[1]: Started n8n: Workflow Automation Tool. mars 29 14:45:31 zeteo.me n8n[14480]: Initializing n8n process mars 29 14:45:31 zeteo.me n8n[14480]: There was an error initializing DB: "The file "localhost" could not be found." mars 29 14:45:31 zeteo.me n8n[14480]: Stopping n8n... mars 29 14:45:31 zeteo.me systemd[1]: n8n.service: Main process exited, code=exited, status=1/FAILURE mars 29 14:45:31 zeteo.me systemd[1]: n8n.service: Failed with result 'exit-code'.

ericgaspar commented 2 years ago

not really surprising according to the creative var naming 😅

fflorent commented 2 years ago

Much better :100:

I attempt add a migration from sqlite to mariadb to the upgrade script.

fflorent commented 2 years ago

The above script does not help much.

I tried this one: https://github.com/techouse/sqlite3-to-mysql

It's promising, but still have an issue:

2022-03-29 15:41:12 ERROR    MySQL failed creating table credentials_entity: 1901 (HY000): Function or expression '`STRFTIME`()' cannot be used in the DEFAULT clause of `createdAt`
1901 (HY000): Function or expression '`STRFTIME`()' cannot be used in the DEFAULT clause of `createdAt

I miss time for continuing working on it today. I try to keep you informed.

fflorent commented 2 years ago

!testme

yunohost-bot commented 2 years ago

May the CI gods be with you! Test Badge

fflorent commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:sunflower: Test Badge

fflorent commented 2 years ago

@ericgaspar Since the migration of the data is not possible yet, I managed to die in case of an upgrade.

What do you think of this idea?