Monday, January 08, 2024
apps@conference.yunohost.org
January
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
31
       
             

[04:05:31] <Yunohost Git/Infra notifications> App prometheus goes down from level 8 to 6 in job [#21970](https://ci-apps.yunohost.org/ci/job/21970)
[05:17:00] <Yunohost Git/Infra notifications> Job [#21977](https://ci-apps.yunohost.org/ci/job/21977) for influxdb_v2 failed miserably :(
[14:21:27] <Yunohost Git/Infra notifications> [nextcloud_ynh] @Snutin opened [issue #645](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/645): Upgrade from 25.0.2 to 27.1.4 failed due to PHP version
[14:25:14] <Yunohost Git/Infra notifications> [nextcloud_ynh] @orhtej2 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/645#issuecomment-1881110558) on [issue #645](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/645) Upgrade from 25.0.2 to 27.1.4 failed due to PHP version: Duplicate of https://github.com/YunoHost-Apps/nextcloud_ynh/issues/617#issuecomment-1785430647 probably, update via old...
[16:05:37] <Yunohost Git/Infra notifications> [nextcloud_ynh] @Snutin closed [issue #645](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/645): Upgrade from 25.0.2 to 27.1.4 failed due to PHP version
[16:10:19] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1881378559) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: Do we have any clues on why the upgrade from 25.x.x doesnt work?
[18:07:20] <thatoo> Good evening,
Is there any Yunohost package needing python3.10 that I could get inspiration from (manifest.toml, install script...)?
[18:08:40] <thatoo> Second question, in install script, 
ynh_exec_warn_less ynh_install_nodejs --nodejs_version=$nodejs_version
I better set the value of $nodejs_version in _common.sh ?
Does this command install yarn and nvm too?
[18:12:19] <Aleks (he/him/il/lui)> yes, usually people set the version of nodejs in _common.sh, and it does install npm, but not yarn
[18:13:00] <Aleks (he/him/il/lui)> you can get yarn with something like this: https://github.com/YunoHost-Apps/mastodon_ynh/blob/bce29b74dd7c802da34c62fd84ed90005ca5f8c0/manifest.toml#L77
[18:21:34] <kayou> > <@Alekswag:matrix.org> weeeeeell now apparently it works but i dunno if that was because it was a fluc or because i restarted php-fpm x_x

The issue may be related to https://github.com/YunoHost-Apps/nextcloud_ynh/pull/541
[18:21:55] <kayou> > By default, cached scripts are revalidated on access to ensure that changes on disk take effect after at most 2 seconds. Since Nextcloud handles cache revalidation internally when required, the revalidation frequency can be reduced or completely disabled to enhance performance. Note, however, that it affects manual changes to scripts, including config.php. To check for changes at most every 60 seconds, use the following setting:

opcache.revalidate_freq = 60
[18:23:26] <kayou> As you restart php-fpm, config.php is reloaded instead of using the cache.
Could we force the cache to be reloaded with an occ command?
[18:26:16] <Aleks (he/him/il/lui)> ooh nice ?
[18:26:41] <Aleks (he/him/il/lui)> cheers for the research :O
[18:34:47] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to testing: Update change_url: properly reload php-fpm when changing trusted_domains ([3259e4ab](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/3259e4abb5126fb4b4f6ab1ef1fd9c6d841b2b89))
[18:37:46] <Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: People keep writing name = Upgrade from x.y.z but then its also added by the test code which ends up in Upgrade from... ([e7b47542](https://github.com/YunoHost/package_check/commit/e7b47542c837f7852995153839bbf3601f678404))
[18:37:46] <kayou> From what I read
[18:37:46] <kayou> I'm not sure, it's just a guess
[18:39:44] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1881626958) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: >Do we have any clues on why the upgrade from 25.x.x doesnt work? Apparently because of connect() to unix:/var/run/p...
[18:48:18] <Aleks (he/him/il/lui)> hmmmmmoooogod
[18:49:59] <Aleks (he/him/il/lui)> `unix:/var/run/php/php8.2-fpm-nextcloud.sock failed (2: No such file or directory)`

probably due to the fact that we do regen the fpm configuration before actually running the upgrade, but then `exec_occ` calls `ynh_install_app_dependencies` and temporarily-ish reinstalls the php dependencies for the php version relevant for that version, which coincindally also moves the php config because it thinks the php version of the app changed
[18:50:24] <Aleks (he/him/il/lui)> and then it comes to the last upgrade
[18:50:50] <Aleks (he/him/il/lui)> but because php8.2 is already install it think it doesnt need to call `ynh_install_app_dependencies` i guess
[18:50:50] <Aleks (he/him/il/lui)> and also doesnt regen the equivs virtual-dep I think, which sounds kinda bad
[18:50:50] <Aleks (he/him/il/lui)> which doesnt move the php conf
[19:01:10] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to testing: Update upgrade: fix inconsistent handling of the php version stuff between major upgrades ([5932bdc2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/5932bdc22ffc36d1f698634c14db69807967a3e8))
[19:02:48] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1881657718) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: :carousel_horse:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12523/ba...
[19:02:49] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1881657638) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: testme
[19:31:57] <lapineige> > <@thatoo:defis.info> Good evening,
> Is there any Yunohost package needing python3.10 that I could get inspiration from (manifest.toml, install script...)?

Python 3.10 is available in Debian 11 and a Yunohost package ?
[19:53:05] <Yunohost Git/Infra notifications> [nextcloud_ynh] @kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#discussion_r1445256499) on pull request #622 Testing / switch to packaging V2 + proper mail config: Just to avoid problems during future migrations, id use YNH_PHP_VERSION here instead suggestion NEXTCLOUD_...
[20:18:02] <Yunohost Git/Infra notifications> [nextcloud_ynh] @orhtej2 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#discussion_r1445282547) on pull request #622 Testing / switch to packaging V2 + proper mail config: Thats a bad idea IMO as YNH_PHP_VERSION comes from packaging v1 and is hardcoded to 7.4
[20:26:16] <Yunohost Git/Infra notifications> Job [#21983](https://ci-apps.yunohost.org/ci/job/21983) for uptime-kuma failed miserably :(
[20:49:16] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#discussion_r1445313465) on pull request #622 Testing / switch to packaging V2 + proper mail config: Heuarg ineed its not in _common.sh anymore so it defaults as 7.4 on bullseye ... Didnt even think that was the case, bu...
[20:53:40] <Aleks (he/him/il/lui)> `49421 WARNING ./upgrade: line 124: syntax error in conditional expression`
[20:53:41] <Aleks (he/him/il/lui)> butwhy.gif
[20:54:31] <Aleks (he/him/il/lui)> ogod i wrote `[[ expr ]` instead of `[[ expr ]]` x_x
[20:54:53] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to testing: Update upgrade: stupid typo ([e1651583](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/e1651583be54bee2d276a150d29211fedb04c119))
[21:10:37] <Yunohost Git/Infra notifications> App uptime-kuma goes down from level 8 to 6 in job [#21983](https://ci-apps.yunohost.org/ci/job/21983)
[21:15:50] <Yunohost Git/Infra notifications> [apps] @alexAubin approved [pull request #1961](https://github.com/YunoHost/apps/pull/1961#pullrequestreview-1810032610) Update app levels according to CI results
[21:40:25] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1881864161) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: testme
[21:40:46] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1881864161) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: testme
[21:40:47] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1881864231) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12524/badge)]...
[21:40:47] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1881864231) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12524/badge)]...
[21:59:56] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to master ([76bffb160cdf...2d5cee23263e](https://github.com/YunoHost/apps/compare/76bffb160cdf...2d5cee23263e))
[21:59:57] <Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1961](https://github.com/YunoHost/apps/pull/1961): Update app levels according to CI results
[21:59:57] <Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch update_app_levels
[21:59:57] <Yunohost Git/Infra notifications> [apps/master] Update app levels according to CI results - root
[22:01:23] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to testing: During upgrade, the backup of config.php should occurr before the maintenance mode is set to on, otherwise the restored ... ([10c7976d](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/10c7976d054d1b17d884185c4a1e35dc7b2d0350))
[22:10:33] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 2 commits to testing ([10c7976d054d...bee14c89729b](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/10c7976d054d...bee14c89729b))
[22:21:03] <thatoo> > Python 3.10 is available in Debian 11 and a Yunohost package ?

Well that's my wonder actualy. Dokos require it. I think Debian 12 has 3.11and Debian 11 has only up to 3.9 so I was wondering if an other package has faced such issue and if there is a clean yunohost way to deal with it.
[22:21:48] <Aleks (he/him/il/lui)> :|
[22:22:01] <lapineige> My guess would be that it is not available by default, and hence not a good idea to configure it globally on Debian 11/Yunohost. Better wait for Debian 12
[22:23:20] <Aleks (he/him/il/lui)> https://forum.yunohost.org/t/use-newer-python-than-3-9/22568
[22:24:14] <Émy - OniriCorpe> I don't understand why devs deprecate such recent versions of python .___.
[22:24:34] <Émy - OniriCorpe> mandatory python3.10 is absurd
[22:24:36] <Aleks (he/him/il/lui)> it's like nothing is ever "new" enough idk
[22:27:24] <Aleks (he/him/il/lui)> but honestly yeah i really wonder what the hell in the changelog is mandating python 3.10, sounds like it could be that the dev just pick a random version, or maybe the newest at the time they wrote their doc, and there's no real hard dependency, maybe can be worth it to try running the app with 3.9 ...
[22:29:26] <tituspijean> I am a bit ashamed of even having looked at it, but would this "help"? https://github.com/niess/python-appimage
`./python3.11.7-cp311-cp311-manylinux2014_x86_64.AppImage -m pip install numpy` would install numpy in `~/.local/lib/python3.11`. In an app context, `~` is simply `$install_dir`
[22:29:31] <Aleks (he/him/il/lui)> jesus christ does anybody has any insight regarding the usefulness of fpm\_free\_footprint in the stupid config panel ~\_~ https://github.com/YunoHost-Apps/nextcloud\_ynh/blob/testing/config\_panel.toml#L26 i want to `rm` it so bad zzzz
[22:30:22] <Aleks (he/him/il/lui)> > <@titus:pijean.ovh> I am a bit ashamed of even having looked at it, but would this "help"? https://github.com/niess/python-appimage
> `./python3.11.7-cp311-cp311-manylinux2014_x86_64.AppImage -m pip install numpy` would install numpy in `~/.local/lib/python3.11`. In an app context, `~` is simply `$install_dir`

djeezus christ please no more cyberfrankenstein, people should just help with moving to Bookworm for christ sake
[22:30:54] *tituspijean knew he should have started to run far away already
[22:30:54] <Aleks (he/him/il/lui)> i mean otherwise let's stay on whichever Debian version forever and run everything in docker with whatever brand new version of everything idk
[22:33:48] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to testing: Fix custom getter for fpm_free_footprint >_> ([fe63ef6d](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/fe63ef6dfc01132a654a65a50d8f825d0b51823a))
[22:34:30] <thatoo> > <@Alekswag:matrix.org> but honestly yeah i really wonder what the hell in the changelog is mandating python 3.10, sounds like it could be that the dev just pick a random version, or maybe the newest at the time they wrote their doc, and there's no real hard dependency, maybe can be worth it to try running the app with 3.9 ...

I'll try then.
[22:35:10] <Aleks (he/him/il/lui)> theofficethankyou.gif
[22:36:29] <hook> Does XMPP/Metronome by default implement RTBL?
[22:39:02] <hook> https://xmppbl.org/
[22:40:26] <hook> Whoops, wrong channel, sorry
[22:48:53] <thatoo> > <@Alekswag:matrix.org> https://forum.yunohost.org/t/use-newer-python-than-3-9/22568

Very interesting, thank you.
[22:51:49] <kayou> > <@Alekswag:matrix.org> jesus christ does anybody has any insight regarding the usefulness of fpm\_free\_footprint in the stupid config panel ~\_~ https://github.com/YunoHost-Apps/nextcloud\_ynh/blob/testing/config\_panel.toml#L26 i want to `rm` it so bad zzzz

i used them in one of my server, because nextcloud was very slow, changing these options was helpful
[22:52:14] <Aleks (he/him/il/lui)> you mean specifically the free_fpm_footprint ?
[22:53:42] <kayou> hmm i mean all 3 of them
[22:53:42] <Aleks (he/him/il/lui)> i mean now that i debug the code i realize this free_fpm_footprint could not even work on the first place so i'm 99.9 nobody used it ...
[22:54:32] <Aleks (he/him/il/lui)> ah or maybe the setter works but not the display so like it will always display 0 but if you change the value it should apply maybe, hmpf
[22:54:32] <kayou> but i just tweak these value, increase the amount of memory for nextcloud, and poof, it was better, without really understand (i mean, yeah, nextcloud had more memory, yay)
[23:01:47] <Aleks (he/him/il/lui)> kayou: hmmm and do you understand this thing with the character set ? x_x I'm confused why we end up having to pass `--default-character-set` to mysql command, doesnt mysql use the character set of the DB ? What sense would it make for mysql to use a different character set than the one the DB is using X_x ?
[23:01:47] <Aleks (he/him/il/lui)> (i don't know anything about this stuff)
[23:02:33] <Aleks (he/him/il/lui)> (was trying to look at how to get rid of the warnings and then checked the diff for the mysq helpers)
[23:03:19] <orhtej2> > <@Alekswag:matrix.org> kayou: hmmm and do you understand this thing with the character set ? x_x I'm confused why we end up having to pass `--default-character-set` to mysql command, doesnt mysql use the character set of the DB ? What sense would it make for mysql to use a different character set than the one the DB is using X_x ?

Character set is a thing you can change even on connection level iirc
[23:04:04] <lapineige> > <@Alekswag:matrix.org> ah or maybe the setter works but not the display so like it will always display 0 but if you change the value it should apply maybe, hmpf

I believe I had the case
[23:04:45] <orhtej2> > <@Alekswag:matrix.org> jesus christ does anybody has any insight regarding the usefulness of fpm\_free\_footprint in the stupid config panel ~\_~ https://github.com/YunoHost-Apps/nextcloud\_ynh/blob/testing/config\_panel.toml#L26 i want to `rm` it so bad zzzz

This stanza is copy - pasted all over ynh
[23:05:16] <Aleks (he/him/il/lui)> yeah that's why i'm so pissed everytime i see it ~_~
[23:06:09] <orhtej2> Also cf https://github.com/YunoHost/issues/issues/2268
[23:06:34] <orhtej2> Imo setting conf for php specifically is broken
[23:06:34] <Aleks (he/him/il/lui)> we discussed at the camp this summer what to do about it and iirc the conclusion was "it's bullshit because it tries to abstract the technical php settings which are not really straightforward, but people probably don't understand either what the hell usage and footprint are supposed to mean, so we should probably just exposed the underlying php settings"
[23:06:58] <Aleks (he/him/il/lui)> aka "refactor everything"
[23:08:13] <Aleks (he/him/il/lui)> and also find a way to standardize it in the core rather than copypasting the same stuff all over the place
[23:10:09] <kayou> > <@Alekswag:matrix.org> kayou: hmmm and do you understand this thing with the character set ? x_x I'm confused why we end up having to pass `--default-character-set` to mysql command, doesnt mysql use the character set of the DB ? What sense would it make for mysql to use a different character set than the one the DB is using X_x ?

because nextcloud needs 4-byte
[23:10:10] <kayou> https://docs.nextcloud.com/server/latest/admin_manual/configuration_database/mysql_4byte_support.html
[23:10:10] <kayou> https://docs.nextcloud.com/server/latest/admin_manual/configuration_database/linux_database_configuration.html
[23:10:36] <kayou> we have to pass `character_set_server = utf8mb4`
[23:10:36] <kayou> we have to set `character_set_server = utf8mb4`
[23:11:01] <Aleks (he/him/il/lui)> yeah but i mean i'm surprised that mysql can't figure out alone what's supposed to be the default encoding when dumping/restoring/piping instructions x_x
[23:12:35] <orhtej2> > <@Alekswag:matrix.org> yeah but i mean i'm surprised that mysql can't figure out alone what's supposed to be the default encoding when dumping/restoring/piping instructions x_x

Even worse, it converts input to storage coding which is inefficient
[23:13:09] <kayou> we can change the ssettings in /etc/mysql/something.cnf
[23:13:29] <kayou> ```
[mysql]
default-character-set=utf8
```
[23:13:30] <kayou> but it will change it for everyone
[23:13:30] <kayou> meh
[23:14:49] <orhtej2> > <@kayou:matrix.org> we can change the ssettings in /etc/mysql/something.cnf

There was someone inquiring about this recently
[23:15:17] <orhtej2> > Even worse, it converts input to storage coding which is inefficient

And what I forgot of course sorting https://dev.mysql.com/doc/refman/8.0/en/charset-charsets.html
[23:18:09] <lapineige> > <@Alekswag:matrix.org> we discussed at the camp this summer what to do about it and iirc the conclusion was "it's bullshit because it tries to abstract the technical php settings which are not really straightforward, but people probably don't understand either what the hell usage and footprint are supposed to mean, so we should probably just exposed the underlying php settings"

Yeah it's very hard to know what's the best option for you, even if it's still better than the raw settings
[23:21:57] <Aleks (he/him/il/lui)> > <@kayou:matrix.org> we can change the ssettings in /etc/mysql/something.cnf

yeah Eric made a PR for this but i'm like "i don't know anything about this and what are the implications for legacy" x_x
[23:22:48] <Aleks (he/him/il/lui)> i'm wondering if we can't just find a trick to query what's the character set of the DB and inject something like --charset="$charset" for every mysql command
[23:29:03] <Aleks (he/him/il/lui)> ```
• --default-character-set=charset_name

Use charset_name as the default character set for the client and connection.

A common issue that can occur when the operating system uses utf8 or another multi-byte character set is that output from the mysql
client is formatted incorrectly, due to the fact that the MariaDB client uses the latin1 character set by default. You can usually
fix such issues by using this option to force the client to use the system character set instead.
```

so it's more about a potential mismatch between client and server ? 🤔
[23:29:52] <Aleks (he/him/il/lui)> hmmyeah i guess this is our usecase because we forced the DB to a specific charset
[23:30:13] <Aleks (he/him/il/lui)> meh trying to understand this at 00:30 is zzz
[23:36:13] <Bram> for python versions we don't use pythonz in the helpers anymore?
[23:49:36] <Aleks (he/him/il/lui)> i don't think we ever did