search

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

Testing : 6.0.2 + packaging v2 + fix install 502 error + timeout on slow hardware + wait for DB full init #391

Closed ericgaspar closed 6 months ago

ericgaspar commented 8 months ago

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 8 months ago

!testme

yunohost-bot commented 8 months ago

May the CI gods be with you! Test Badge

alexAubin commented 8 months ago

Meh, I dont understand why it 502 yet the service seems running and no error can be found in logs x_x

ericgaspar commented 8 months ago

!testme

yunohost-bot commented 8 months ago

Alrighty! Test Badge

ericgaspar commented 7 months ago

!testme

yunohost-bot commented 7 months ago

:rocket: Test Badge

lapineige commented 6 months ago

!testme

yunohost-bot commented 6 months ago

:carousel_horse: Test Badge

lapineige commented 6 months ago

!testme

yunohost-bot commented 6 months ago

May the CI gods be with you! Test Badge

alexAubin commented 6 months ago

!testme

yunohost-bot commented 6 months ago

Meow :cat2: Test Badge

alexAubin commented 6 months ago

Sooo tests are passing, for me we're good to go :+1:

lapineige commented 6 months ago

That install didn't work on my server… I already had a report of a failed update.

A so massive change requires manual testing imho…

alexAubin commented 6 months ago

It it fails then can we have logs x_x ...

lapineige commented 6 months ago

So I tried again (on 2 instances) with your latest changes, and it works now. I don't know what fixed it.

But for someone else, it failed. Logs : https://paste.yunohost.org/raw/yecofedamu

INFO - Provisioning database...
DEBUG - Executing command '['sh', '-c', '/bin/bash -x "./upgrade"  7>&1']'
DEBUG - + source _common.sh
DEBUG - ++ nodejs_version=20
DEBUG - +++ echo peertube
DEBUG - +++ sed s/peertube//g
DEBUG - ++ db_suffix=
DEBUG - + source /usr/share/yunohost/helpers
DEBUG - +++ set +o
DEBUG - +++ grep xtrace
DEBUG - ++ readonly 'XTRACE_ENABLE=set -o xtrace'
DEBUG - ++ XTRACE_ENABLE='set -o xtrace'
WARNING - ./upgrade: line 16: admin: unbound variable
DEBUG - + admin_mail=
DEBUG - + ynh_exit_properly
DEBUG - + [[ upgrade =~ ^install$|^upgrade$|^restore$ ]]
ERROR - Could not upgrade peertube: An error occurred inside the app upgrade script
ericgaspar commented 6 months ago

I have pushed to testing some fixes for the NGINX config: __DATA_DIR__storage/ to __DATA_DIR__/storage/ I don't know if this solve issues beeing reported in the forum.

lapineige commented 6 months ago

So referencing it https://github.com/YunoHost-Apps/peertube_ynh/pull/404