Friday, November 24, 2023
apps@conference.yunohost.org
November
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
     
             

[00:00:15] <Aleks (he/him/il/lui)> also you probably don't really need to define the ask string or `choices` for the "domain" install question, it's a special question automatically handled by the core : https://github.com/YunoHost-Apps/mlmmj_ynh/blob/master/manifest.toml#L41
[00:00:26] <Aleks (he/him/il/lui)> shoud look like this : https://github.com/YunoHost/example_ynh/blob/master/manifest.toml#L56
[00:00:46] <Raphael> > <@Alekswag:matrix.org> ah so your app doesnt have any "permission" resource

well, I’m not sure how to handle it, as mlmmj needs the creation of a tree, with users and group being nobody:nogroup (same as for postfix writing permissions)
[00:01:28] <Aleks (he/him/il/lui)> needs the creation of a tree ? what kind of tree ? x_X how does that relates to the permission ?
[00:01:42] <Aleks (he/him/il/lui)> NB permission in that context corresponds to SSOwat ACLs basically
[00:03:10] <Raphael> Ah ok, sorry I misunderstood
[00:03:33] <Aleks (he/him/il/lui)> regarding https://github.com/YunoHost-Apps/mlmmj_ynh/blob/master/scripts/install#L14-L16 eeeer i guess it works, but the thing is that you're not supposed to have to "fetch" the settings anymore, the variables $domain, $language and $install_dir should be directly available in this context
[00:03:51] <Raphael> It’s a totally new universe to me, so I may do some silly things
[00:04:51] <Raphael> ok, I’ll test with removing these lines then
[00:06:45] <Aleks (he/him/il/lui)> but back to the initial issue : i'm pretty sure the bug happens because basically every app is expected to have at least a "main" permission (or again to say it otherwise, SSOwat ACL basically), so you gotta add the `[resource.permission]` block
[00:07:01] <Aleks (he/him/il/lui)> if your app is not a webapp, maybe you need to set `url = null` or something like this
[00:09:21] <Raphael> $install_dir in low caps, I wrote $INSTALL_DIR I messed with __INSTALL_DIR__
[00:10:20] <Raphael> __ INSTALL_DIR __ (without makdown formating)
[00:10:25] <Raphael> > <@Alekswag:matrix.org> if your app is not a webapp, maybe you need to set `url = null` or something like this

ok
[00:15:11] <Raphael> main.url = null, I try that
[00:18:18] <Raphael> Apparently I can put nothing at all :
url: The relative URI corresponding to this permission. Typically / or /something. This property may be omitted for non-web permissions.
[00:18:38] <Raphael> from here https://yunohost.org/fr/packaging_apps_resources
[00:23:18] <Raphael> I see this in api logs, maybe related
2023-11-24 01:22:01,437 WARNING yunohost.app (unknown function) - [536.47] Failed to get label for app mlmmj ?

