[10:52:39]
<Yunohost Git/Infra notifications> [webhooks] orhtej2 [commented](https://github.com/YunoHost/webhooks/pull/10#discussion_r3566149808) on pull request #10 Generate MOST_POPULAR_APPS based on catalog contents: I mean this makes the situation not worse than before :P OTOH yeah, periodical regeneration makes a lot of sense, will ...
[10:52:39]
<Yunohost Git/Infra notifications> [webhooks] orhtej2 [commented](https://github.com/YunoHost/webhooks/pull/10#discussion_r3566148922) on pull request #10 Generate MOST_POPULAR_APPS based on catalog contents: last time I checked it was unreferenced
[12:13:52]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to backup-tar-rework: backup: Drop custom tarfile bufsize ([369068da](https://github.com/YunoHost/yunohost/commit/369068da170f0b67084ffb062747be84c96e27fa))
[12:19:18]
<Yunohost Git/Infra notifications> [yunohost] kay0u [commented](https://github.com/YunoHost/yunohost/pull/2321#issuecomment-4951148429) on [issue #2321](https://github.com/YunoHost/yunohost/pull/2321) backup: Rework tar archive reading (single pass) and compress with pigz: In order to test my code, I did that on my ynh-dev: ----------------------------------------- # Backup bash root@...
[12:33:36]
<kayou> I am currently working on zstd compression for backups, as gzip is slow and less efficient
[12:33:36]
<kayou> but this PR can be reviewed and merged like that
[12:35:22]
<kayou> and if someone can review my SSE PR also, at least the #2291, because my cheap backup server crash without it, due to a lack of ram/slow SD card io...
[12:49:29]
<orhtej2> > <@kayou:matrix.org> I am currently working on zstd compression for backups, as gzip is slow and less efficient
What about pigz? Still slow?
[12:49:30]
<kayou> It's way faster with pigz but still, if I remember well, zstd is better for both compression speed and space
[12:53:56]
<kayou> Random article https://www.redpill-linpro.com/techblog/2024/12/18/compression-tool-test.html
TLDR, zstd is better for compression speed, size and decompression speed. It uses more RAM though
[13:05:44]
<orhtej2> https://linuxreviews.org/Comparison_of_Compression_Algorithms bzip2 looks like a reasonable tradeoff
[13:05:45]
<kayou> compression 19 is hardcore for zstd :P
[13:07:17]
<kayou> Anyway, we could also give the users the choice at a cost of a few lines of code
[13:09:20]
<kayou> It seems not a huge rework
[13:43:38]
<Salamandar> yes, a slight benchmark on some files of paste.yunohost.org showed me that 4 seems like the best compromise
[13:43:38]
<Salamandar> Yes, we should also configure zstd for logrotate
[13:57:43]
<orhtej2> Thing is data_dir is mostly blobs
[13:57:44]
<Salamandar> yeah, that's for sure…
[14:04:47]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to backup-zstd: backup: Add zstd as a backup compression option ([331325c6](https://github.com/YunoHost/yunohost/commit/331325c6a0d3997809cd6e1153308bb48500d366))
[14:05:17]
<Yunohost Git/Infra notifications> [yunohost] kay0u opened [pull request #2322](https://github.com/YunoHost/yunohost/pull/2322) : WIP backup: Add zstd as a backup compression option
[14:06:41]
<Yunohost Git/Infra notifications> [yunohost] kay0u converted to draft the [pull request #2322](https://github.com/YunoHost/yunohost/pull/2322) : WIP backup: Add zstd as a backup compression option
[15:00:12]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to backup-zstd: backup: Add setting migration and tests for compressed backups ([3899af47](https://github.com/YunoHost/yunohost/commit/3899af4708c568aa555e8af50b0fc5ac2a4f0997))
[15:00:12]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to backup-zstd: backup: Add a zstd_small compression level for smaller archives ([e3d7f90b](https://github.com/YunoHost/yunohost/commit/e3d7f90bcea742b0db088e6efd14820c2394085e))
[15:00:19]
<Yunohost Git/Infra notifications> [yunohost] kay0u just made [pull request #2322](https://github.com/YunoHost/yunohost/pull/2322) ready for review: WIP backup: Add zstd as a backup compression option
[15:00:30]
<Yunohost Git/Infra notifications> [yunohost] kay0u edited [pull request #2322](https://github.com/YunoHost/yunohost/pull/2322) : backup: Add zstd as a backup compression option
[15:02:57]
<Yunohost Git/Infra notifications> [yunohost] kay0u edited [pull request #2322](https://github.com/YunoHost/yunohost/pull/2322) : backup: Add zstd as a backup compression option
[15:07:26]
<Yunohost Git/Infra notifications> [yunohost] kay0u edited [pull request #2321](https://github.com/YunoHost/yunohost/pull/2321) : backup: Rework tar archive reading (single pass) and compress with pigz
[15:10:37]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to backup-zstd: backup: Add setting migration and tests for compressed backups ([f8fda291](https://github.com/YunoHost/yunohost/commit/f8fda291f3656abc4fc27acec7f3fcee4e714179))
[15:11:22]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to backup-zstd: backup: Add setting migration and tests for compressed backups ([2d66c2a4](https://github.com/YunoHost/yunohost/commit/2d66c2a45613335737f42355b2c080ac88d0f07c))
[15:59:21]
<kayou> gitlab runner updated, but it seems that it fails to start new jobs now :/
[15:59:23]
<kayou> https://gitlab.com/YunoHost/yunohost/-/jobs/15304606405
[16:05:22]
<Yunohost Git/Infra notifications> [yunohost-ci] kay0u pushed to main: Use image info instead of image list, broken since incus 7.x ([4ef77a38](https://github.com/YunoHost/yunohost-ci/commit/4ef77a3877891ac032589d695d5b54236c639d3c))
[16:07:15]
<kayou> fixed :)