[00:13:59]
<Yunohost Git/Infra notifications> App getsimple goes down from level 8 to 7 in job [#27316](https://ci-apps.yunohost.org/ci/job/27316)
[00:59:44]
<rodinux> Wishlist antifapp...
[01:53:18]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 27 pending update PRs
- 13 new apps PRs
- 6 failed apps updates: calibreweb, focalboard, invoiceninja5, minetest, shiori, snserver
See the full log here: http://paste.yunohost.org/raw/erozezacoq
[02:33:08]
<Yunohost Git/Infra notifications> App wetty failed all tests in job [#27325](https://ci-apps.yunohost.org/ci/job/27325) :(
[03:01:04]
<Yunohost Git/Infra notifications> App endi failed all tests in job [#27328](https://ci-apps.yunohost.org/ci/job/27328) :(
[06:01:38]
<@chrichri:ween.de> Good morning :)
[06:08:02]
<@chrichri:ween.de> https://aria.im/_matrix/media/v1/download/ween.de/B2wiLxpSC9C1VaM4WBcsu2LuiamfjQdp
[06:08:43]
<@chrichri:ween.de> Why is it marked broken? Yesterday I ran a test again and it has been good: https://github.com/YunoHost-Apps/microblog.pub_ynh/pull/5#issuecomment-2156619027
[06:27:15]
<Thomas> You have to wait for the PR of Friday :)
[06:30:01]
<@chrichri:ween.de> > <@thovi98:matrix.org> You have to wait for the PR of Friday :)
next friday? Oh, I thought the catalog updates every ~4h. Then, yes, well...
[06:34:19]
<Thomas> If you update the app, it's true, but if you change something in the yunohost/apps repo, it will be updated on friday
[06:53:01]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to piwigo: Update apps.toml ([bcfb4f26](https://github.com/YunoHost/apps/commit/bcfb4f26ecb473e0409e83e50a601c7563443619))
[06:53:02]
<Yunohost Git/Infra notifications> [apps] ericgaspar created new branch piwigo
[06:53:19]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: Update apps.toml ([bcfb4f26](https://github.com/YunoHost/apps/commit/bcfb4f26ecb473e0409e83e50a601c7563443619))
[06:54:34]
<Yunohost Git/Infra notifications> [apps] ericgaspar opened [pull request #2404](https://github.com/YunoHost/apps/pull/2404): Update apps.toml
[08:43:33]
<Salamandar> yeah okay so i don’t understand fully the config_panel.toml i think
[08:43:33]
<Salamandar> https://ci-apps-bookworm.yunohost.org/ci/job/5263
[08:46:30]
<Aleks (he/him/il/lui)> > <@Salamandar:matrix.org> yeah okay so i don’t understand fully the config_panel.toml i think
in https://github.com/YunoHost-Apps/mautrix_telegram_ynh/commit/a6dd24cf0d4012181aac7d1c478db7ba333042e0 you moved the "ask" and "type" keys to a section
[08:46:57]
<Aleks (he/him/il/lui)> but a section is not a question so this doesnt make sense
[08:47:40]
<Aleks (he/him/il/lui)> the initial thing with `[advanced.help.text]` seemed good, but probably the section shoulnt be named `help`, so maybe something like `[advanced.helptext.text]` idk
[08:56:30]
<Salamandar> yeah alright
[08:56:39]
<Salamandar> should be fixed
[09:00:19]
<@chrichri:ween.de> > <@thovi98:matrix.org> If you update the app, it's true, but if you change something in the yunohost/apps repo, it will be updated on friday
Hm, the change to the yunohost/apps repo happened only once and since then it is displayed as broken. I thought the status would update automatically when CI runs for the app?
Anyway - if I update (let's say to ynhX+1) then it would update on the catalog within ~4h?
[09:03:51]
<Aleks (he/him/il/lui)> the catalog is rebuilt every ~4hours, but CI levels are updated every friday night (more specifically it opens a PR, cf for example https://github.com/YunoHost/apps/pull/2393 )
[09:04:22]
<Aleks (he/him/il/lui)> (right now it's hanging because we're waiting for the go helper fix (smol poke Salamandar ))
[09:05:21]
<Salamandar> hmmm
[09:05:25]
<Salamandar> it’s done… ?
[09:06:48]
<Salamandar> can confirm https://github.com/YunoHost/yunohost/commits/dev/
[09:07:00]
<Salamandar> > helpers/go: fix missing git fetch
[09:07:36]
<Aleks (he/him/il/lui)> but the apps were having a bug on .14 nonetheless x_x
[09:07:41]
<Salamandar> oh ?
[09:07:43]
<Salamandar> let me see
[09:08:05]
<Aleks (he/him/il/lui)> > <@Alekswag:matrix.org> (poke Salamandar if that means something to you 😬)
(^)
[09:08:31]
<Aleks (he/him/il/lui)> > <@Salamandar:matrix.org> was not available at aaall, gonna fully fix this on monday
😬
[09:14:39]
<Salamandar> https://github.com/go-nv/goenv/releases/tag/2.2.0
[09:14:41]
<Salamandar> that’s not sure
[09:15:10]
<Aleks (he/him/il/lui)> what's not sure ?
[09:15:39]
<Salamandar> oops wrong chan for the 2nd message
[09:15:46]
<Salamandar> this goenv release is broken i guess
[09:15:49]
<Salamandar> …
[09:15:53]
<Aleks (he/him/il/lui)> 😢
[09:16:53]
<Salamandar> i thought it was only a "fluke" of the master branch… but looks like it’s a real change
[09:17:15]
<Aleks (he/him/il/lui)> ```
++ goenv latest --print 1.20
goenv: no versions installed
+ local final_go_version=
```
is it related to this ?
[09:18:27]
<Salamandar> yes
[10:08:09]
<Salamandar> ok found out the underlying issue and a fix
[10:12:11]
<Salamandar> grmbl
[10:12:14]
<Salamandar> that’s why we can’t have nice things
[10:12:28]
<Salamandar> `goenv latest`, before, was calling the plugin goenv-latest (git style)
[10:12:45]
<Salamandar> now, it’s just assuming you want to call `goenv local latest`, e.g mark as local version the latest installed…
[10:13:34]
<Salamandar> https://github.com/go-nv/goenv/blob/c5364af1d725bcb3f2c64828b5a5eadb5c96aa8b/libexec/goenv#L142
[10:14:18]
<Aleks (he/him/il/lui)> :|
[11:39:03]
<Salamandar> nice, zero outdated tests
[11:39:03]
<Salamandar> https://apps.yunohost.org/dash?filter=outdated
[11:41:32]
<Yunohost Git/Infra notifications> App dex failed all tests in job [#27182](https://ci-apps.yunohost.org/ci/job/27182) :(
[11:47:42]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/dolibarr_ynh/blob/master/scripts/install#L64
[11:47:48]
<Aleks (he/him/il/lui)> `ynh_exec_fully_quiet sleep 5`
[11:47:50]
<Aleks (he/him/il/lui)> wtf
[11:47:58]
<Aleks (he/him/il/lui)> is the command snorring or something ?
[11:48:10]
<Salamandar> lol
[11:48:34]
<Aleks (he/him/il/lui)> we should really have a "wat" video but about bash and yunohost scripts
[11:48:48]
<Salamandar> o no
[11:48:53]
<Salamandar> rebuilding the ci images
[11:49:27]
<Salamandar> is it image_builder rebuild stable bullseye ?
[11:54:45]
<Aleks (he/him/il/lui)> yeah, check `image_builder --help`
[11:55:59]
<Salamandar> yeah
[11:56:08]
<Salamandar> but some said last week that i rebuilt with testing ?
[12:01:05]
<Aleks (he/him/il/lui)> nah, this was the changelog
[12:01:22]
<Aleks (he/him/il/lui)> https://github.com/YunoHost/yunohost/blob/dev/debian/changelog#L10
[12:01:31]
<Aleks (he/him/il/lui)> there's a "stable/testing/unstable/whatever" flag in the changelog
[12:02:05]
<Aleks (he/him/il/lui)> but then when building the .deb (on forge2, not devbaseimg), you can build it for stable/testing/unstable
[12:02:25]
<Aleks (he/him/il/lui)> but then later, in `yunohost --version`, we want to display if the version is from stable, testing, unstable
[12:02:46]
<Aleks (he/him/il/lui)> but there's no trivial way to know if a package was installed from the stable, testing, unstable apt "channel"
[12:02:56]
<Aleks (he/him/il/lui)> so instead the python code relies on the changelog
[12:03:21]
<Aleks (he/him/il/lui)> tl;dr we should have a check in the vinaigrette build script that the changelog info is consistent with the apt channel
[12:24:08]
<Salamandar> ooooh oupsies i guess
[12:24:22]
<Aleks (he/him/il/lui)> yeah not a huge deal but people are easily panicking because it's written "testing zomg!!1!?"
[12:24:23]
<Aleks (he/him/il/lui)> anyway i fixed the make_changelog script
[12:24:46]
<Salamandar> oh yes finally it succeeds https://ci-apps.yunohost.org/ci/job/27182
[13:11:27]
<Yunohost Git/Infra notifications> App fider stays at level 3 in job [#27079](https://ci-apps.yunohost.org/ci/job/27079)
[14:11:47]
<Aleks (he/him/il/lui)> Salamandar: btw are you aware this script exists ? ;P https://github.com/YunoHost/yunohost/blob/bookworm/doc/generate_json_schema.py
[14:11:57]
<Aleks (he/him/il/lui)> (only in bookworm though because it relies on pydantic)
[14:12:37]
<Salamandar> hmmm
[14:12:38]
<Salamandar> no i was not
[14:14:00]
<Aleks (he/him/il/lui)> (axolotle doesnt know if it works really well, might need a bit of polishing but on the mid-term this means we can autogenerate it)
[14:17:26]
<Salamandar> but technically what's the point ?
[14:18:00]
<Aleks (he/him/il/lui)> well it's autogenerated from the code so you don't have to manually maintain the schema ?
[14:19:23]
<Salamandar> ah, well
[14:19:48]
<Salamandar> okay
[15:20:54]
<Salamandar> erf
[15:20:55]
<Salamandar> https://github.com/YunoHost-Apps/incus_ynh/blob/e7fd56e0e4692b50f93fbdc6f10af8d527430282/manifest.toml#L47
[15:20:57]
<Salamandar> this doesn't work
[15:21:09]
<Salamandar> i mean it works in bullseye but not bookworm
[15:21:32]
<Salamandar> ```
key: values for key, values in self.extras.items() if values["packages"]
~~~~~~^^^^^^^^^^^^
KeyError: 'packages'
```
[15:26:11]
<Salamandar> or hum
[15:26:20]
<Salamandar> idk
[15:31:20]
<Salamandar> that's weird though, the code looks vaid
[15:34:43]
<Salamandar> ah ah found the bug
[15:34:58]
<Aleks (he/him/il/lui)> 🤔
[15:34:58]
<Salamandar> there's a bug if there are more than 1 extras apt source…
[15:35:06]
<Aleks (he/him/il/lui)> wut
[15:36:59]
<Salamandar> ye
[15:37:04]
<Aleks (he/him/il/lui)> (why is it even a `packages_from_raw_bash` anyway instead of a regular list ?)
[15:37:07]
<Aleks (he/him/il/lui)> also I think in the `repo` key now we can use `__YNH_DEBIAN_VERSION__`
[15:37:19]
<Salamandar> > <@Alekswag:matrix.org> (why is it even a `packages_from_raw_bash` anyway instead of a regular list ?)
https://github.com/YunoHost/yunohost/pull/1869
[15:37:38]
<Aleks (he/him/il/lui)> wtf
[15:37:42]
<Salamandar> > <@Alekswag:matrix.org> (why is it even a `packages_from_raw_bash` anyway instead of a regular list ?)
because it needs to be conditional
[15:37:46]
<Aleks (he/him/il/lui)> is it in production right no ?
[15:37:54]
<Salamandar> > <@Alekswag:matrix.org> also I think in the `repo` key now we can use `__YNH_DEBIAN_VERSION__`
ah well that would help indeed :D
[15:38:33]
<Aleks (he/him/il/lui)> ah that's for apt repo hm
[15:39:55]
<Salamandar> > <@Alekswag:matrix.org> is it in production right now ?
yeah that's "old" code
[15:39:59]
<Salamandar> just, i think theres no app with multiple extras?
[15:40:04]
<Aleks (he/him/il/lui)> yeah maybe
[15:40:09]
<Salamandar> hmm
[15:40:12]
<Salamandar> looks like there is though
[15:40:15]
<Salamandar> peertube-search-index/manifest.toml is the only manifest with 2 apt.extras
[15:40:17]
<Aleks (he/him/il/lui)> uuuh owntracks
[15:40:19]
<Salamandar> no ?
[15:40:21]
<Salamandar> ah yes now
[15:40:22]
<Salamandar> indeed
[15:40:23]
<Salamandar> hmm
[15:40:25]
<Salamandar> that's weird
[15:40:31]
<Aleks (he/him/il/lui)> bonfire, ogod
[15:40:42]
<Salamandar> oh wait
[15:41:03]
<Salamandar> the bug just triggers when using packages_from_raw_bash in the second apt.extras
[15:41:10]
<Aleks (he/him/il/lui)> :|
[15:41:29]
<Salamandar> because the bug occurs when "packages" does not exist by default in the manifest (but is filled by the bash command)
[15:43:01]
<Salamandar> anyways
[15:43:34]
<Salamandar> let's merge the fix and i'll use `__YNH_DEBIAN_VERSION__` for incus
[15:50:46]
<Aleks (he/him/il/lui)> so the fix is the right one ?
[15:54:05]
<Yunohost Git/Infra notifications> [my_webapp_ynh] CodeShakingSheep [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/131#issuecomment-2158733208) on [issue #131](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/131) When changing PHP version, all dependencies are not reinstalled: I also encountered a strange behavior regarding PHP versions. When PHP was upgraded to version 8.0 I have this content i...
[16:13:50]
<Salamandar> > <@Alekswag:matrix.org> so the fix is the right one ?
yes
[16:14:09]
<Salamandar> you can see it's a "for extra in extras.items()" inside a similar loop
[16:14:11]
<Salamandar> that's fishy
[16:45:37]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([4d1bd6ce](https://github.com/YunoHost/apps/commit/4d1bd6cefe0d8a3582ea6b979673c112bb0b4fda))
[16:45:37]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2393](https://github.com/YunoHost/apps/pull/2393): Update app levels according to CI results
[16:45:42]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([ba9b9d8c](https://github.com/YunoHost/apps/commit/ba9b9d8cd2bf9c0197e666000d06e0eed4ff62b7))
[16:45:46]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2393](https://github.com/YunoHost/apps/pull/2393): Update app levels according to CI results
[16:45:49]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: Update apps.toml ([ba9b9d8c](https://github.com/YunoHost/apps/commit/ba9b9d8cd2bf9c0197e666000d06e0eed4ff62b7))
[16:45:56]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([6c0d8de2](https://github.com/YunoHost/apps/commit/6c0d8de2f831653dd427cfa5a62b44bdc0d9a83d))
[16:45:58]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2393](https://github.com/YunoHost/apps/pull/2393): Update app levels according to CI results
[16:46:11]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: Update apps.toml ([6c0d8de2](https://github.com/YunoHost/apps/commit/6c0d8de2f831653dd427cfa5a62b44bdc0d9a83d))
[16:46:13]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2393](https://github.com/YunoHost/apps/pull/2393): Update app levels according to CI results
[16:46:23]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([f0d46d43](https://github.com/YunoHost/apps/commit/f0d46d43abc12327e92fc403d03ed4766f641370))
[16:46:55]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([1693bb6a](https://github.com/YunoHost/apps/commit/1693bb6ae4a1f8c1eaf4d0f8289795bdee78afc8))
[16:47:03]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: Update apps.toml ([f0d46d43](https://github.com/YunoHost/apps/commit/f0d46d43abc12327e92fc403d03ed4766f641370))
[16:47:07]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: Update apps.toml ([1693bb6a](https://github.com/YunoHost/apps/commit/1693bb6ae4a1f8c1eaf4d0f8289795bdee78afc8))
[16:47:16]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2393](https://github.com/YunoHost/apps/pull/2393): Update app levels according to CI results
[16:47:28]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2393](https://github.com/YunoHost/apps/pull/2393#pullrequestreview-2108293593) Update app levels according to CI results
[16:48:41]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Update app levels according to CI results (#2393) * Update app levels according to CI results * Update apps.toml * Up... ([b72f2a2d](https://github.com/YunoHost/apps/commit/b72f2a2da82abcf9a5cb647297204e0d2ed4e8ed))
[16:48:42]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2393](https://github.com/YunoHost/apps/pull/2393): Update app levels according to CI results
[16:48:43]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch update_app_levels
[18:51:39]
<Aleks (he/him/il/lui)> (please refrain from pinging specific people as if it's their personal responsability, this can induce unecessary stress/pressure, this is a community project, many people can maintain apps)
[18:52:24]
<Gavi> apologies if this is the wrong place for this, but a major security vulnerability was just disclosed for conduit and a fix is rolling out this wednesday. i can help test again if needed
[18:52:51]
<Gavi> my apologies! removed the ping
[18:54:43]
<Aleks (he/him/il/lui)> i don't know much about this app but i guess the relevant stuff to update is https://github.com/YunoHost-Apps/conduit_ynh/blob/testing/manifest.toml#L61
[18:55:32]
<Aleks (he/him/il/lui)> it sounds weird that it seems to point to the latest "master" commit/artifact but with a fixed checksum ?
[18:55:44]
<Gavi> conduit is a bit strange with the way they handle releases
[19:01:57]
<Gavi> I'll tinker around a bit when the update drops, and potentially add a draft for coturn support settings to it if i can get the chance to as well. im not *super* accustomed to bash and yunohost packaging setup but theres no better time to learn than now
[19:30:18]
<eric_G> https://famedly.gitlab.io/conduit/deploying/generic.html
[19:45:52]
<tituspijean> Damn I was so sure it had the autoupdater enabled that I have programmed it to run 5 minutes after their announced release....
[19:45:52]
<tituspijean> ... only to find out it does *not* have the autoupdater 😅
[23:31:51]
<Yunohost Git/Infra notifications> App chuwiki goes down from level 7 to 4 in job [#27355](https://ci-apps.yunohost.org/ci/job/27355)