[00:23:30] <Raphael> It still doesn’t work
[00:26:20] <Raphael> Thanks for your help, I’ll try again tomorrow, I’ll have more luck, I’m sure of it 🙂
[00:34:53] <Aleks (he/him/il/lui)> Ah yes the label thing is related to permission as well
[01:18:07] <Yunohost Git/Infra notifications> App framaforms goes down from level 6 to 1 in job [#20868](https://ci-apps.yunohost.org/ci/job/20868)
[02:37:14] <Yunohost Git/Infra notifications> [nextcloud_ynh] @Limezy [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1825074451) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: @ericgaspar thanks for your answer and also thanks for maintaining this parallel universe
[07:02:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to testing: [fix] not a valid identifier ([92a914e2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/92a914e28638d5e325123bc27510901b9a6366bd))
[07:02:42] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README ([63429112](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/63429112a4d6ed453028cf0ca4b9b089f5115c46))
[07:03:04] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1825228386) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packagingv2 + proper mail config: :stuck_out_tongue_winking_eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/ap...
[07:03:05] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1825228345) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packagingv2 + proper mail config: testme
[07:40:58] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 20 commits to v2_PostgreSQL ([e47de152eb18...77e8a18ac7c5](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/e47de152eb18...77e8a18ac7c5))
[07:44:18] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: add post upgrade message ([f57d8316](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/f57d83163625260f420351753c64fdb61918a7f0))
[07:44:50] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 2 commits to v2_PostgreSQL ([77e8a18ac7c5...62d7f2caf647](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/77e8a18ac7c5...62d7f2caf647))
[07:44:55] <Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] add post upgrade message - Éric Gaspar
[07:44:59] <Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - Éric Gaspar
[07:59:19] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update restore ([4d975cc3](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/4d975cc3e2a6982bd1828d78b3c4f9cc1410b166))
[08:00:15] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update extra_php-fpm.conf ([d687230c](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d687230c664ae9da86b6f93bf636e3cdcbdcf53e))
[08:01:07] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update nginx.conf ([04371225](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/043712257ee9c89f898537828cc74431639ad553))
[08:01:19] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 2 commits to v2_PostgreSQL ([d687230c664a...c5db82b07aba](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/d687230c664a...c5db82b07aba))
[08:01:24] <Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Update nginx.conf - Éric Gaspar
[08:01:30] <Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - Éric Gaspar
[08:42:18] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to 27.1.4: 27.1.4 ([130b27ec](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/130b27ec0fa9009c2613b2a719c1302db80c35b6))
[08:42:20] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur created new branch 27.1.4
[08:49:30] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur opened [pull request #631](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/631): 27.1.4
[08:50:03] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to 27.1.4: 27.1.4 manifest ([42a12799](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/42a127992be57c96652db7c785c21a4402954a5a))
[08:50:10] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to 27.1.4: Auto-update README ([5fdccc48](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/5fdccc4865ddfec97d76ccdf4eab9e876526a0ef))
[08:50:13] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur edited [pull request #631](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/631): 27.1.4
[08:50:17] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/631#issuecomment-1825329979) on [issue #631](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/631) 27.1.4: testme
[08:50:18] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/631#issuecomment-1825330027) on [issue #631](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/631) 27.1.4: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/11275/badge)](h...
[09:05:59] <Charles P.> Uh, "Setting up source files" is taking a while, it's been at least 8 minutes since that step started, usually the whole install test runs in less than that 👀
https://ci-apps-dev.yunohost.org/ci/job/11275
[09:06:21] <Charles P.> (that's the test for NC 27.1.4)
[09:14:35] <azahto> Hi there!
I am still working on my hotcrp_ynh package. I think I got the basic install done, now I need to go through nginx/php/mysql config.
Currently, I cannot access the hotcrp page after installation. It is probably due to nginx config.
I am still a total newbie for this kind of stuff, so I would appreciate some guidance.
[09:17:30] <azahto> So, currently, I have my app path being `/crp` when installing. But nginx redirect me to the admin page when I try to go to `myyunohostserverdomain/crp`. SO I guess I have to change something in the nginx conf file, any idea?
The current code is available at https://github.com/Azzaare/hotcrp_ynh/tree/newskin
[09:30:21] <Mateusz Szymański> > So, currently, I have my app path being `/crp` when installing. But nginx redirect me to the admin page when I try to go to `myyunohostserverdomain/crp`. SO I guess I have to change something in the nginx conf file, any idea?
> The current code is available at https://github.com/Azzaare/hotcrp_ynh/tree/newskin

do you have install logs available? nothing really stands out as obviously wrong here
[09:36:54] <Mateusz Szymański> > So, currently, I have my app path being `/crp` when installing. But nginx redirect me to the admin page when I try to go to `myyunohostserverdomain/crp`. SO I guess I have to change something in the nginx conf file, any idea?
> The current code is available at https://github.com/Azzaare/hotcrp_ynh/tree/newskin

alos, is the tile for your app appearing in the sso portal? what happens if you click it?
[09:36:55] <azahto> I assume I have to handle the following situation (from the log)
```
This is a dummy disclaimer to display after the install

The app url is `yunopack.local/crp`

The app install dir is `/var/www/hotcrp`
```

The app id is `hotcrp`
[09:37:16] <Mateusz Szymański> > I assume I have to handle the following situation (from the log)
> ```
> This is a dummy disclaimer to display after the install
>
> The app url is `yunopack.local/crp`
>
> The app install dir is `/var/www/hotcrp`
> ```
>
> The app id is `hotcrp`

log as in tools->logs->installation of x
[09:37:21] <Mateusz Szymański> and share that with yunopaste
[09:38:46] <azahto> > <@orhtej2:matrix.org> alos, is the tile for your app appearing in the sso portal? what happens if you click it?

Ha, this is a good question. I am redirected to the admin page when I try to reach the SSO portal. Probably because of the domain name (my dev server is accessed online in the end, I was not planning for it at first)
[09:40:00] <azahto> > <@orhtej2:matrix.org> and share that with yunopaste

Oh, this is such a great feature, I had no idea: https://paste.yunohost.org/raw/utexuxuzim
[09:46:57] <Mateusz Szymański> aside from setting systemd service and logrotate this looks fine(ish)
[09:47:16] <Mateusz Szymański> likely something to do with nginx config
[09:47:28] <Mateusz Szymański> ah, and remove the step with curl calling random url
[09:47:46] <Mateusz Szymański> but if you say you have no access to portal - perhaps that's the first thing to look at
[09:47:57] <Mateusz Szymański> meaning perhaps its your env that broken, not the app
[09:51:00] <azahto> OK, I am gonna tackle the env first, and then clean the curl thing, and probably uncomment systemd and logrotate lines (I think I had errors at some point, but I was planning to figure that out later)
[09:51:42] <azahto> Thank you so much by the way, I am lacking many skill (yet) needed for that, so I greatly appreciate the help
[10:09:44] <azahto> The main issue was indeed my env. Once fixed, I was able to reach the horcrp main page (where everything is broken, but that is normal, as I need to handle basic php conf for the web app).
[11:15:43] <Mateusz Szymański> azahto: in the wake of upcoming Bookworm transition - can you please check if the app works with newer PHP version? 8.2 preferably. It's unclear from the HotCRP GitHub page if that's the case.
[12:45:01] <Yunohost Git/Infra notifications> [apps] @CodeShakingSheep opened [pull request #1880](https://github.com/YunoHost/apps/pull/1880): Add missing links and descriptions to wishlist.toml
[12:52:08] <Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1880](https://github.com/YunoHost/apps/pull/1880): Add missing links and descriptions to wishlist.toml
[12:52:09] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([cf845d50ecb7...97d3c03b8d39](https://github.com/YunoHost/apps/compare/cf845d50ecb7...97d3c03b8d39))
[12:52:12] <Yunohost Git/Infra notifications> [apps/master] Add missing links and descriptions to wishlist.toml - CodeShakingSheep
[12:52:21] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1880 from CodeShakingSheep/add-missing-app-info Add missing links and descriptions to wishlist.toml - Alexandre Aubin
[13:02:38] <Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1879](https://github.com/YunoHost/apps/pull/1879): Add Koreader Sync Server to wishlist
[13:02:39] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to master ([97d3c03b8d39...41b4c39e9b98](https://github.com/YunoHost/apps/compare/97d3c03b8d39...41b4c39e9b98))
[13:02:43] <Yunohost Git/Infra notifications> [apps/master] Add Koreader Sync Server to wishlist - yunohost-bot
[13:02:47] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1879 from YunoHost/add-to-wishlist-koreader-sync-server Add Koreader Sync Server to wishlist - eric_G
[13:02:49] <Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1878](https://github.com/YunoHost/apps/pull/1878): Add Hauk to wishlist
[13:02:50] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to master ([41b4c39e9b98...7b565816c90e](https://github.com/YunoHost/apps/compare/41b4c39e9b98...7b565816c90e))
[13:02:54] <Yunohost Git/Infra notifications> [apps/master] Add Hauk to wishlist - yunohost-bot
[13:03:03] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1878 from YunoHost/add-to-wishlist-hauk Add Hauk to wishlist - eric_G
[13:03:53] <Mateusz Szymański> https://www.php.net/supported-versions.php \<- perhaps linter should actively discourage usage of PHP \<= 8.0?
[13:05:53] <Mateusz Szymański> or we could at least add 'relies on EOL software stack' antifeature (and prevent apps that are unmaintained and/or rely on unmaintained software from reaching 8*)?
[13:08:27] <Aleks (he/him/il/lui)> meh i guess we could but so far the rule of thumb was : we use what's shipped in Debian, and Debian Bullseye ships php 7.4
[13:08:52] <Aleks (he/him/il/lui)> though nowadays we automatically include sury and switching to a different php version is not the pain in the ass that it was before
[13:09:18] <Aleks (he/him/il/lui)> but still would feel funky to complain that app use what's shipped in Debian vanilla
[13:10:21] <Aleks (he/him/il/lui)> imho the main issue here is that PHP have a super tight release cycle, yet by the time one PHP version makes it into Debian stable, it's already 6 months old, and then one year later it's end of life despite that same debian stable being well alive
[13:11:01] <Aleks (he/him/il/lui)> and also it's related to the fact that Buster had 7.3, yet Bullseye only had 7.4
[13:11:34] <Aleks (he/him/il/lui)> imho if there are big security issues with PHP 7.4 it will surely get patched by the debian folks one way or another
[13:13:17] <eric_G> my guess is that there is less than 20 apps using PHP7.4
[13:17:12] <Aleks (he/him/il/lui)> https://paste.yunohost.org/raw/qavukomake
[13:17:20] <Aleks (he/him/il/lui)> 38ish ;D
[13:17:31] <Aleks (he/him/il/lui)> though didn't check what level they are
[13:18:54] <eric_G> I am gonna fix this...
[13:24:35] <Aleks (he/him/il/lui)> my god php8.0 theoretical security support ends tomorrow what the heck
[13:24:53] <Mateusz Szymański> > <@Alekswag:matrix.org> my god php8.0 theoretical security support ends tomorrow what the heck

that's why I said `PHP <= 8.0` :P
[13:36:55] <Aleks (he/him/il/lui)> > <@charlesp:stratus.family> Uh, "Setting up source files" is taking a while, it's been at least 8 minutes since that step started, usually the whole install test runs in less than that 👀
> https://ci-apps-dev.yunohost.org/ci/job/11275

eeeeh what the heck, i didn't kill the job because i wanted to investigate what's the process it's stuck on but ... it finally worked ?
[13:37:37] <Aleks (he/him/il/lui)> and i see that the "Setting up source files" took between 15 and 39 minutes ... what the heck
[13:39:20] <Aleks (he/him/il/lui)> it basically only calls ynh_setup_source, what the hell can take 30 minutes
[13:40:07] <Aleks (he/him/il/lui)> apparently, `wget`
[13:40:47] <Aleks (he/him/il/lui)> despite the `--timeout=900` x_x
[13:41:23] <Aleks (he/him/il/lui)> @_@
[13:41:51] <Aleks (he/him/il/lui)> i can't even with computers
[13:42:42] <Aleks (he/him/il/lui)> yeah there's definitely some einstein general relativity distorting space time or something
[13:42:53] <Aleks (he/him/il/lui)> maybe some node_modules close by on the hard drive is warping time
[13:43:19] <Mateusz Szymański> yeah, the sheer weight of `node_modules` could cause that
[13:43:52] <Aleks (he/him/il/lui)> MFW general relativity also applies to cyberspacetime
[13:44:16] <Mateusz Szymański> OTOH what does `timeout` do exactly? Perhaps it's 'assume other party is dead if no data was transfered within X' rather than 'get this before X elapses'
[13:44:36] <Aleks (he/him/il/lui)> yeah i assume the download happens but is just slow as hell
[13:44:44] <Aleks (he/him/il/lui)> maybe our CI server go throttled by the other side idk
[13:45:54] <Aleks (he/him/il/lui)> >The "time" of this timeout refers to idle time: if, at any point in the download, no data is received for more than the specified number of seconds, reading fails and the download is restarted. This option does not directly affect the duration of the entire download.
[13:46:34] <Aleks (he/him/il/lui)> ah that's the point where i say "thanks to packaging v2™ we could tweak package_check to pre-fetch assets only once per CI job"
[14:46:19] <Yunohost Git/Infra notifications> App homarr rises from level 7 to 8 in job [#20884](https://ci-apps.yunohost.org/ci/job/20884) !
[15:05:40] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar approved [pull request #631](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/631#pullrequestreview-1748100233) 27.1.4
[15:05:45] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar merged [pull request #631](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/631): 27.1.4
[15:05:46] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 4 commits to master ([bc59f53dbb91...9c6d1eea8e0a](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/bc59f53dbb91...9c6d1eea8e0a))
[15:05:48] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar deleted branch 27.1.4
[15:05:49] <Yunohost Git/Infra notifications> [nextcloud_ynh/master] 27.1.4 manifest - ljf (zamentur)
[15:05:53] <Yunohost Git/Infra notifications> [nextcloud_ynh/master] Auto-update README - yunohost-bot
[15:06:15] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 5 commits to testing ([043712257ee9...b975cd690466](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/043712257ee9...b975cd690466))
[15:06:19] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Auto-update README - yunohost-bot
[15:06:26] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Merge pull request #631 from YunoHost-Apps/27.1.4 27.1.4 - eric_G
[15:06:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622): Testing / switch to packagingv2 + proper mail config
[15:23:36] <Aleks (he/him/il/lui)> hmkay folks i dunno what you think about ci-apps-bookworm but the queue is madness, i'm gonna flush the pending job and only trigger a job for the 125-ish apps of the "BookwormFailingApps" spreadsheet, there's no big reason to run the CI for the other apps which did pass test until we're down to something like 5~10% failing apps
[15:24:12] <ljf> For information: i have just tested the 27.1.4 upgrade on 2 prod servers, everything seems ok
[15:31:12] <eric_G> I yolo pushed it to master 😬
[15:42:10] <lapineige> YaaS. Yolohost as a Service
[15:49:03] <Mateusz Szymański> > <@Alekswag:matrix.org> hmkay folks i dunno what you think about ci-apps-bookworm but the queue is madness, i'm gonna flush the pending job and only trigger a job for the 125-ish apps of the "BookwormFailingApps" spreadsheet, there's no big reason to run the CI for the other apps which did pass test until we're down to something like 5~10% failing apps

https://www.youtube.com/watch?v=FaLyasJPyUU
[16:08:45] <Mateusz Szymański> > <@Alekswag:matrix.org> hmkay folks i dunno what you think about ci-apps-bookworm but the queue is madness, i'm gonna flush the pending job and only trigger a job for the 125-ish apps of the "BookwormFailingApps" spreadsheet, there's no big reason to run the CI for the other apps which did pass test until we're down to something like 5~10% failing apps

thanks, although I have a sinking feeling that jobs for `audiobookshelf` and `indexhibit` are stuck and not killed given they run since ~8AM
[16:09:05] <Mateusz Szymański> `time dilation` strikes again!
[16:09:20] <Aleks (he/him/il/lui)> 🤔
[16:10:15] <Mateusz Szymański> (not recycling them for debugging purposes)
[16:11:20] <Aleks (he/him/il/lui)> `Nov 24 08:07:09 ci-apps-bookworm python[1640052]: peewee.OperationalError: database or disk is full`
[16:11:24] <Aleks (he/him/il/lui)> 🙀
[16:12:54] <Aleks (he/him/il/lui)> hmmm nice we have a bunch of logs waiting more than 10MB what the frak
[16:15:08] <Aleks (he/him/il/lui)> anyway
[16:15:08] <Aleks (he/him/il/lui)> and journalctl logging 2.2GB
[16:15:15] <Aleks (he/him/il/lui)> cleaning done, should be better hopefully
[16:15:25] <Aleks (he/him/il/lui)> restarting the two jobs
[16:16:09] <Aleks (he/him/il/lui)> ooor i guess we're running gitlab now ...
[16:25:45] <Aleks (he/him/il/lui)> HgO: getting lock issues with vpnclient ?
[16:27:53] <HgO> Always!
[16:29:27] <HgO> But I think I found why, it's because of systemd…
[16:29:31] <HgO> Anyways, in my PR I won't need the lock anymore
[16:30:19] <Aleks (he/him/il/lui)> yeah ... it's a bit tricky to explain but usually the lock automatically gets "transferred/allowed" for childs of the main yunohost process, hence why you can run `yunohost` commands from inside for example `yunohost app install` .... BUT when using for example `systemctl start vpnclient` where the vpnclient script is likely to run `yunohost` commands, this triggers a completely different process tree, hence the lock doesnt get transferred
[16:30:57] <Aleks (he/him/il/lui)> hence why you need to use `yunohost service start vpnclient` which will transfer the lock (assuming it was registered with --need-locksomething)
[16:31:05] <HgO> Yes that's what I figured out a bit late
[16:31:27] <HgO> > <@Alekswag:matrix.org> hence why you need to use `yunohost service start vpnclient` which will transfer the lock (assuming it was registered with --need-locksomething)

Ah I'll check that
[16:32:23] <Aleks (he/him/il/lui)> it's kind of a pain in the ass, it also bited me the other day working on hotspot iirc
[16:34:33] <Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/hotspot_ynh/pull/113 ah maybe we can solve the boring non-free thingy now that https://github.com/YunoHost/yunohost/pull/1729 is merged
[16:39:57] <Aleks (he/him/il/lui)> but for this we should first make a release annnnd before making a release maybe checking/fixing the CI aaaaa
[16:40:03] <Aleks (he/him/il/lui)> psychokwak emoji when
[16:50:35] <HgO> > <@Alekswag:matrix.org> hence why you need to use `yunohost service start vpnclient` which will transfer the lock (assuming it was registered with --need-locksomething)

Yes it works! Good to know :) But as I said I won't need that anymore 😼
[16:52:04] <HgO> So my PR is ready : https://github.com/YunoHost-Apps/vpnclient_ynh/pull/118
[16:52:26] <HgO> I'm sorry there are a lot of changes…
[17:00:28] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([c96e4add](https://github.com/YunoHost/apps/commit/c96e4add27ea378faf6039d35b03316315d69d91))
[17:00:28] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1881](https://github.com/YunoHost/apps/pull/1881): Update app levels according to CI results
[17:00:29] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch update_app_levels
[17:10:49] <Yunohost Git/Infra notifications> [package_linter] @alexAubin pushed 1 commit to master: Tighten some info as warnings (should only affect a handful of level 7+ apps nowadays) ([67202bfa](https://github.com/YunoHost/package_linter/commit/67202bfa7b8d283e3cef5744240c072c69949ccf))
[17:12:37] <Aleks (he/him/il/lui)> > <@hgo:chat.batato.be> I'm sorry there are a lot of changes…

working benevolently for a technical free software project ? you should be ashamed of yourself 🔔
[17:14:21] <Yunohost Git/Infra notifications> [apps] @ericgaspar created new branch freescout
[17:14:40] <eric_G> what? me?
[17:15:10] <Aleks (he/him/il/lui)> everybody :D
[17:15:16] <Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch freescout
[17:15:20] <Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch add-to-wishlist-koreader-sync-server
[17:15:22] <Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch add-to-wishlist-hauk
[17:16:00] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([dfe0dd03](https://github.com/YunoHost/apps/commit/dfe0dd03084bdfb6dd4b85f9547c9c9aab7fb4b3))
[17:16:07] <Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1881](https://github.com/YunoHost/apps/pull/1881): Update app levels according to CI results
[17:16:51] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([380e42fa](https://github.com/YunoHost/apps/commit/380e42fad164762b262712b917593ff17f4c5623))
[17:16:52] <Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1881](https://github.com/YunoHost/apps/pull/1881): Update app levels according to CI results
[17:26:59] <HgO> https://aria.im/_matrix/media/v1/download/chat.batato.be/KOnhKUuGjkrSqdcfMDsUMlEE
[17:29:45] <eric_G> on attend l'enclume 😂
[17:33:03] <Mateusz Szymański> https://aria.im/_matrix/media/v1/download/matrix.org/qayUkHXvHgKfmZifVijjvCtx
[17:33:24] <Mateusz Szymański> thanks Mac for not compressing the image x_x
[18:26:10] <Yunohost Git/Infra notifications> App firefish goes down from level 7 to 1 in job [#20777](https://ci-apps.yunohost.org/ci/job/20777)
[19:03:57] <lapineige> Capitalism revenge against communism, I guess
[19:31:24] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([4aa3318b](https://github.com/YunoHost/apps/commit/4aa3318b32e501b172dd35667836b3f84db8091c))
[19:31:24] <Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1881](https://github.com/YunoHost/apps/pull/1881): Update app levels according to CI results
[19:31:28] <Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1881](https://github.com/YunoHost/apps/pull/1881#pullrequestreview-1748373535) Update app levels according to CI results
[19:33:39] <Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1881#issuecomment-1826026741) on [issue #1881](https://github.com/YunoHost/apps/pull/1881) Update app levels according to CI results: :tada: :partying_face:
[19:33:46] <Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1881](https://github.com/YunoHost/apps/pull/1881): Update app levels according to CI results
[19:33:46] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 5 commits to master ([7b565816c90e...65c7510761bc](https://github.com/YunoHost/apps/compare/7b565816c90e...65c7510761bc))
[19:33:46] <Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[19:33:50] <Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[19:33:50] <Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch update_app_levels