Thursday, September 26, 2024
apps@conference.yunohost.org
September
Mon Tue Wed Thu Fri Sat Sun
            1
2
3 4 5 6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
           

[03:37:04] <Yunohost Git/Infra notifications> App filebrowser goes down from level 8 to 1 in job [#30825](https://ci-apps.yunohost.org/ci/job/30825)
[06:48:38] <kayou> > <@Salamandar:matrix.org> in post_user_create, the setfacl recursive wouldn't be even worse than the current situation ?

I don't understand why it would be worse for post\_user\_create.
I have plenty of errors in the nextcloud admin panel because nextcloud can't access files in my /home while it scans files, applying such acl should fix the issue. But maybe it slows down the upgrade (like a chown -R), idk
[07:15:43] <Salamandar> > <@gredin67:matrix.fdn.fr> Salamandar: I tested your PR https://github.com/YunoHost-Apps/fab-manager_ynh/pull/126
> but the app is not able to send email
> https://github.com/YunoHost-Apps/fab-manager_ynh/issues/131#issuecomment-2307207454

hi, hmmm, tbh, i don't really remember what was blocking this PR, except that it didn't work.
[07:16:23] <Salamandar> > <@kayou:matrix.org> I don't understand why it would be worse for post\_user\_create.
> I have plenty of errors in the nextcloud admin panel because nextcloud can't access files in my /home while it scans files, applying such acl should fix the issue. But maybe it slows down the upgrade (like a chown -R), idk

worse in terms of perfs, sorry
[07:16:47] <Salamandar> like recursive chown
[07:17:11] <Salamandar> we had discussions about nextcloud upgrades taking ages (like, multiple hours) because of a recursive chown/chmod
[09:01:11] <Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar pushed 1 commit to testing: Update manifest.toml ([6cef743f](https://github.com/YunoHost-Apps/wordpress_ynh/commit/6cef743f1a14b4068e0cee1e4fad45cb9a48bf44))
[09:01:31] <Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar opened [pull request #242](https://github.com/YunoHost-Apps/wordpress_ynh/pull/242): Update manifest.toml
[09:01:33] <Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/242#issuecomment-2376369476) on [issue #242](https://github.com/YunoHost-Apps/wordpress_ynh/pull/242) Update manifest.toml: testme
[09:06:10] <Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/241#issuecomment-2376380346) on [issue #241](https://github.com/YunoHost-Apps/wordpress_ynh/pull/241) [autopatch] Automatic patch attempt for helpers 2.1: testme
[09:08:27] <Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar pushed 1 commit to testing: Update manifest.toml ([b97a41e8](https://github.com/YunoHost-Apps/wordpress_ynh/commit/b97a41e880a52fd6fc7b2d17ef6088fbd85d29c9))
[09:47:26] <orhtej2> autoupdater didn't run
[10:42:36] <Salamandar> gniii
[11:12:33] <orhtej2> (or they're dead and FF is lying to me)
[11:12:33] <Salamandar> the server provider has some issues in their DC
[11:12:33] <orhtej2> there's some mixup, https://ci-apps-dev.yunohost.org/ci/ and https://ci-apps-bookworm-dev.yunohost.org/ci/ redirect to https://ci-apps.yunohost.org/ci/
[11:12:33] <orhtej2> (the latter)
[11:12:33] <Salamandar> the server behind the dev cis is dead
[11:12:45] <Salamandar> well the server is not dead, just the ethernet links :D
[11:58:32] <eric_G> ok... What is the right permission applyed to data_dir, is it `chown -R "$app:$app" "$data_dir"` or `chown -R "$app:www-data" "$data_dir"`?
[12:12:07] <tituspijean> for which app? If you rely on NGINX to read files, you need `www-data` involved. If everything is handled by reverse proxy, then it should be `$app:$app`.
[12:15:22] <eric_G> here for the restore script in example_ynh we apply `chown -R "$app:www-data" "$data_dir"` https://github.com/YunoHost/example_ynh/blob/09ce2ea730029839546f8f61703937b0dd51f306/scripts/restore#L31C7-L31C37
[12:17:13] <orhtej2> > <@ericg:matrix.org> here for the restore script in example_ynh we apply `chown -R "$app:www-data" "$data_dir"` https://github.com/YunoHost/example_ynh/blob/09ce2ea730029839546f8f61703937b0dd51f306/scripts/restore#L31C7-L31C37

in that regard example_ynh is broken as this is not the permission $data_dir will get upon install
[12:19:02] <eric_G> should we remove `chown -R "$app:www-data" "$data_dir"` and let the helper apply the right permission?
[12:19:07] <orhtej2> https://github.com/YunoHost/yunohost/blob/ea926845391c6c29a872aaeb1da4be1c7d4dcd87/src/utils/resources.py#L1025 yup $app:$app
[12:21:19] <eric_G> `ynh_restore` is not applying any permissions is it?
[12:21:44] <eric_G> ah yes `_ynh_apply_default_permissions "$target"`
[12:21:51] <eric_G> https://github.com/YunoHost/yunohost/blob/5d15c00d921927825a0bf98b0c5d872dac57d1b7/helpers/helpers.v2.1.d/backup#L183
[16:06:07] <Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-to-wishlist-narrowlink-gate-client-agent: Add NarrowLink (Gate Client Agent) to wishlist ([2e902a36](https://github.com/YunoHost/apps/commit/2e902a36c95b5db65d727d990d71122c1259c596))
[16:06:07] <Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2599](https://github.com/YunoHost/apps/pull/2599): Add NarrowLink (Gate Client Agent) to wishlist
[16:06:10] <Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch add-to-wishlist-narrowlink-gate-client-agent
[16:06:18] <Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2599](https://github.com/YunoHost/apps/pull/2599): Add NarrowLink (Gate Client Agent) to wishlist
[17:37:59] <Yunohost Git/Infra notifications> [apps] ericgaspar pushed 6 commits to update_app_levels ([c815783dad6f^...23293ccb48f5](https://github.com/YunoHost/apps/compare/c815783dad6f^...23293ccb48f5))
[17:37:59] <Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch update_app_levels
[17:37:59] <Yunohost Git/Infra notifications> [apps] ericgaspar created new branch update_app_levels
[17:38:21] <Yunohost Git/Infra notifications> [apps] ericgaspar created new branch Add-stirling-PDF
[17:39:27] <Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to Add-stirling-PDF: Update apps.toml ([be3db913](https://github.com/YunoHost/apps/commit/be3db9133d9f087d26a75acc35df8add2724f232))
[17:39:59] <Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to Add-stirling-PDF: Update wishlist.toml ([59e4693d](https://github.com/YunoHost/apps/commit/59e4693d77e30bdcaf7f91e51ed7c957329a1a66))
[17:40:57] <Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to Add-stirling-PDF: Create stirling-pdf.png ([b6677799](https://github.com/YunoHost/apps/commit/b6677799a3aa1b3df4160d676a0c2933c946286d))
[21:37:28] <orhtej2> I feel like the catalogue is not rebuilding properly, https://apps.yunohost.org/app/nextcloud shows version `29.0.7~ynh1` [despite `29.0.7~ynh2` being pushed to master yesterday](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733)