[00:21:21]
<Yunohost Git/Infra notifications> App wireguard_client goes down from level 8 to 6 in job [#29176](https://ci-apps.yunohost.org/ci/job/29176)
[00:39:06]
<Yunohost Git/Infra notifications> App wireguard_client goes down from level 8 to 6 in job [#29178](https://ci-apps.yunohost.org/ci/job/29178)
[01:48:51]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 22 pending update PRs
- 13 new apps PRs
- 7 failed apps updates: domoticz, focalboard, plainpad, roundcube, snserver, vikunja, webtrees
See the full log here: http://paste.yunohost.org/raw/divabeyofu
[03:23:50]
<Yunohost Git/Infra notifications> Job [#29180](https://ci-apps.yunohost.org/ci/job/29180) for qr failed miserably :(
[05:55:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#discussion_r1697938911) on pull request #417 Add notify push option: Yes we should, but I still work to do on this PR. For example, this: (expand Alternatively, you can do this automatica...
[06:18:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u edited a [comment](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#discussion_r1697938911) on pull request #417 Add notify push option: Yes we should, but I still work to do on this PR. For example, this: (expand Alternatively, you can do this automatica...
[09:56:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to speedup-chown: fix find condition ([8d682cbd](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/8d682cbdbcbd1579a9c901f71429dd7354449ae8))
[09:57:11]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#issuecomment-2260130091) on [issue #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711) find files before chown them: testme
[09:57:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#issuecomment-2260130215) on [issue #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711) find files before chown them: :books: :worm:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2...
[09:57:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#issuecomment-2260130202) on [issue #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711) find files before chown them: :rocket:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/18039/badge)](ht...
[10:03:18]
<kayou> is there a way to have a button in the config panel to start an action (like the old action panel)
[10:51:58]
<tituspijean> I think so, though I cannot remember if an app actually uses that
[12:05:14]
<Aleks (he/him/il/lui)> yea
[12:07:04]
<Aleks (he/him/il/lui)> kayou: https://github.com/YunoHost-Apps/kiwix_ynh/blob/master/config_panel.toml#L17
[12:07:13]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/kiwix_ynh/blob/master/scripts/config#L11
[12:07:32]
<kayou> perfect
[12:08:07]
<kayou> I'll add this in nextcloud to chown the data dir, and remove it from the upgrade script then
[13:00:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to speedup-chown: find ... chmod is not faster, at least avoid to chown -R the data_dir in the upgrade script ([706d5071](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/706d5071c6f4e04af2050546b563400a4e2649f4))
[13:11:17]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to speedup-chown: add a button in the config panel to run chown/chmod on data_dir ([d240d552](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d240d552b47d717fd23154e5cd1eb236b5c29126))
[13:12:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u edited [pull request #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711): Avoid chown data_dir during the upgrade process
[13:13:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to speedup-chown: we can avoid this chmod too i guess? ([30b2d812](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/30b2d8123a13d315c803486d9abee3607cc2162f))
[13:13:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#issuecomment-2260496204) on [issue #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711) Avoid chown data_dir during the upgrade process: testme
[13:13:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#issuecomment-2260496317) on [issue #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711) Avoid chown data_dir during the upgrade process: :worm:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2074/badg...
[13:13:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#issuecomment-2260496299) on [issue #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711) Avoid chown data_dir during the upgrade process: :stuck_out_tongue_winking_eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/ap...
[15:27:02]
<kayou> it's working: https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711
but I need opinion on it
[15:29:31]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u edited [pull request #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711): Avoid chown data_dir during the upgrade process
[15:45:44]
<Yunohost Git/Infra notifications> [nextcloud_ynh] CodeShakingSheep [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#issuecomment-2260830715) on [issue #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711) Avoid chown data_dir during the upgrade process: Thanks for working on that. Upgrading my nextcloud from 29.0.2 to 29.0.4 ran into a Gateway timeout (HTTP 504). [image]...
[15:50:52]
<Aleks (he/him/il/lui)> >Can take up to several hours if users have a lot of data
Hmyeah and in the meantime it will hold the yunohost lock which is super meh ... I guess an improvement could be to use some trick to run the command in background using something like `systemd-run` but that's more an enhancement than a real issue
[15:56:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#discussion_r1698752914) on pull request #711 Avoid chown data_dir during the upgrade process: suggestion chown -R app:www-data "install_dir" chmod og-rwx "install_dir/config" (me being pa...
[15:56:22]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#discussion_r1698752011) on pull request #711 Avoid chown data_dir during the upgrade process: suggestion chown -R app:www-data "install_dir" chmod og-rwx "install_dir/config" chown -R app: "data_dir" (...
[16:05:18]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#issuecomment-2260870190) on [issue #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711) Avoid chown data_dir during the upgrade process: @CodeShakingSheep (? with these changes or with the new stable version?) Then just wait, and dont try to restart your s...
[17:00:50]
<Aleks (he/him/il/lui)> (another victim of the file permission thing : https://paste.yunohost.org/raw/esetisinog )
[17:01:07]
<Aleks (he/him/il/lui)> (upgraded ended up in 502 timeout and disaster etc)
[17:02:05]
<Aleks (he/him/il/lui)> `chown -R nextcloud: /var/www/nextcloud /home/yunohost.app/nextcloud` took 40 minutes, jesus christ
[17:02:47]
<Aleks (he/him/il/lui)> two times x_x
[17:03:26]
<Aleks (he/him/il/lui)> and then `chown -R nextcloud: /home/yunohost.app/nextcloud` another 40 minutes
[17:39:10]
<rodinux> I am afraid to upgrade my nextcloud instance in production...
[17:40:08]
<Salamandar> :D
[17:40:12]
<Salamandar> yeah we indeed need to fix that
[17:40:21]
<Aleks (he/him/il/lui)> well you can easily measure how long that chown takes for you by just running `chown -R nextcloud: /home/yunohost.app/nextcloud` manually ...
[19:26:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh] CodeShakingSheep [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#issuecomment-2261267897) on [issue #711](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711) Avoid chown data_dir during the upgrade process: > @CodeShakingSheep (? with these changes or with the new stable version?) Then just wait, and dont try to restart your...
[19:51:08]
<Yunohost Git/Infra notifications> [apps] MadMan247 closed [pull request #2373](https://github.com/YunoHost/apps/pull/2373): Update apps.toml for packages not maintained
[21:13:56]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#discussion_r1699113988) on pull request #711 Avoid chown data_dir during the upgrade process: Idk, do you think its not enough? https://github.com/YunoHost-Apps/nextcloud_ynh/blob/30b2d8123a13d315c803486d9abee360...
[22:14:21]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/711#discussion_r1699157452) on pull request #711 Avoid chown data_dir during the upgrade process: Eeeh chmod 640 will still allow www-data to read the file, so in the event that www-data gets compromised (or some funky...