[01:52:27]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 33 pending update PRs
- 17 new apps PRs
See the full log here: http://paste.yunohost.org/raw/ijegibijeq
[08:12:55]
<Yunohost Git/Infra notifications> [package_check] selfhoster1312 opened [pull request #167](https://github.com/YunoHost/package_check/pull/167): Improve the README setup to mention incus
[08:15:30]
<selfhoster1312> hmmm the usage help in the README hasn't been updated in a while
[08:15:47]
<selfhoster1312> i think it deserves a PR to make a script to keep it synchronized with the help string :) :)
[08:31:20]
<orhtej2> ci-dev seems dead
[08:33:26]
<Aleks (he/him/il/lui)> i'm rebooting the machine
[08:38:35]
<Aleks (he/him/il/lui)> it's back online
[08:54:02]
<Yunohost Git/Infra notifications> App django-for-runners rises from level 0 to 8 in job [#29990](https://ci-apps.yunohost.org/ci/job/29990) !
[09:46:43]
<Yunohost Git/Infra notifications> [package_check] selfhoster1312 [commented](https://github.com/YunoHost/package_check/pull/166#issuecomment-2309800266) on [issue #166](https://github.com/YunoHost/package_check/pull/166) Add a -v/--verbose flag to print debug output: I also updated the usage section in README with a lib/update_readme.py script.
[09:47:36]
<selfhoster1312> i added a new commit with a script to update the readme with actual usage parsed from package_check.sh
[09:48:16]
<Salamandar> Meh, readme shouldn't contain usage IMHO actually
[09:48:23]
<Salamandar> but the --verbose is a nice addition <3
[09:49:22]
<selfhoster1312> Salamandar, i got other patches to review there if you're feeling adventurous <3
[09:49:55]
<Salamandar> sorry, not the day. $dayjob is a bit awful these days
[09:50:09]
<selfhoster1312> oof, that sounds painful, sorry for you
[09:50:18]
<selfhoster1312> good luck friend <3
[09:51:06]
<Salamandar> don't worry i am fine :) i just have too many subjects on my plate
[09:52:23]
<selfhoster1312> hope they're tasty :P
[09:58:56]
<selfhoster1312> so i'm probably doing tests.toml wrong, but i get at test 2/5:
> 640 ERROR Pick a valid value for the argument 'path': Option is mandatory
[12:15:47]
<Yunohost Git/Infra notifications> [package_check] selfhoster1312 [commented](https://github.com/YunoHost/package_check/pull/165#issuecomment-2310064211) on [issue #165](https://github.com/YunoHost/package_check/pull/165) Fail startup/network timeout earlier. Exit whole test suite.: So it turns out those timeouts are too low in real life conditions... I dont understand why/how LXD can be so slow on a...
[12:15:49]
<Yunohost Git/Infra notifications> [package_check] selfhoster1312 [commented](https://github.com/YunoHost/package_check/pull/165#issuecomment-2310064211) on [issue #165](https://github.com/YunoHost/package_check/pull/165) Fail startup/network timeout earlier. Exit whole test suite.: So it turns out those timeouts are too low in real life conditions... I dont understand why/how LXC can be so slow on a...
[13:15:24]
<PapaDragon> Question
[13:15:44]
<PapaDragon> Question about the package check: did it recently change?
[13:15:50]
<PapaDragon> Or some criterias?
[13:16:43]
<PapaDragon> The Streams package was rated 8 and went down to 6, apparently because it uses git clone rather that dowloading a tarball
[13:17:10]
<PapaDragon> Previously it was just a "possible improvement", now it's a "warning"
[13:17:47]
<orhtej2> > <PapaDragon> The Streams package was rated 8 and went down to 6, apparently because it uses git clone rather that dowloading a tarball
https://github.com/YunoHost/package_linter/commit/26baf92a58e718171546bfcb316bdf036d635dae
[13:17:56]
<orhtej2> so: yes
[13:19:17]
<orhtej2> one would say you can use tagged releases? https://codeberg.org/streams/streams/tags
[13:19:50]
<orhtej2> I believe codeberg is even supported by the autoupdater
[13:21:06]
<PapaDragon> That would be nice. Is there any doco about how to have the autoupdater handlling the package?
[13:23:06]
<orhtej2> add `autoupdate.strategy = "latest_forgejo_tag"` below the asset and run autoupdater locally once I guess?
[13:23:19]
<orhtej2> cf https://github.com/YunoHost-Apps/forgejo_ynh/blob/a37cf3ed99d0a72bbceaa149bebe10c684543a83/manifest.toml#L49
[13:25:23]
<orhtej2> this may be a more concise example: https://github.com/YunoHost-Apps/digitools\_ynh/blob/c80b08bae76b5057b703b7e394a953e82606c950/manifest.toml#L46C9-L50C1
[13:26:22]
<PapaDragon> Thanks! I'll look into that. But first I'll have to figure how to switch back to the tarball dowload on the already installed websites. Should be loads of fun.
[13:30:10]
<orhtej2> > <PapaDragon> Thanks! I'll look into that. But first I'll have to figure how to switch back to the tarball dowload on the already installed websites. Should be loads of fun.
a good old `ynh_setup_source --full_replace=1` should do the trick assuming the data is kept elsewhere
[13:46:22]
<PapaDragon> > > <PapaDragon> Thanks! I'll look into that. But first I'll have to figure how to switch back to the tarball dowload on the already installed websites. Should be loads of fun.
>
> a good old `ynh_setup_source --full_replace=1` should do the trick assuming the data is kept elsewhere
Hum... I guess not. There are two folders that must be created during the install. I could probably create them in /home/yunohost.app/streams/ but then I'd have to ln -s them from the app's install folder. Does it sound safe and/or not too dirty?
[13:47:20]
<PapaDragon> Also there's a log file but it should be easy to move it elsewhere (would /var/log/ be the most suitable place?)
[13:51:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to add-notify_push-option: fix indent ([eb7d8693](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/eb7d8693aab7a757afaf5382ca52862692f550ed))
[13:52:38]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310271806) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: Ready to merge, opinion needed
[13:52:53]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310271806) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: Ready to merge, tested on my side, opinion needed
[13:53:10]
<kayou> ljf: smoll ping, if you have time to review/give me a feedback on this https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417
[13:59:47]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310289250) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: erratum, I have a warning: > systemctl status nextcloud-notify-push-watcher.path ○ nextcloud-notify-push-watcher.path -...
[14:13:18]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to add-notify_push-option: enable/disable the notify push path ([f6829db6](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/f6829db6639348b02c93094bbb8c680072be8e31))
[14:28:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to add-notify_push-option: fix path unit ([81afc1b2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/81afc1b28f8966693bab902dcf3b139c543567d6))
[14:38:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to add-notify_push-option: exec only for nextcloud user ([ca5228c5](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ca5228c5e40f46d73200de763f563297f3fe2b4d))
[14:38:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310379323) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: testme
[14:38:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310379427) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :carousel_horse:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/18524/ba...
[14:38:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310379451) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :books: :worm:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2...
[14:51:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310408009) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :worm:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2274/badg...
[14:51:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310407851) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: testme
[14:51:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to add-notify_push-option: move sock file to /var/run ([bf4e212a](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/bf4e212a13f43f37a9965bb5a9d91ae75781414f))
[14:51:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310407960) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: Meow :cat2:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/18526/badge)]...
[14:54:12]
<kayou> zomg, it was working because the .path unit wasn't enabled
[14:55:55]
<kayou> because when the .path unit is enabled, during the chown/chmod on the /var/www/nextcloud, the .path is triggered, so the /var/www/nextcloud/notify-push.sock doesn't exist anymore (the service is restared), and the chmod fails befause "/var/www/nextcloud/notify-push.sock doesn't exist"
[14:56:12]
<Aleks (he/him/il/lui)> zomg spoilers?
[14:56:18]
<Aleks (he/him/il/lui)> test
[14:56:34]
<kayou> just technical stuff
[14:57:11]
<kayou> nobody care about this, but I needed to say it
[15:03:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to add-notify_push-option: start notify-push after the watcher ([046d7b2e](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/046d7b2e53777b6632de4a428491acdc89d1e529))
[15:05:31]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to add-notify_push-option: ² ([4e2e3fa1](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/4e2e3fa1a7a7366a05944f9b9bca0833cfd6ecb4))
[15:05:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310440344) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: testme
[15:05:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310440483) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :book:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2275/badg...
[15:06:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310440466) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/18527/badge)](h...
[15:23:36]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to add-notify_push-option: remove warning ([ca8c2d61](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ca8c2d6120aea288ac691d6d84038f240e9677c0))
[15:23:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310477952) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: testme
[15:24:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310440344) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: testme
[15:24:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310440466) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/18527/badge)](h...
[15:24:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310408009) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :worm:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2274/badg...
[15:24:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310440483) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :book:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2275/badg...
[15:24:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310478061) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :stuck_out_tongue_winking_eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/ap...
[15:24:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310478090) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :bug:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2276/badge...
[15:24:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310407960) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: Meow :cat2:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/18526/badge)]...
[15:32:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310407851) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: testme
[15:32:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310379451) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :books: :worm:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2...
[15:33:06]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310379427) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: :carousel_horse:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/18524/ba...
[15:33:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-2310379323) on [issue #417](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417) Add notify push option: testme
[17:20:58]
<Aleks (he/him/il/lui)> wtf we can't use `__DATA_DIR__` in config panel binds ? '-'
[17:25:13]
<Aleks (he/him/il/lui)> ah yes that's the magnificent `local bind_file="$(echo "$bind" | cut -d: -f2 | sed s@__INSTALL_DIR__@$install_dir@ | sed s@__FINALPATH__@$final_path@ | sed s/__APP__/$app/)"`
[19:18:13]
<PapaDragon> Quick question: when installing an app, is a /var/log/$app directory automatically created?
[19:23:48]
<Aleks (he/him/il/lui)> If i remember correctly it'd created when calling the logrotate helper
[19:48:18]
<selfhoster1312> https://xmpp-upload.kl.netlib.re/upload/lLIlfNtCPkQi1698/bash-sith.png
[21:47:53]
<Yunohost Git/Infra notifications> [gitlab_ynh] kay0u opened [pull request #260](https://github.com/YunoHost-Apps/gitlab_ynh/pull/260): Testing
[21:47:53]
<Yunohost Git/Infra notifications> [gitlab_ynh] kay0u [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/260#issuecomment-2311158448) on [issue #260](https://github.com/YunoHost-Apps/gitlab_ynh/pull/260) Testing: testme
[21:47:56]
<Yunohost Git/Infra notifications> [gitlab_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/260#issuecomment-2311158518) on [issue #260](https://github.com/YunoHost-Apps/gitlab_ynh/pull/260) Testing: :books: :worm:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2...
[21:47:57]
<Yunohost Git/Infra notifications> [gitlab_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/260#issuecomment-2311158522) on [issue #260](https://github.com/YunoHost-Apps/gitlab_ynh/pull/260) Testing: :stuck_out_tongue_winking_eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/ap...