[11:36:37]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/icfFPlHfNGvxZzQeNrePHdSg
[11:36:37]
<Salamandar> %3D%3D
[11:36:38]
<Salamandar> hmmm what is you guys' impression on `ynh_app_setting_set foo=$foo_value`
[11:36:38]
<Salamandar> Hmmmmm no nevermind that's not the issue. But I again had SSO redirecting me to the apps list instead of the queried app
[11:36:39]
<Aleks (he/him/il/lui)> i'm wondering about a shorter name too like uh idk `ynh_set` or idk
[11:36:39]
<Aleks (he/him/il/lui)> yeah i was wondering too about reducing the goddamn boring syntax `--key=key --value=value`
[11:36:39]
<Aleks (he/him/il/lui)> (but then that's quite a breaking change even for helpers2.1 which are now release ... except if we do some trickery to detect that the arg doesnt start with `--` or idk)
[11:36:40]
<Salamandar> yeah idk that was just a though for helpers v2.2
[11:36:41]
<Yunohost Git/Infra notifications> [custom-debian-iso] alexAubin pushed 1 commit to main: Yolotweak the grub.cfg to try to fix scenario on some machine booting on Grub (I guess?) instead of isolinux (? i have n... ([2b8511ff](https://github.com/YunoHost/custom-debian-iso/commit/2b8511ffe8917e9b34cff35e1d647c59413ff4bc))
[11:43:47]
<Yunohost Git/Infra notifications> [appsourcesautoupdate] App sources auto-update failed miserably
[11:55:03]
<Yunohost Git/Infra notifications> [yunohost-admin] Axolotle pushed 5 commits to vue3 ([be8b077586ed...c294b1708e6b](https://github.com/YunoHost/yunohost-admin/compare/be8b077586ed...c294b1708e6b))
[11:55:03]
<Yunohost Git/Infra notifications> [yunohost-admin/vue3] fix: handle api not responding errors - axolotle
[11:56:19]
<Yunohost Git/Infra notifications> [yunohost-admin] Axolotle pushed 1 commit to vue3: fix: update router key in router after hook ([7a8061e2](https://github.com/YunoHost/yunohost-admin/commit/7a8061e201efc969d7b2f8d72a197eebffade825))
[12:28:38]
<Yunohost Git/Infra notifications> [yunohost-portal] yunohost-bot opened [pull request #13](https://github.com/YunoHost/yunohost-portal/pull/13): Translations update from Weblate
[12:31:25]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> Was having a server available in the infra for contributors to test apps discussed at Yunocamp?
sort of yes, i was thinking about having sandbox machines for the app group on the new server
[12:45:11]
<Yunohost Git/Infra notifications> [yunohost] github-actions[bot] created new branch ci-auto-update-n-v10.0.0
[12:45:11]
<Yunohost Git/Infra notifications> [yunohost] github-actions[bot] pushed 1 commit to ci-auto-update-n-v10.0.0: Update n to v10.0.0 ([25862b69](https://github.com/YunoHost/yunohost/commit/25862b690497f4af17e5297d01309e0a139febb8))
[12:46:57]
<Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1442618050](https://gitlab.com/YunoHost/yunohost/-/pipelines/1442618050) failed on branch ci-auto-update-n-v10.0.0
[13:15:16]
<benc> la doc aussi?
[13:15:16]
<benc> Le site web est en panne (yunohost.org)
[13:15:16]
<benc> Salut,
[13:15:22]
<Aleks (he/him/il/lui)> zblerg
[13:15:23]
<Aleks (he/him/il/lui)> chez moi ça marche
[13:15:25]
<benc> ca remarche
[13:35:41]
<Yunohost Git/Infra notifications> [yunohost] github-actions[bot] pushed 1 commit to ci-auto-update-n-v10.0.0: Update n to v10.0.0 ([6b1bc17f](https://github.com/YunoHost/yunohost/commit/6b1bc17f8237211ad953aa4f4d6ebf9e72d37018))
[13:36:41]
<Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1443977858](https://gitlab.com/YunoHost/yunohost/-/pipelines/1443977858) failed on branch ci-auto-update-n-v10.0.0
[13:42:06]
<Aleks (he/him/il/lui)> jesus fucking christ please no python helper ffs -_-
[13:42:14]
<Salamandar> why not ?
[13:42:15]
<Salamandar> but i know the debate is on python version installer helper
[13:42:17]
<Salamandar> Hmmm i'm wondering about how we could handle dependencies on nodejs, go, ruby, (rust?), python specific versions.
For now go has a way to know which versions are used and thus cleanup unused versions.
Nodejs has a buggy `ynh_app_version` flag file (that is full of duplicates on my server…)
If we want python helpers we need that too
Could we leverage Apt to handle that ?
[13:42:18]
<Salamandar> i mean we should at least have python venv helpers
[13:42:20]
<Salamandar> well idk
[13:42:20]
<Salamandar> the current `ynh_nodejs_remove` is completely buggy
[13:42:21]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/GAKUCFxRRViujTEAsyBAfiJA
[13:42:32]
<eric_G> will it make sense to rename `ynh_nodejs_remove` -> ynh_remove_nodejs`? (same for Go and Ruby...)
[13:42:32]
<eric_G> will it make sense to rename `ynh_nodejs_remove` -> `ynh_remove_nodejs`? (same for Go and Ruby...)
[13:42:33]
<Salamandar> anyways my current question is more about the already existing helpers ^^
[13:45:52]
<eric_G> when using `--wait_until`, what log path is the helper looking for?
[13:45:57]
<eric_G> we don't have log_path set https://github.com/YunoHost/example_ynh/blob/09ce2ea730029839546f8f61703937b0dd51f306/scripts/install#L190
[13:46:00]
<eric_G> a yes, default to `/var/log/$app/$app.log` as per doc
[13:46:06]
<eric_G> any thought on `ynh_maintenance_mode_ON` helper? usefull?
[13:49:37]
<Salamandar> Or maybe with a "localhost" nginx server config
[13:49:37]
<Salamandar> I'm afraid about this breaking local curl configuration...
[13:49:37]
<Salamandar> > <@Salamandar:matrix.org> I'm afraid about this breaking local curl configuration...
Maybe with this kind of tricks https://serverfault.com/a/380645
We could allow requests from localhost for local curl but prevent remote requests ?
[13:49:37]
<Salamandar> But TBH I like this...
[13:49:38]
<Salamandar> > <@ericg:matrix.org> any thought on `ynh_maintenance_mode_ON` helper? usefull?
We discussed this point with Aleks, he's not really a fan. In any case he's for moving this into the core to have a uniform behaviour.
[13:49:44]
<Aleks (he/him/il/lui)> are we talking about the upgrade procedure ?
[13:49:48]
<Aleks (he/him/il/lui)> it's not even clear to me what it is supposed to achieve
[13:49:48]
<Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1443695800](https://gitlab.com/YunoHost/yunohost/-/pipelines/1443695800) failed on branch dev, ci-auto-update-n-v10.0.0
[13:49:51]
<Salamandar> Where we move the real config to temporarily
[13:50:13]
<eric_G> I guess it's about big instances, to prevent user losing their files during app updates...
[13:51:28]
<Salamandar> > <@Alekswag:matrix.org> it's not even clear to me what it is supposed to achieve
tl;dr make the app unaccessible to users during procedures
[13:57:59]
<eric_G> I am trying to install a node app. I guess the dev branche didn't get the patch that fixes this issue
[13:57:59]
<eric_G> ```
sudo --preserve-env=PATH -u scratch npm install
sudo: npm: command not found
```
[13:58:49]
<eric_G> -> https://forum.yunohost.org/t/why-file-permission-fix-needed-in-restore/31048/8
It might be a coincidence, but I've seen quite a few permission issues on data_dir on the restore script in the CI
[13:58:54]
<eric_G> cf https://ci-apps-dev.yunohost.org/ci/job/19168
[13:58:56]
<eric_G> this is with helpers_2.1
[13:58:56]
<eric_G> https://ci-apps-dev.yunohost.org/ci/job/19132
[14:04:44]
<eric_G> c'esttoutpété https://ci-apps-dev.yunohost.org/ci/ 🔥
[14:37:39]
<Aleks (he/him/il/lui)> not sure what happened, restarted incus and restarted the jobs
[15:27:49]
<eric_G> another data_dir permission issue on restore for Peertube: https://ci-apps-dev.yunohost.org/ci/job/19198
[16:42:03]
<orhtej2> what's the Forgejo support status as source of packages? I mean I remember struggling installing from Gitea branches due to naming, is that different (more aligned with Github perhaps?) in Forgejo?
[16:48:38]
<anubister> > what's the Forgejo support status as source of packages? I mean I remember struggling installing from Gitea branches due to naming, is that different (more aligned with Github perhaps?) in Forgejo?
I dont now but maybe you can check this app which seems successfull with Forgejo ? https://codeberg.org/flohmarkt/flohmarkt_ynh
[16:51:47]
<orhtej2> Even better, I can try any app: https://git.yunohost.org/YunoHost-Apps ;)
[16:52:57]
<orhtej2> gg rip `YunoHost app repository URLs are expected to look like https://domain.tld/path/to/repo\_ynh`
[16:53:11]
<orhtej2> thanks element for mangling the formatting
[16:53:39]
<orhtej2> still, impossible to use custom branch, will have to pick up where I left off 2 years ago ;)
[17:02:26]
<Yunohost Git/Infra notifications> [issues] audryhome opened [issue #2439](https://github.com/YunoHost/issues/issues/2439): Error 504 Gateway timeout when updating yunohost
[17:02:26]
<Yunohost Git/Infra notifications> [issues] audryhome labeled :space_invader: bug on [issue #2439](https://github.com/YunoHost/issues/issues/2439): Error 504 Gateway timeout when updating yunohost
[17:06:10]
<Yunohost Git/Infra notifications> [issues] audryhome [commented](https://github.com/YunoHost/issues/issues/2439#issuecomment-2336755531) on [issue #2439](https://github.com/YunoHost/issues/issues/2439) Error 504 Gateway timeout when updating yunohost: While updating Yunohost through WebAdmin After a while, get Error 504 Gateway timeout Attached screen copy [image](h...
[17:24:03]
<Aleks (he/him/il/lui)> ?
[17:24:04]
<Aleks (he/him/il/lui)> @_@
[17:24:04]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> cf https://ci-apps-dev.yunohost.org/ci/job/19168
i dont get it, in https://github.com/YunoHost-Apps/readeck_ynh/pull/15 you re-added the chown -R for the data dir but there are still perm issues
[17:24:05]
<eric_G> I know
[17:24:05]
<eric_G> crazy right?
[17:24:27]
<eric_G> and this occurring in various apps
[17:26:56]
<Aleks (he/him/il/lui)> watdafaq
[17:29:58]
<orhtej2> ```
2024-09-08 07:20:28,254: DEBUG - + chmod 400 /home/yunohost.app/readeck
2024-09-08 07:20:28,255: DEBUG - + chown root:root /home/yunohost.app/readeck
2024-09-08 07:20:28,256: DEBUG - + chown -R readeck:www-data /home/yunohost.app/readeck
```
you what?
[17:30:46]
<orhtej2> is the final permission/ownership valid && expected (?) `750 $app:www-data`?
[17:31:45]
<orhtej2> hmmm permissions are inherited from backup probably, so just a matter of if `$app` can `ls /home/yunohost.app/$app` and `cat` a file there I guess?
[17:35:19]
<eric_G> > <@ericg:matrix.org> and this occurring in various apps
https://ci-apps-dev.yunohost.org/ci/job/19217
[17:46:44]
<orhtej2> [helpers](https://github.com/YunoHost/yunohost/blob/2c92a644406f0cc42b2f58fbf5c6319e03259362/helpers/helpers.v2.1.d/utils#L217) yield different result than achieved with [default provisioning of `$data_dir`](https://github.com/YunoHost/yunohost/blob/dev/src/utils/resources.py#L1012) IMO (400 root:root vs 750 $app:$app), the `ynh_restore` helper is confused when restoring `$data_dir` as it's not a file within `$data_dir` nor `$install_dir`.
[17:48:54]
<orhtej2> IDK what's the solution is here as `tar` file contains UID:GID of owner that may map invalidly in new server, no clue how to validly remap :/
[18:25:56]
<Yunohost Git/Infra notifications> [issues] anubister opened [issue #2440](https://github.com/YunoHost/issues/issues/2440): fail2ban on Yunohost 12 : increase StartLimitBurst to cover app with many firewall rules ?