YunoHost-Apps / peertube_ynh

Federated (ActivityPub) video streaming platform using P2P for YunoHost
https://joinpeertube.org/fr/
GNU Affero General Public License v3.0
87 stars 27 forks source link

Systemd wait for dbinit #400

Closed lapineige closed 6 months ago

lapineige commented 6 months ago

https://github.com/YunoHost-Apps/peertube_ynh/pull/397#issuecomment-1870324009

Database isn't initialized completely all the time before using it (LDAP plugin installation), so we should wait for it.

Thanks @alexAubin and @Chocobozzz for you help ! 🙂

Fix #390 #386

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)

lapineige commented 6 months ago

!testme

yunohost-bot commented 6 months ago

Fingers crossed! Test Badge

lapineige commented 6 months ago

Manual testing on a Raspberry Pi 3 + HDD worked (actually it was with #399 but this should be even better). Install on CI is working (failing but as in other current tests). I believe it should improve installation on slow hardware in particular (where is less likely that the DB is initialized properly before the next install step start).

Ready for review and merge :)

lapineige commented 6 months ago

!testme

yunohost-bot commented 6 months ago

:v: Test Badge