[00:25:23]
<kayou> And about the tests, I'm not sure, it's too far away in my mind, maybe it just doesn't work 🫣
[00:29:45]
<kayou> @florent:matrix.skeptikon.fr: https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/scripts%2Fconfig#L68-L133
This option is not displayed during the install because it's an edge case, a bit too techy for the YNH spirit. That's why we can only enable it on the config panel
[00:31:27]
<kayou> Probably the tests don't test the `enable_notify_push` option since https://github.com/YunoHost-Apps/nextcloud_ynh/pull/744
[01:01:26]
<Yunohost Git/Infra notifications> App pixelfedglitch failed all tests in job [#20226](https://ci-apps.yunohost.org/ci/job/20226) :(
[08:17:23]
<Yunohost Git/Infra notifications> App phpinfo rises from level 3 to 8 in job [#20235](https://ci-apps.yunohost.org/ci/job/20235) !
[08:28:27]
<Yunohost Git/Infra notifications> App rocketchat rises from level 0 to 7 in job [#20236](https://ci-apps.yunohost.org/ci/job/20236) !
[08:37:16]
<Yunohost Git/Infra notifications> App pixelfedglitch failed all tests in job [#20226](https://ci-apps.yunohost.org/ci/job/20226) :(
[11:26:00]
<florent[m]> After a quick test with a package on which I contribute, I tend to think it would not work (unless there is something that I do wrong on my side) 🙂
[11:28:04]
<florent[m]> I also have a similar issue : a component (Celery) that requires around 150Mb (on some case it does not matter, but can be lots of RAM in other cases) for a few set of features (sending emails basically).
I would like to let the user opt-in for that option. But that complexifies the package.
That's the reason why I would like to not support installation using Celery and offer the option in the config panel. But I would also like my package to be tested with that.
[11:43:52]
<Yunohost Git/Infra notifications> [apps_tools] fflorent edited [pull request #41](https://github.com/YunoHost/apps_tools/pull/41): Use editorconfig here and autopatch apps projects
[12:51:27]
<florent[m]> The `preinstall` script in package_check seems to fail erratically sometimes on my side to install a specific package (minio).
I suspect there are some ratelimit on the server serving the binary:
```
+ sudo yunohost app install minio -f -a 'domain=minio.domain.tld&init_main_permission=visitors&admin=package_checker&password=minio_password1234'
Info: Installing minio…
Info: Provisioning sources...
+ local returncode=137
+ log_debug 'Return code: 137'
+ echo 'Return code: 137'
[12:52:37]
<florent[m]> Also it looks like that takes a lot of time for the CI for the docs package, which does these steps:
https://github.com/YunoHost-Apps/docs_ynh/blob/f933b76b74524085e68254685f1f5c68771bc39d/tests.toml#L11-L19
[12:53:18]
<florent[m]> I wonder if there aren't room for speeding up things, especially by snapshoting the preinstall script
[13:11:52]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean pushed 1 commit to changelog: Add changelog bot command ([1f9c82b7](https://github.com/YunoHost/apps_tools/commit/1f9c82b74b210c9933cd1eaccab632bb848ff541))
[13:11:53]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean created new branch changelog
[13:12:15]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean opened [pull request #42](https://github.com/YunoHost/apps_tools/pull/42): Add changelog bot command
[13:13:08]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean edited [pull request #42](https://github.com/YunoHost/apps_tools/pull/42): Add changelog bot command
[13:16:57]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean [commented](https://github.com/YunoHost/apps_tools/pull/42#issuecomment-3094514422) on [issue #42](https://github.com/YunoHost/apps_tools/pull/42) Add changelog bot command: Aaaah no, body only has the first 100 characters of the comment. I guess we can remove that restriction and select onl...
[13:21:17]
<Yunohost Git/Infra notifications> [package_check] fflorent opened [pull request #194](https://github.com/YunoHost/package_check/pull/194): Fix is_webapp grep pattern
[13:22:22]
<Yunohost Git/Infra notifications> [package_check] fflorent edited [pull request #194](https://github.com/YunoHost/package_check/pull/194): Fix is_webapp grep pattern
[14:00:42]
<Yunohost Git/Infra notifications> lapineige created repository plakar_ynh: Backup solution powered by Kloset and ptar https://github.com/YunoHost-Apps/plakar_ynh
[14:05:33]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean pushed 1 commit to changelog: Detect keywords in the first 100 characters, but keep full comment for changelog and wishlist rejection ([c3087209](https://github.com/YunoHost/apps_tools/commit/c3087209a654e0384755cb4682bd2321ddedba38))
[14:06:41]
<Yunohost Git/Infra notifications> [package_check] alexAubin merged [pull request #194](https://github.com/YunoHost/package_check/pull/194): Fix is_webapp grep pattern
[14:11:57]
<Yunohost Git/Infra notifications> [package_check] alexAubin [commented](https://github.com/YunoHost/package_check/pull/192#issuecomment-3094560402) on [issue #192](https://github.com/YunoHost/package_check/pull/192) Only require a single depth while cloning the package repository: Hmmm it actually uses other commits during upgrade : https://github.com/YunoHost/package_check/blob/main/lib/tests.sh#L4...
[14:21:51]
<Yunohost Git/Infra notifications> [package_check] fflorent closed [pull request #192](https://github.com/YunoHost/package_check/pull/192): Only require a single depth while cloning the package repository
[14:21:51]
<Yunohost Git/Infra notifications> [package_check] fflorent [commented](https://github.com/YunoHost/package_check/pull/192#issuecomment-3094565904) on [issue #192](https://github.com/YunoHost/package_check/pull/192) Only require a single depth while cloning the package repository: Thats a good point I see that git fetch --depth=1 <remote> <commit> works, but the refspec for commits should be th...
[14:25:36]
<florent[m]> I am testing a patch of `package_check` that I may propose today
[14:33:17]
<Yunohost Git/Infra notifications> [mastodon_ynh] tio-trom opened [issue #505](https://github.com/YunoHost-Apps/mastodon_ynh/issues/505): Is the update process normal to take so long and so much disk space?
[15:34:47]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] alexAubin pushed 1 commit to testing: Update screenshot ([d72f2878](https://github.com/YunoHost-Apps/stirling-pdf_ynh/commit/d72f2878b011a83271356c939d21d73825b8d84e))
[15:49:17]
<eric_G> I thought the screenshot had already been updated? (in master) 🤔
[16:09:49]
<Aleks (he/him/il/lui)> ye sorry that's me just nitpicking because the jpg quality is "eh" despite that it's from the upstream repo ...
[16:09:57]
<Aleks (he/him/il/lui)> and there's an entire column of features hidden
[16:30:51]
<eric_G> nitpicking is good
[16:56:14]
<Yunohost Git/Infra notifications> Job [#20239](https://ci-apps.yunohost.org/ci/job/20239) for mattermost failed miserably :(
[16:56:33]
<Aleks (he/him/il/lui)> (restarted)
[17:09:38]
<Yunohost Git/Infra notifications> [package_check] fflorent opened [pull request #195](https://github.com/YunoHost/package_check/pull/195): Snapshot after running postinstall
[17:23:09]
<Yunohost Git/Infra notifications> [package_check] fflorent opened [pull request #196](https://github.com/YunoHost/package_check/pull/196): Fix shellcheck error in common.sh
[17:25:17]
<Yunohost Git/Infra notifications> [package_check] alexAubin merged [pull request #196](https://github.com/YunoHost/package_check/pull/196): Fix shellcheck error in common.sh
[17:26:48]
<eric_G> https://aria.im/_bifrost/v1/media/download/ASNGmPqfwZ8_QPFyK0mAP2Xzucda9XTAAGTs5Gkqg16a4lXRxdSU8Pfo_RRg-S-tjSG-qlipIklPH4E-ADWKE6dCeYLgYZOQAG1hdHJpeC5vcmcvcGhZTkpCQmRKbFlsb3h4b2VwV29QVFZE
[17:27:10]
<eric_G> ...still in V.2
[17:28:26]
<Yunohost Git/Infra notifications> [package_check] alexAubin approved [pull request #195](https://github.com/YunoHost/package_check/pull/195#pullrequestreview-3036103740) Snapshot after running postinstall
[17:29:02]
<Aleks (he/him/il/lui)> ( florent : is it really for merge ? did you test it ? 👀)
[17:29:45]
<florent[m]> I have tested it with this package : https://github.com/YunoHost-Apps/docs_ynh/blob/main/tests.toml
[17:30:32]
<florent[m]> Quite hard to tell whether it brings regression. If you have some idea how to do that correctly, I will be pleased to follow it.
[17:30:50]
<Aleks (he/him/il/lui)> lgtm
[17:30:51]
<Aleks (he/him/il/lui)> lets goo
[17:31:01]
<Yunohost Git/Infra notifications> [package_check] alexAubin merged [pull request #195](https://github.com/YunoHost/package_check/pull/195): Snapshot after running postinstall
[17:31:58]
<Yunohost Git/Infra notifications> [package_check] fflorent [commented](https://github.com/YunoHost/package_check/pull/195#discussion_r2217896512) on pull request #195 Snapshot after running postinstall: @alexAubin Just for the record, I have remove this line as it sounds very useless to me (below we restaure another snaps...
[17:32:54]
<florent[m]> Aleks (he/him/il/lui): FYI: https://github.com/YunoHost/package_check/pull/195/files#r2217896512
Also another question: are snapshot removed with the containers in LXC / incus? I am not 100% sure of that.
[17:34:58]
<Aleks (he/him/il/lui)> hmmm yeah they should ?
[17:35:14]
<Aleks (he/him/il/lui)> not sure how exactly, i would guess when we remove a container the snapshots get removed too maybe
[17:36:29]
<florent[m]> Alright, I am taking a close look to be sure and come back if the answer interests you too
[17:39:04]
<Aleks (he/him/il/lui)> i would be quite surprised if they didn't get removed somehow, that would fill up space on the CI pretty quickly 😅
[17:44:22]
<florent[m]> Yeah, sounds like `lxc delete {container}` does the job. Quoting this page:
https://www.cyberciti.biz/faq/delete-container-with-lxc-lxd-command-on-linux/
> WARNING: The following commands will delete the LXD instance/container including all snapshots. Always back up all critical data, including LXD containers, as this action is irreversible. See how to backup and restore LXD instance for more info.
Where the commands mentioned includes `lxc delete {container}`.
It should be removed by `LXC_RESET` which is invoked right before creating recreating container.
[17:57:12]
<miro5001> ```
[resources.data_dir]
dir="/home/yunohost.multimedia/__ADMIN__/__APP__"
```
Does not work as I thought.
How can I define the data_dir to be in __APP__ folder in yunohost.multimedia?
[17:58:51]
<Aleks (he/him/il/lui)> :|
[17:58:56]
<Aleks (he/him/il/lui)> god no plz no
[17:59:12]
<Aleks (he/him/il/lui)> what's your actual goal
[18:02:53]
<miro5001> https://github.com/YunoHost-Apps/directorylister_ynh/pull/33
In directorylister it is now possible to define files path
I wanted this to be accessible by other apps like nextcloud.
[18:05:38]
<miro5001> In the current package, files path is in the current folder which is install_dir
[18:06:08]
<miro5001> Every update wipes everything
[18:06:19]
<miro5001> And uploading files is not eady
[18:06:26]
<miro5001> Easy
[18:06:33]
<Aleks (he/him/il/lui)> have you looked into the "classical" multimedia helpers ?
[18:06:42]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #3051](https://github.com/YunoHost/apps/pull/3051): Add mautrix-googlechat to wishlist
[18:06:43]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #3051](https://github.com/YunoHost/apps/pull/3051): Add mautrix-googlechat to wishlist
[18:08:02]
<miro5001> > <@Alekswag:matrix.org> have you looked into the "classical" multimedia helpers ?
I didn't. That's amazing
[18:10:00]
<miro5001> Can dest_dir be `$admin/$app` ?
[18:10:46]
<Aleks (he/him/il/lui)> dest_dir ?
[18:12:07]
<Aleks (he/him/il/lui)> (the whole story about this "multimedia" thing is still confusing, it was designed like idk almost 10 years ago and hasnt been very polished or whatever, to me it's not clear what exactly we want, what are the use cases etc ... imho there's definitely work that's needed around here, but at least the helpers exists so there's a standard-ish thing)
[18:12:27]
<miro5001> In
```
ynh_multimedia_addfolder --source_dir="source_dir" --dest_dir="dest_dir"
```
[18:13:34]
<Aleks (he/him/il/lui)> typically apps do something like `ynh_multimedia_addfolder --source_dir="$data_dir/completed" --dest_dir="share/Torrents"` (for example for a torrent-related app)
[18:13:54]
<Aleks (he/him/il/lui)> in fact there's only about 5 apps using those helpers...
[18:14:12]
<Aleks (he/him/il/lui)> ```
airsonic/scripts/install:34:ynh_multimedia_addfolder --source_dir="/home/yunohost.app/$app/Podcasts" --dest_dir="share/Podcasts"
airsonic/scripts/install:35:ynh_multimedia_addfolder --source_dir="/home/yunohost.app/$app/Playlists" --dest_dir="share/Playlists"
koel/scripts/install:24:ynh_multimedia_addfolder --source_dir="$data_dir" --dest_dir="$app"
qbittorrent/scripts/install:37:ynh_multimedia_addfolder --source_dir="$data_dir" --dest_dir="share/Torrents"
retroarch/scripts/install:40:ynh_multimedia_addfolder --source_dir="$install_dir/assets/cores/Game" --dest_dir="/share/Game"
transmission/scripts/install:70:ynh_multimedia_addfolder --source_dir="$data_dir" --dest_dir="share/Torrents"
transmission/scripts/install:73:ynh_multimedia_addfolder --source_dir="$data_dir/completed" --dest_dir="share/Torrents"
transmission/scripts/install:76:ynh_multimedia_addfolder --source_dir="$data_dir/watched" --dest_dir="share/Torrent to download"
```
[18:15:44]
<miro5001> I'm using multimedia folder a lot. In fact, in nextcloud, I only use multimedia and shared multimedia folders.
[18:16:40]
<miro5001> Anyway, that's a good way to go with directorylister
[18:18:52]
<Aleks (he/him/il/lui)> the system should probably be turned into an app resource maybe, or properties for the $data_dir idk, i'm really not familiar with how it's used in real life x_x
[18:20:07]
<Aleks (he/him/il/lui)> or even something internal to yunohost like a "global setting" thingy idk
[18:27:24]
<florent[m]> I look like to have introduced a regression due to this change (though not a grave one I think):
https://github.com/YunoHost/package_check/pull/195/files#diff-4410220b236057e5f3880b07bec41e32c66e5afd10b6688a6250d3fea6b45785L211-L212
The biggest effect is that curl_tests pass due to an error.
I am testing a fix
[18:27:40]
<florent[m]> (sorry for that)
[18:49:31]
<Yunohost Git/Infra notifications> [package_check] fflorent opened [pull request #197](https://github.com/YunoHost/package_check/pull/197): Fix wrong assignment leading to error
[18:50:31]
<Yunohost Git/Infra notifications> [package_check] alexAubin approved [pull request #197](https://github.com/YunoHost/package_check/pull/197#pullrequestreview-3036123751) Fix wrong assignment leading to error
[18:52:26]
<florent[m]> Aleks (he/him/il/lui): My PR is still in draft. If you are confident, you may merge, but otherwise you may wait a few minutes, I am qualifying it.
[18:55:32]
<Yunohost Git/Infra notifications> Job [#20241](https://ci-apps.yunohost.org/ci/job/20241) for grr failed miserably :(
[18:55:53]
<Aleks (he/him/il/lui)> (restarted)
[18:59:23]
<Yunohost Git/Infra notifications> App grr failed all tests in job [#20241](https://ci-apps.yunohost.org/ci/job/20241) :(
[19:00:00]
<Aleks (he/him/il/lui)> (guessing that's due to the current regression)
[19:01:16]
<Yunohost Git/Infra notifications> [package_check] fflorent just made [pull request #197](https://github.com/YunoHost/package_check/pull/197) ready for review: Fix wrong assignment leading to error
[19:01:16]
<florent[m]> Aleks (he/him/il/lui): The tests are conclusive
[19:01:17]
<Yunohost Git/Infra notifications> [package_check] alexAubin merged [pull request #197](https://github.com/YunoHost/package_check/pull/197): Fix wrong assignment leading to error
[19:01:29]
<Yunohost Git/Infra notifications> Job [#20241](https://ci-apps.yunohost.org/ci/job/20241) for grr failed miserably :(
[19:37:38]
<Yunohost Git/Infra notifications> [apps_tools] Josue-T [commented](https://github.com/YunoHost/apps_tools/pull/41#discussion_r2217932081) on pull request #41 Use editorconfig here and autopatch apps projects: In most of my apps I have 4 space indentation in nginx config so I suppose the nginx config indentation seem not really ...
[19:43:02]
<Yunohost Git/Infra notifications> [apps_tools] fflorent [commented](https://github.com/YunoHost/apps_tools/pull/41#discussion_r2217933611) on pull request #41 Use editorconfig here and autopatch apps projects: The less exceptions there are, the better for me. So OK for 4-space indentations in Nginx.
[19:58:38]
<Yunohost Git/Infra notifications> [package_check] fflorent opened [pull request #198](https://github.com/YunoHost/package_check/pull/198): Improve pre* scripts invocation
[20:17:48]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean pushed 1 commit to changelog: Include new lines for better changelog formatting ([bed1c244](https://github.com/YunoHost/apps_tools/commit/bed1c244b45625ffff3ffd43fb389607dcee6e3b))
[20:28:53]
<Yunohost Git/Infra notifications> [package_check] fflorent just made [pull request #198](https://github.com/YunoHost/package_check/pull/198) ready for review: Improve pre* scripts invocation
[20:29:31]
<Yunohost Git/Infra notifications> [package_check] fflorent [commented](https://github.com/YunoHost/package_check/pull/198#issuecomment-3094761166) on [issue #198](https://github.com/YunoHost/package_check/pull/198) Improve pre* scripts invocation: Run successfully on this package: https://github.com/YunoHost-Apps/docs_ynh
[20:50:00]
<Yunohost Git/Infra notifications> [package_check] alexAubin merged [pull request #198](https://github.com/YunoHost/package_check/pull/198): Improve pre* scripts invocation
[20:51:03]
<Paprika> > <@miro5001:matrix.org> I am maintaining indico which requires python 3.12, installed via a custom script. But I think immich also uses a different python version using uv
Thank you so much for your input there. After inspecting immich and indico, I managed to package Wizarr for YunoHost. Truly great help man.
[22:25:05]
<Yunohost Git/Infra notifications> [piped_ynh] yunohost-bot opened [pull request #220](https://github.com/YunoHost-Apps/piped_ynh/pull/220): Upgrade sources
[22:26:44]
<Yunohost Git/Infra notifications> [searxng_ynh] yunohost-bot opened [pull request #410](https://github.com/YunoHost-Apps/searxng_ynh/pull/410): Upgrade to v2025.07.19
[22:32:23]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 39 pending update PRs
- 22 new apps PRs
- 9 failed apps updates: autobd, haste, homebox, khatru-pyramid, omeka-s, openemr, prestashop, pterodactyl, teddit
See the full log here: https://paste.yunohost.org/raw/xayodicixo
Autoupdate dashboard: https://apps.yunohost.org/dash?filter=autoupdate
[22:33:54]
<miro5001> > <@botagiuks:tiesiog.lt> Thank you so much for your input there. After inspecting immich and indico, I managed to package Wizarr for YunoHost. Truly great help man.
The most important part is not to break the default python version
[22:38:10]
<miro5001> > <@Alekswag:matrix.org> typically apps do something like `ynh_multimedia_addfolder --source_dir="$data_dir/completed" --dest_dir="share/Torrents"` (for example for a torrent-related app)
On my server it runs smoothly :
https://paste.yunohost.org/raw/dafunujula
but on the Ci I get :
```
setfacl -RnL -m g:multimedia:rwX,g::rwX,o:r-X /home/yunohost.app/directorylister
14797 INFO WARNING - setfacl: Option -m: Invalid argument near character 3
```
https://ci-apps-dev.yunohost.org/ci/job/11283
[23:00:13]
<Yunohost Git/Infra notifications> Job [#20246](https://ci-apps.yunohost.org/ci/job/20246) for ihatemoney failed miserably :(
[23:30:07]
<Yunohost Git/Infra notifications> App phpinfo rises from level 3 to 8 in job [#20249](https://ci-apps.yunohost.org/ci/job/20249) !