[17:06:06]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Revert "..." This reverts commit 6953d958e603de85fb443ef7e229a728837d9216. ([27c4baa9](https://github.com/YunoHost/package_check/commit/27c4baa97bd2bfe73b6205fc90775e33f50b9d2f))
[17:09:55]
<kayou> What's the problem with package_check?
[17:13:20]
<Aleks (he/him/il/lui)> hmmm the general problem is that it's a huge bash soup and it's a pain to debug when something funky happens, but anyway right now i was just trying to understand / debug my stupid issues in redirect_ynh packaging v2 thinking they were related to package_check
[17:13:27]
<Aleks (he/him/il/lui)> 😬
[17:38:23]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1950](https://github.com/YunoHost/apps/pull/1950): Add AnySync to wishlist
[17:38:23]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-anysync: Add AnySync to wishlist ([61384c20](https://github.com/YunoHost/apps/commit/61384c20f31c5020c95cd260c84dcef53b7cd6f9))
[17:38:23]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch add-to-wishlist-anysync
[17:59:05]
<Yunohost Git/Infra notifications> [apps] @tituspijean [commented](https://github.com/YunoHost/apps/pull/1950#issuecomment-1874358603) on [issue #1950](https://github.com/YunoHost/apps/pull/1950) Add AnySync to wishlist: Not-totally-free components for the clients: https://tech.anytype.io/how-to/self-hosting?id=conditions
[18:09:35]
<tituspijean> https://mo8it.com/blog/quadlet/
Nice for docker-only apps? (ping jedie1024, cf. your forum thread)
[18:10:03]
<kayou> Do we have a preinstalled hello world app listening on port 1234 during the tests? O.o
[18:12:41]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/redirect_ynh/blob/version-2/scripts/_common.sh#L18
[18:39:51]
<Aleks (he/him/il/lui)> anybody will to review my mess before we merge to master ? 😬
[18:39:51]
<Aleks (he/him/il/lui)> merged to testing
[18:40:22]
<Tag> will read!
[18:40:22]
<Aleks (he/him/il/lui)> \o/
[18:41:16]
<Tag> (I'll not be able to properly test it before saturday tho')
[19:21:26]
<Charles P.> Hello, j'ai un petit soucis sur une PR. L'upstream a bump la version de PHP qui est requise à 8.3, donc j'ai set YNH_PHP_VERSION à "8.3". Maintenant, quand j'exécute ynh_add_fpm_config, la variable $phpversion reste sur "8.2" (le défaut) et PHP 8.3 ne semble pas installé. Est-ce que c'est normal ?
PR : https://github.com/YunoHost-Apps/firefly-iii-di_ynh/pull/8
Run 1 (on voit que phpversion reste à 8.2) : https://ci-apps-dev.yunohost.org/ci/job/12366
Run 2 (on voit que la 8.3 elle est pas installée) : https://ci-apps-dev.yunohost.org/ci/job/12367
[19:22:07]
<Aleks (he/him/il/lui)> tu devrais modifier ton manifest.toml tout en bas pour passer les dépendances à 8.3, ce qui passera implicitement la variable $phpversion à 8.3 etc
[19:23:16]
<Aleks (he/him/il/lui)> d'ailleurs c'est cool aussi si tu peux rajouter expliciter `php8.X-fpm` dans tes dépendances apt (nécessaire pour que l'app fonctionne bien sur bookworm)
[19:23:20]
<Charles P.> Ah oui, j'ai oublié ces dépendances, merci 😅
Est-ce que même quand elle sont à 8.3 j'ai besoin d'override YNH_PHP_VERSION ?
[19:23:26]
<Aleks (he/him/il/lui)> (ah quoique je dis ptete de la merde)
[19:23:41]
<Aleks (he/him/il/lui)> (oubli ma dernière remarque sur bookworm, je confonds je crois)
[19:24:27]
<Charles P.> dans le doute j'ajoute pas, pour laisser ma PR focus sur l'upgrade, mais je note ça dans un coin de ma tête pour une eventuelle PR pour la compat Bookworm
[19:25:16]
<Aleks (he/him/il/lui)> > <@charlesp:stratus.family> Ah oui, j'ai oublié ces dépendances, merci 😅
> Est-ce que même quand elle sont à 8.3 j'ai besoin d'override YNH_PHP_VERSION ?
hmmm la manière dont on gère PHP a changé mille fois mais en vrai je suis pas sur que y'ai encore besoin d'avoir cette variable YNH_PHP_VERSION, normalement il suffit d'utiliser `$phpversion` qui est automagiquement définie par Yunohost en fonction de tes dépendances apt (et qui s'occupe aussi de bouger l'ancienne conf PHP je sais plus comment)
[19:31:46]
<Charles P.> Ça fonctionne mieux avec les dépendances à la bonne version, au moins l'installation basique fonctionne. Plus qu'à attendre la fin du test
[19:35:44]
<eric_G> are we sure we want a warning for `--line_match="Started"`? (it brings Peertube to level 6)
[19:40:03]
<Aleks (he/him/il/lui)> ¯\_(ツ)_/¯ "Started" is matched as soon as the systemd service starts because it's contained in the default systemd message, it's misleading, it makes it looks as if we're waiting for the service to fully start when it's the opposite :| ...
[19:41:04]
<Aleks (he/him/il/lui)> and it's actually causing random issues on slow hardware, cf the thing where the LDAP plugin can't be installed because the plugin install is triggered before the service finishes to start
[19:41:40]
<Aleks (he/him/il/lui)> the most obvious thing to do would be to reapply the corresponding changes from https://github.com/YunoHost-Apps/peertube_ynh/pull/408
[19:42:07]
<Aleks (he/him/il/lui)> `--line_match="HTTP server listening on 127.0.0.1"`
[19:42:37]
<Aleks (he/him/il/lui)> (or `localhost` depending on the `hostname` parameter in `production.yaml`)
[19:43:55]
<eric_G> that I don't understand... we still get the warning with this line: https://github.com/YunoHost-Apps/peertube_ynh/blob/ddccb1c559324ebcbf35fe7255ab01de745c8a37/scripts/install#L106
[19:44:44]
<kayou> the warning is on upgrade
[19:44:44]
<kayou> https://github.com/YunoHost-Apps/peertube_ynh/pull/408
[19:44:45]
<Aleks (he/him/il/lui)> you mean on the CI ?
[19:44:47]
<eric_G> https://ci-apps-dev.yunohost.org/ci/job/12303
[19:45:37]
<Aleks (he/him/il/lui)> yeah that's from the upgrade script
[19:45:44]
<eric_G> ah yes
[19:46:42]
<eric_G> my 2024 is starting slowly 😶🌫️
[19:46:55]
<Yunohost Git/Infra notifications> Job [#21674](https://ci-apps.yunohost.org/ci/job/21674) for retroarch failed miserably :(
[19:47:28]
<Yunohost Git/Infra notifications> [apps] @alexAubin edited [pull request #1944](https://github.com/YunoHost/apps/pull/1944): Update app levels according to CI results
[20:39:22]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to update_app_levels: Update apps.toml: retroarch back to level 8 ([b7c76669](https://github.com/YunoHost/apps/commit/b7c7666920421512f43a8f9f4e0f3a4e92ff0f88))
[20:39:27]
<Yunohost Git/Infra notifications> [apps] @alexAubin edited [pull request #1944](https://github.com/YunoHost/apps/pull/1944): Update app levels according to CI results
[20:39:36]
<Yunohost Git/Infra notifications> [apps] @alexAubin approved [pull request #1944](https://github.com/YunoHost/apps/pull/1944#pullrequestreview-1800980564) Update app levels according to CI results
[20:43:14]
<Charles P.> https://aria.im/_matrix/media/v1/download/stratus.family/QRuLGjuMKDwzMeARNktcxRxX
[20:43:34]
<Charles P.> https://aria.im/_matrix/media/v1/download/stratus.family/toGkCiyeECEVFBivTebblpIA
[20:44:05]
<Charles P.> Quantum RAM, allowing negative usage
[20:44:56]
<Aleks (he/him/il/lui)> yes, not trivial to measure ram usage of an entire LXC while other LXCs are running on the same host
[21:43:54]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 15 commits to update_app_levels ([b7c766692042...2df7cb47d916](https://github.com/YunoHost/apps/compare/b7c766692042...2df7cb47d916))
[21:43:58]
<Yunohost Git/Infra notifications> [apps/update_app_levels] Merge pull request #1933 from YunoHost/add-to-wishlist-opensign Add OpenSign to wishlist - eric_G
[21:44:11]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 4 commits to master ([56862da04cbb...89053cf5c0c8](https://github.com/YunoHost/apps/compare/56862da04cbb...89053cf5c0c8))
[21:44:12]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1944](https://github.com/YunoHost/apps/pull/1944): Update app levels according to CI results
[21:44:14]
<Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch update_app_levels
[21:44:15]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml: retroarch back to level 8 - Alexandre Aubin
[21:44:35]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 3 commits to master ([89053cf5c0c8...d411ffa402ad](https://github.com/YunoHost/apps/compare/89053cf5c0c8...d411ffa402ad))
[21:44:36]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1925](https://github.com/YunoHost/apps/pull/1925): Add Vod2PodRSS to wishlist
[21:44:40]
<Yunohost Git/Infra notifications> [apps/master] Add Vod2PodRSS to wishlist - yunohost-bot
[21:54:39]
<kayou> > <@Alekswag:matrix.org> https://github.com/YunoHost-Apps/redirect_ynh/blob/version-2/scripts/_common.sh#L18
btw can't we do that in the test.toml file?
[21:56:01]
<Aleks (he/him/il/lui)> ah you mean in preinstall instructions
[21:56:34]
<kayou> yeah
[21:56:35]
<Aleks (he/him/il/lui)> tealcindeed.gif
[21:56:56]
<kayou> we can create an helloworld proxy etc
[21:56:57]
<kayou> ZZZzzz
[23:00:14]
<lapineige> > <@Alekswag:matrix.org> `--line_match="HTTP server listening on 127.0.0.1"`
Yeah I did change it to that in the past, the PR should incluse this change
[23:01:12]
<lapineige> In both install and upgrade scripts if I remember correctly