[01:50:07]
<Yunohost Git/Infra notifications> [apps] OniriCorpe [commented](https://github.com/YunoHost/apps/pull/2538#issuecomment-2311374874) on [issue #2538](https://github.com/YunoHost/apps/pull/2538) Add LibreLingo to wishlist: i dont get why it would be relevant to self host it...
[01:51:04]
<Yunohost Git/Infra notifications> [apps] OniriCorpe labeled opinion needed on [pull request #2538](https://github.com/YunoHost/apps/pull/2538): Add LibreLingo to wishlist
[02:03:49]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 30 pending update PRs
- 9 new apps PRs
See the full log here: http://paste.yunohost.org/raw/xaqipufoze
[06:24:30]
<Yunohost Git/Infra notifications> [gitlab_ynh] yunohost-bot pushed 1 commit to testing: Merge pull request #260 from YunoHost-Apps/testing Testing ([b63972a8](https://github.com/YunoHost-Apps/gitlab_ynh/commit/b63972a8167c4114995418a9030d8091d586e446))
[06:24:30]
<Yunohost Git/Infra notifications> [gitlab_ynh] kay0u pushed 3 commits to master ([539910fdb8ca...b63972a8167c](https://github.com/YunoHost-Apps/gitlab_ynh/compare/539910fdb8ca...b63972a8167c))
[07:35:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u created new branch trying-to-fix-the-cron-task
[07:43:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to trying-to-fix-the-cron-task: cron can sometimes fail when the database is not fully initialized, retries the cron job several times and catch catch t... ([454e9f55](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/454e9f55bb4e0aa501268430d5c88e3bff321e41))
[07:43:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u just made [pull request #720](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/720) ready for review: cron can sometimes fail when the database is not fully initialized, r…
[07:43:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/720#issuecomment-2311797072) on [issue #720](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/720) cron can sometimes fail when the database is not fully initialized, r…: testme
[07:44:03]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u opened [pull request #720](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/720): cron can sometimes fail when the database is not fully initialized, r…
[08:48:32]
<selfhoster1312> sooooo with the btrfs driver incus is like 10x faster than with the dir driver, is there a reason we don't strongly recommend zfs/btrfs?
[08:48:53]
<selfhoster1312> i mean ZFS is special because you need an out-of-tree kernel driver and it may mess with the system
[08:50:14]
<selfhoster1312> but `apt install btrfs-progs` will get you the btrfs magic for free?
[08:54:59]
<selfhoster1312> it's apparently the default on incus when available, but not in LXD
[08:55:10]
<kayou> mainly because there is some limitation on btrfs, and for historical reason probably
[08:55:21]
<kayou> i still need to PR my nginx conf to be able to use incus-ui
[08:55:26]
<selfhoster1312> kayou, what's the limitation?
[08:55:37]
<kayou> swapfile
[08:55:40]
<kayou> for example
[08:56:34]
<selfhoster1312> i don't see anything about this in the incus docs... maybe it's only in LXD? or it was fixed? do you have a ticket i can follow about this?
[09:00:41]
<selfhoster1312> https://github.com/lxc/lxcfs/issues/534 was about reporting swap size, and says it was completed, but i don't see anything even in LXD that says swap should not work (with or without btrfs)
[09:01:23]
<kayou> https://wiki.archlinux.org/title/Btrfs#Swap_file
[09:01:37]
<kayou> swapfile on btrfs is not "just a swapfile" there is some extra step
[09:02:27]
<kayou> and some apps create a swapfile during installation (which is probably a bad thing)
[09:02:28]
<selfhoster1312> yeah but that's on Archlinux, « extra steps not included » ®
[09:03:27]
<selfhoster1312> ah yes
[09:03:27]
<selfhoster1312> let me try one
[09:04:50]
<selfhoster1312> hmm i see for example flarum only does it outside of package_check: https://github.com/YunoHost-Apps/flarum_ynh/blob/master/scripts/install#L65
[09:09:54]
<kayou> it's not about arch
[09:09:56]
<kayou> https://man.archlinux.org/man/btrfs.5#SWAPFILE_SUPPORT
[09:10:59]
<Yunohost Git/Infra notifications> [package_check] selfhoster1312 opened [issue #168](https://github.com/YunoHost/package_check/issues/168): Abort early if no manifest.toml
[09:11:01]
<selfhoster1312> yeah i'm trying to find an app to test it in real life conditions
[09:11:39]
<kayou> https://github.com/YunoHost-Apps/gitlab_ynh/blob/master/scripts/install
[09:12:30]
<selfhoster1312> not sure how much RAM VMs have by default let me check
[09:13:40]
<selfhoster1312> ok let's go with 2 gigs and see if it blows up :)
[09:28:57]
<selfhoster1312> hmmm it's not applied? :(
[09:29:40]
<selfhoster1312> or maybe it sees whole host memory but can't use it ?
[09:39:13]
<selfhoster1312> ok it just needed more config:
```
config:
limits.memory: 2GB
security.nesting: "true"
security.syscalls.intercept.mknod: "true"
security.syscalls.intercept.setxattr: "true"
security.syscalls.intercept.sysinfo: "true"
```
as found here: https://discuss.linuxcontainers.org/t/limits-memory-not-working/15400/10
[09:49:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to trying-to-fix-the-cron-task: oupsie ([cbed1ef7](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/cbed1ef727602c5ff5a5dce57ecdee93720de202))
[09:49:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/720#issuecomment-2312062870) on [issue #720](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/720) cron can sometimes fail when the database is not fully initialized, r…: testme
[10:34:24]
<Salamandar> > <@kayou:matrix.org> swapfile
i think not anymore
[11:53:03]
<selfhoster1312> Salamandar, actually i think kayou was right, but hopefully it's easy to workaround :)
[12:14:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 3 commits to add-notify_push-option ([ca8c2d6120ae...4c8bcebc6a2a](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/ca8c2d6120ae...4c8bcebc6a2a))
[12:14:59]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u merged [pull request #720](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/720): cron can sometimes fail when the database is not fully initialized, r…
[12:15:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u deleted branch trying-to-fix-the-cron-task
[12:28:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2312428056) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: Tested on my side, upgrade from the current version without this feature, enable notify_push via config panel, backup th...
[12:31:28]
<kayou> for the config panel, it would be way more easier to have all the key in the name of a function, for example:
https://github.com/YunoHost-Apps/nextcloud_ynh/blob/4c8bcebc6a2aacda991757702d8b8394489f6a4b/scripts/config#L96
https://github.com/YunoHost-Apps/nextcloud_ynh/blob/4c8bcebc6a2aacda991757702d8b8394489f6a4b/config_panel.toml#L29
we could set the key `main.notify_push.enabled`, and the set function `set__notify_push_enabled`, because for now, if we have another key `main.superfeature.enabled`, there is a conflict in function name
[12:32:01]
<kayou> i can't remember why it was designed like that
[12:33:15]
<Aleks (he/him/il/lui)> hmmyeah i think the thing is "option names should be unique"
[12:33:45]
<Aleks (he/him/il/lui)> ie often you'll want to repeat stuff to have like `main.notify_push.notify_push_enabled`
[12:35:38]
<kayou> yeah --'
[12:35:38]
<Aleks (he/him/il/lui)> ah yeah that's what you did, it's just called `main.notify_push.enable_notify_push`
[12:36:21]
<Aleks (he/him/il/lui)> part of the motivation for this design is that if it's not tight-coupled to the panel/section names, you can easily reorganize panel/section/options later without having to worry about the actual data key name changing
[12:41:34]
<kayou> it doesn't seem like something that happens very often
[12:41:38]
<kayou> idk
[12:42:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#discussion_r1732776630) on pull request #417 Add notify push option: suggestion Le backend Hautes Performances est une application sur Nextcloud qui devrait accélérer linstance, plus d...
[12:42:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#discussion_r1732777446) on pull request #417 Add notify push option: suggestion High Performance Backend is an application on Nextcloud that should speed up the instance, more informatio...
[12:43:02]
<Aleks (he/him/il/lui)> > <@kayou:matrix.org> it doesn't seem like something that happens very often
yes and no, we did have the case recently with domain settings for the new portal stuff
[12:43:18]
<Aleks (he/him/il/lui)> (since the config panel is shared with domain and global settings)
[12:44:08]
<Aleks (he/him/il/lui)> but yeah agreed that it easily creates confusion
[12:44:39]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to add-notify_push-option: Apply suggestions from code review Co-authored-by: eric_G <46165813+ericgaspar@users.noreply.github.com> ([4854f500](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/4854f500160527e090fa0b5004bf36956f2092c1))
[12:52:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar approved [pull request #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#pullrequestreview-2263261242) Add notify push option
[12:52:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: Add notify push option (#417) * add notify_push option * self-test notify_push * fix notify_app * new permission hpb... ([d7d24034](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d7d24034cfb81c63ca63e9ee388faba53debd39d))
[12:52:33]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar merged [pull request #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417): Add notify push option
[12:52:35]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar deleted branch add-notify_push-option
[12:52:57]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar opened [pull request #721](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/721): Testing
[12:53:53]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to v2_PostgreSQL ([291157fd435c...42a511f7ed7d](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/291157fd435c...42a511f7ed7d))
[13:07:00]
<eric_G> I upgraded Nextcloud successfully (PostgreSQL branch with High Performance Backend activated through config panel) 🎉
[13:12:06]
<eric_G> It's hard to assess the performance gain though
[13:14:08]
<kayou> > <@ericg:matrix.org> It's hard to assess the performance gain though
yeah, i used it since 2021 (right before creating the PR) and I can't tell if there is any gain
[13:14:13]
<kayou> but hey, it works^^
[13:14:40]
<kayou> did you noticed any after the psql migration?
[13:16:01]
<eric_G> yes, psql seems faster
[13:17:28]
<kayou> one day™, i'll migrate
[14:08:42]
<selfhoster1312> kayou, is it reasonable to fix ynh_add_swap to always skip when inside package_check?
[14:09:11]
<selfhoster1312> most apps already do it
[14:32:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 4 commits to testing ([d7d24034cfb8...9da7de29d740](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/d7d24034cfb8...9da7de29d740))
[14:32:47]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u merged [pull request #717](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/717): Fix removing ACL permissions on uninstall
[14:32:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u edited [pull request #721](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/721): Testing
[14:35:39]
<kayou> don't you think we need a --recursive [here](https://github.com/YunoHost-Apps/nextcloud_ynh/blob/9bef6d077bca3179a17cfc6522ccc5e682daeb54/scripts/install#L150) so each new files will get this ACL too?
[14:36:47]
<kayou> even better: https://serverfault.com/a/3035
[14:36:59]
<kayou> > setfacl --recursive --modify u:foo:rwX,d:u:foo:rwX test
[14:41:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u closed [pull request #637](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637): [enh] Faster permissions setter on big datadir
[14:41:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637#issuecomment-2312761815) on [issue #637](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637) [enh] Faster permissions setter on big datadir: closed by #711
[14:43:29]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u deleted branch enh-faster-permissions-setter
[14:51:23]
<selfhoster1312> does swapon even work in some cases in LXD?
[14:51:36]
<selfhoster1312> even with dir driver i get "Operation not permitted" (even after removing all the weird security settings i added)
[14:51:59]
<kayou> ¯\_(ツ)_/¯
[14:53:47]
<selfhoster1312> i mean i personally don't see the difference between btrfs and dir driver in this case, and i don't see a reason why we would ever want to swap in LXD
[14:55:28]
<selfhoster1312> and like i said most packages already don't do it in package_check (probably because it doesn't work):
https://github.com/YunoHost-Apps/flarum_ynh/blob/master/scripts/install#L65
[14:55:57]
<selfhoster1312> Aleks (he/him/il/lui), do you have an opinion? maybe this check should just be added to ynh_add_swap to avoid problems?
[15:05:59]
<kayou> HAAAAA
[15:06:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to testing: change x to X, add --recursive, add a default acl for new elements ([8b3220ef](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/8b3220ef8ca67fe94fbef17e9553aa71fb72a2e2))
[15:06:32]
<kayou> i push untested code in testing
[15:08:11]
<selfhoster1312> living dangerous :)
[15:09:43]
<kayou> i don't know about swap stuff, i think it's a mess
[15:18:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u created new branch fix-acl
[15:21:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u opened [pull request #722](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/722): change x to X, add --recursive, add a default acl for new elements
[15:22:28]
<kayou> fixed
[15:24:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u edited [pull request #722](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/722): change x to X, add --recursive, add a default acl for new elements
[17:03:49]
<eric_G> what is this `Waiting for status to be reported`? https://github.com/YunoHost-Apps/element_ynh/pull/183
[17:06:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to testing ([9da7de29d740...047cfa3d2a44](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/9da7de29d740...047cfa3d2a44))
[18:34:00]
<Émy - OniriCorpe> > <@ericg:matrix.org> what is this `Waiting for status to be reported`? https://github.com/YunoHost-Apps/element_ynh/pull/183
Waiting for CI results
[18:36:47]
<Émy - OniriCorpe> The CI didn’t run for the latest commit, so the PR is still waiting the CI validation
[20:18:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 7 commits to v2_PostgreSQL ([42a511f7ed7d...5f3de8b607c3](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/42a511f7ed7d...5f3de8b607c3))