[15:39:22]
<john> question about yunohost app vars: is there a maximum size? I'm thinking it'd be nice to have an app that'll generate an index.html, and the admin panel would just be "supply css" and "supply markdown" for a static landing page
[18:39:51]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: remove send email ([2ec2d5cf](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/2ec2d5cfabc38e80ddfa0f6783d2641576d6bbcb))
[18:39:59]
<Aleks (he/him/il/lui)> > <@titus:pijean.ovh> What I plan to do:
> ```
> #pseudocode
> if CI or personal_instance:
> source ./maintenance/finish_install.sh
> ynh_add_config... to replace tweak_yunorunner
> git clone https://github.com/YunoHost/package_check "./package_check"
> setup_lxd
> add_cron_jobs
> else:
> run ./maintenance/finish_install.sh
> ```
hmmm
[18:39:59]
<Aleks (he/him/il/lui)> also the `setup_lxd` can probably be simplified because we don't actually use the cluser config thingy
[18:58:32]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 1 commit to v2: Apply suggestions from code review ([402abc21](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/402abc21d571d6168e336cf446b7caaabf8ed54e))
[18:58:45]
<Aleks (he/him/il/lui)> > <@titus:pijean.ovh> it's hella confusing 😅
ogod
[18:58:46]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107#issuecomment-1505805376) on [issue #107](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107) V2: (rebooting job abobe ^)
[18:58:46]
<Aleks (he/him/il/lui)> we should fix it in the core ...
[18:58:46]
<Aleks (he/him/il/lui)> didn't realize this
[18:58:51]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107#issuecomment-1505805376) on [issue #107](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107) V2: (rebooting job above ^)
[18:59:03]
<tituspijean[m]> > <@Alekswag:matrix.org> we should fix it in the core ...
on it, I've the line on hand
[18:59:03]
<tituspijean[m]> mmmh... should I simply add a `settings["app"] = app` there? https://github.com/YunoHost/yunohost/blob/749f0c5b1f4b694fc562c8ad01ed540978206ea5/src/app.py#L2015
[18:59:09]
<eric_G> ^ for my_webapp
[18:59:17]
<eric_G> (I am jumping subjects)
[19:00:31]
<Yunohost Git/Infra notifications> App librarian rises from level 7 to 8 in job [#14976](https://ci-apps.yunohost.org/ci/job/14976) !
[19:00:37]
<Yunohost Git/Infra notifications> App osticket rises from level 6 to 8 in job [#14978](https://ci-apps.yunohost.org/ci/job/14978) !
[19:01:10]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/114#issuecomment-1506415630) on [issue #114](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/114) Multiple platform selection: Both mentioned platforms look like they provide their own webserver.
This is currently not in the scope of my_webapp (s...
[19:01:10]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @eldertek opened [issue #114](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/114): Multiple platform selection
[19:01:13]
<Yunohost Git/Infra notifications> App zerotier rises from level 6 to 7 in job [#14989](https://ci-apps.yunohost.org/ci/job/14989) !
[19:02:41]
<tituspijean[m]> > Identity file is missing. Please report to YunoHost Support if your ZeroTier instance does not work.
I _am_ the Support. Damnit.
[19:05:20]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: add DB info ([5b38abe3](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/5b38abe3683e49d983d6d3512cb089689ba40fe4))
[19:05:31]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: cleaning ([e2df6be8](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/e2df6be87c6c5512e8d1ce961f0e7b9cf3942d3e))
[19:06:30]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: fix ([c6619653](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/c66196538c96598f82dfe28a99edc2f015443bc6))
[19:07:12]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: Update install ([e3ede5ca](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/e3ede5ca2965e31a6af9775f7aaf3df4238197a7))
[19:07:32]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: fix ([6314f2bc](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/6314f2bc263c3c1de13c750a3e345977b9d1a2a1))
[19:08:33]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: cleaning ([67c99b38](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/67c99b38e796b0c0b349d288c23d6e0424b10d32))
[19:09:05]
<eric_G> strange... the `__APP__` variable doen't get replaced in `POST_INSTALL.md` : https://github.com/YunoHost-Apps/my_webapp_ynh/blob/v2/doc/POST_INSTALL.md 🤔
[19:09:13]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107#issuecomment-1505788377) on [issue #107](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107) V2: testme
[19:09:44]
<tituspijean[m]> > <@ericg:matrix.org> strange... the `__APP__` variable doen't get replaced in `POST_INSTALL.md` : https://github.com/YunoHost-Apps/my_webapp_ynh/blob/v2/doc/POST_INSTALL.md 🤔
ah yeah it should be `__ID__` now
[19:09:55]
<tituspijean[m]> it's hella confusing 😅
[19:11:02]
<Yunohost Git/Infra notifications> App diaspora rises from level 2 to 8 in job [#14962](https://ci-apps.yunohost.org/ci/job/14962) !
[19:11:14]
<Yunohost Git/Infra notifications> App mautrix_telegram rises from level 0 to 8 in job [#14311](https://ci-apps.yunohost.org/ci/job/14311) !
[19:11:20]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u merged [pull request #215](https://github.com/YunoHost-Apps/gitlab_ynh/pull/215): Testing
[19:12:47]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u pushed 3 commits to master ([2536a83fb77e...8836d9cfa2e2](https://github.com/YunoHost-Apps/gitlab_ynh/compare/2536a83fb77e...8836d9cfa2e2))
[19:12:49]
<Yunohost Git/Infra notifications> [gitlab_ynh/master] 15.10.2 - Kay0u
[19:13:00]
<Yunohost Git/Infra notifications> [gitlab_ynh/master] Auto-update README - yunohost-bot
[19:13:13]
<Yunohost Git/Infra notifications> [gitlab_ynh/master] Merge pull request #215 from YunoHost-Apps/testing Testing - Kayou
[19:13:53]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/113#discussion_r1164200629) on pull request #113 Update DESCRIPTION.md with info on how to use SFTP: Nitpicking but i dunno if this bit really is useful, it just explains what should be visible one scroll further up in th...
[19:14:05]
<Yunohost Git/Infra notifications> App bookwyrm rises from level 6 to 7 in job [#14968](https://ci-apps.yunohost.org/ci/job/14968) !
[19:14:12]
<tituspijean[m]> Do y'all know what is "Error: The database witnessdb is missing ! Something gone wrong !"? (in https://ci-apps-dev.yunohost.org/ci/job/6645)
[19:14:45]
<tituspijean[m]> I mean, in that context, can $app equal `nextcloud__2` for example?
[19:14:57]
<Aleks (he/him/il/lui)> it should yes, because it's the name used to retrieve the settings.yml inside /etc/yunohost/apps ... but i'm confused about the `if app == settings['id']` o_O
[19:15:04]
<Aleks (he/him/il/lui)> ogod the commit that introduced this line is 8 years old hmpf
[19:17:14]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107#issuecomment-1505805376) on [issue #107](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107) V2: (rebooting job above ^) (also being fixed upstream with https://github.com/YunoHost/yunohost/pull/1645)
[19:17:14]
<tituspijean[m]> I'll let you clean that up 😛 https://github.com/YunoHost/yunohost/pull/1645
[19:17:37]
<tituspijean[m]> > <@Alekswag:matrix.org> it should yes, because it's the name used to retrieve the settings.yml inside /etc/yunohost/apps ... but i'm confused about the `if app == settings['id']` o_O
looks like a patch for a hell of an edge case 😆
[19:17:47]
<Yunohost Git/Infra notifications> App aeneria failed all tests in job [#14957](https://ci-apps.yunohost.org/ci/job/14957) :(
[19:19:12]
<Aleks (he/him/il/lui)> Yeah :|
[19:19:14]
<Aleks (he/him/il/lui)> Could be stuff when the app is being installed or restored but the setting file aint there yet
[19:19:21]
<eric_G> > <@titus:pijean.ovh> ah yeah it should be `__ID__` now
Thanks!
[19:19:34]
<eric_G> and now, I can't figure out the `change_url` failing... https://ci-apps-dev.yunohost.org/ci/job/6742
[19:19:44]
<tituspijean[m]> `ynh_add_config --template=nginx.conf` : `The provided template nginx.conf doesn'\''t exist'`
[19:19:54]
<tituspijean[m]> we only got `nginx_no_php.conf` and `nginx_with_php.conf`
[19:21:07]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 5 commits to version-2-#101 ([e280d90298b3...109bc1aed8c2](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/e280d90298b3...109bc1aed8c2))
[19:21:33]
<Yunohost Git/Infra notifications> [my_webapp_ynh/version-2-#101] Update tests.toml - Éric Gaspar
[19:21:35]
<Yunohost Git/Infra notifications> [my_webapp_ynh/version-2-#101] remove visible password - Éric Gaspar
[19:21:43]
<Yunohost Git/Infra notifications> [my_webapp_ynh/version-2-#101] Merge branch v2 into version-2-#101 - Éric Gaspar
[19:21:46]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/115#issuecomment-1507035946) on [issue #115](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/115) Version 2 #101: testme
[19:22:43]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/115#issuecomment-1507036001) on [issue #115](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/115) Version 2 #101: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/6755/badge)](...
[19:22:52]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/115#issuecomment-1507041496) on [issue #115](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/115) Version 2 #101: Not sure if we can remove this : https://github.com/YunoHost-Apps/my_webapp_ynh/blob/109bc1aed8c2827340a1fdc016bf72d3d4b...
[19:23:03]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/187#issuecomment-1507084234) on [issue #187](https://github.com/YunoHost/example_ynh/pull/187) Rewrite updater.sh -> .py. Remove redundant info in updater.yml.: Whats the point of leaving the Github Actions ? I really like being able to manually run the update merge request gith...
[19:23:12]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/187#issuecomment-1507084234) on [issue #187](https://github.com/YunoHost/example_ynh/pull/187) Rewrite updater.sh -> .py. Remove redundant info in updater.yml.: Whats the point of leaving the Github Actions ? I really like being able to manually run the update merge request gith...
[19:23:26]
<tituspijean[m]> > <@titus:pijean.ovh> Issue reported upstream: https://gitlab.com/aeneria/aeneria-app/-/issues/35
Issue fixed, job restarted
[19:23:48]
<Aleks (he/him/il/lui)> the point of this line is that, yeah, in actual production context, we want to disable all those services to try to save up RAM etc, because yunorunner will be "alone" on the server so there's no point in having those
[19:24:34]
<tituspijean[m]> if the finish_install script disables `mysql php7.4-fpm metronome rspamd dovecot postfix redis-server postsrsd yunohost-api avahi-daemon`, we cannot do much on the server
[19:24:44]
<tituspijean[m]> yeah ok, so that's the assumption here. Maybe not in the context of yunorunner_ynh then?
[19:26:05]
<Aleks (he/him/il/lui)> yup
[19:26:11]
<tituspijean[m]> I mean, I used to have my own CI, and it definitely coexisted with other apps 😛
[19:26:14]
<Aleks (he/him/il/lui)> zblerg
[19:26:21]
<Aleks (he/him/il/lui)> in that case you can re-enable them manually but yeah idk, i added these because initially that script was ran manually and meant to deploy a production server on yunohost infra
[19:26:29]
<Aleks (he/him/il/lui)> and because you don't want to have to think again about all the tweak you gotta run everytime we redeploy a CI somewhere on our infra
[19:26:32]
<Aleks (he/him/il/lui)> i guess ultimately that could correspond to some sort of question like "context = official-infrastructure" or "personal-instance" or whatever
[19:28:51]
<Aleks (he/him/il/lui)> zblerg it's package_check creating a dummy "witnessdb" in mysql before running tests and then checking if it still does exists after the app is installed
[19:28:54]
<Aleks (he/him/il/lui)> which was meant to check against app erasing like ... everything ... though honestly nowadays it's very unlikely to happen
[19:29:03]
<Aleks (he/him/il/lui)> similar stuff are done with a dummy nginx conf file etc
[19:29:10]
<Aleks (he/him/il/lui)> hm this error seems to happen somewhat consistently though ?
[19:29:18]
<Aleks (he/him/il/lui)> ah that's yunorunner
[19:29:24]
<Aleks (he/him/il/lui)> hmmm
[19:29:33]
<Aleks (he/him/il/lui)> could maybe due to yunorunner disabling mysql maybe ?
[19:30:13]
<tituspijean[m]> I guess running `./maintenance/finish_install.sh` is messing things up
[19:30:50]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([d8ce9a8c](https://github.com/YunoHost/apps/commit/d8ce9a8c39ece56bb98dab2745173634544a1810))
[19:30:57]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1661](https://github.com/YunoHost/apps/pull/1661#pullrequestreview-1385864352) Update app levels according to CI results
[19:32:15]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1661](https://github.com/YunoHost/apps/pull/1661): Update app levels according to CI results
[19:32:15]
<Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch update_app_levels
[19:32:15]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to master ([1c3fea6e50a3...ba0882c3b9b8](https://github.com/YunoHost/apps/compare/1c3fea6e50a3...ba0882c3b9b8))
[19:32:29]
<Yunohost Git/Infra notifications> [apps/master] Update app levels according to CI results - root
[19:33:18]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1661 from YunoHost/update_app_levels Update app levels according to CI results - Éric Gaspar
[19:52:44]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @eldertek closed [issue #114](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/114): Multiple platform selection
[22:38:26]
<Yunohost Git/Infra notifications> App documize rises from level 7 to 8 in job [#15033](https://ci-apps.yunohost.org/ci/job/15033) !