[00:22:18]
<Yunohost Git/Infra notifications> App zerotier rises from level 7 to 8 in job [#16944](https://ci-apps.yunohost.org/ci/job/16944) !
[04:23:45]
<Yunohost Git/Infra notifications> App etherpad_mypads goes down from level 8 to 6 in job [#16948](https://ci-apps.yunohost.org/ci/job/16948)
[07:17:55]
<Yunohost Git/Infra notifications> App cypht goes down from level 8 to 1 in job [#16954](https://ci-apps.yunohost.org/ci/job/16954)
[08:12:51]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create gossa.png ([3db14eea](https://github.com/YunoHost/apps/commit/3db14eea927e96594d7cc5983be327fe7a0310a7))
[08:49:47]
<lapineige> It was already done in fact
[09:14:54]
<Yunohost Git/Infra notifications> [apps] @lapineige opened [pull request #1694](https://github.com/YunoHost/apps/pull/1694): Add Peertube remote runners
[09:16:00]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited review [pull request #1694](https://github.com/YunoHost/apps/pull/1694#pullrequestreview-1520872811): Add Peertube remote runners
[09:16:01]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1694#discussion_r1257445488) on pull request #1694 Add Peertube remote runners: suggestion
[09:16:45]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1694](https://github.com/YunoHost/apps/pull/1694#pullrequestreview-1520872946) Add Peertube remote runners
[09:16:53]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1694](https://github.com/YunoHost/apps/pull/1694): Add Peertube remote runners
[09:16:54]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 3 commits to master ([3db14eea927e...abf5356dd20e](https://github.com/YunoHost/apps/compare/3db14eea927e...abf5356dd20e))
[09:16:57]
<Yunohost Git/Infra notifications> [apps/master] Add Peertube remote runners - lapineige
[09:17:01]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[09:17:03]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1694 from lapineige/patch-8 Add Peertube remote runners - Éric Gaspar
[09:17:16]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1691](https://github.com/YunoHost/apps/pull/1691#pullrequestreview-1520873008) Osada is deprecated
[09:17:21]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1691](https://github.com/YunoHost/apps/pull/1691): Osada is deprecated
[09:17:22]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to master ([abf5356dd20e...f183b2642690](https://github.com/YunoHost/apps/compare/abf5356dd20e...f183b2642690))
[09:17:25]
<Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch tituspijean-patch-1
[09:17:27]
<Yunohost Git/Infra notifications> [apps/master] Osada is deprecated cf. https://github.com/YunoHost/issues/issues/2211 - tituspijean
[09:17:28]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1691 from YunoHost/tituspijean-patch-1 Osada is deprecated - Éric Gaspar
[09:18:27]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([49a398c3](https://github.com/YunoHost/apps/commit/49a398c37b67393b72737c5fde3abe1ff1d52a71))
[09:18:36]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1693](https://github.com/YunoHost/apps/pull/1693): Update app levels according to CI results
[09:31:17]
<Yunohost Git/Infra notifications> [apps] @lapineige [commented](https://github.com/YunoHost/apps/pull/1694#issuecomment-1627660087) on [issue #1694](https://github.com/YunoHost/apps/pull/1694) Add Peertube remote runners: @ericgaspar you rocks :D
[10:08:22]
<Yunohost Git/Infra notifications> App etherpad_mypads goes down from level 8 to 6 in job [#16948](https://ci-apps.yunohost.org/ci/job/16948)
[10:12:28]
<Yunohost Git/Infra notifications> App rocketchat failed all tests in job [#16725](https://ci-apps.yunohost.org/ci/job/16725) :(
[10:18:39]
<lapineige> In the app manifest, can we generate a random number ? I need it to register a worker name that has to be unique
[10:30:19]
<Yunohost Git/Infra notifications> App seafile failed all tests in job [#16821](https://ci-apps.yunohost.org/ci/job/16821) :(
[10:43:04]
<Aleks (he/him/il/lui)> > In the app manifest, can we generate a random number ? I need it to register a worker name that has to be unique
Zblerg
[10:43:09]
<Aleks (he/him/il/lui)> I dont think so
[10:43:30]
<Aleks (he/him/il/lui)> But you could hack something in the install script with like :
[10:45:02]
<Aleks (he/him/il/lui)> if [ $PACKAGE_CHECK_EXEC -eq 1]
then
runner=$(generate_random_stuff)
fi
[10:45:08]
<Aleks (he/him/il/lui)> And maybe also overriding the setting
[10:46:05]
<Aleks (he/him/il/lui)> So basically "if in CI context, ignore the value from the manifest and generate a random one instead"
[11:06:16]
<orhtej2> @lapineige config management in `peertube-runner` seems not ideal: https://github.com/sindresorhus/env-paths/blob/f1729272888f45f6584e74dc4d0af3aecba9e7e8/index.js#L36, perhaps this needs to be patched and built from source?
[11:06:44]
<orhtej2> it uses user profile dir as storage rather than ynh's dedicated `$data_dir`
[11:06:52]
<orhtej2> so backing up will be challenging
[13:37:19]
<Yunohost Git/Infra notifications> [apps] @ericgaspar created new branch directorylister
[13:58:59]
<lapineige> > <@Alekswag:matrix.org> So basically "if in CI context, ignore the value from the manifest and generate a random one instead"
I don't need it for CI, more for real life use
[13:58:59]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to directorylister: Create directorylister.png ([002286e5](https://github.com/YunoHost/apps/commit/002286e506f549ec10ea65746a8aaa6a51aae037))
[13:59:44]
<Aleks (he/him/il/lui)> ah, then yes just .. idk, use whichever method you want to generate a random string in the install script ..?
[13:59:54]
<lapineige> Yes
[14:00:02]
<Aleks (he/him/il/lui)> for example there's `ynh_string_random` ?
[14:00:09]
<lapineige> I wanted to check if there were a more standard ynh way
[14:00:19]
<lapineige> > <@Alekswag:matrix.org> for example there's `ynh_string_random` ?
Oh yes perfect
[14:01:11]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to directorylister: Update apps.toml ([9db38169](https://github.com/YunoHost/apps/commit/9db381694a87b5610cb1b2a158275acb4a605d06))
[14:01:12]
<Yunohost Git/Infra notifications> [apps] @ericgaspar opened [pull request #1695](https://github.com/YunoHost/apps/pull/1695): Add Directorylister
[14:04:07]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Yolo-add transversal test that config panel does work after install/upgrade/restore ([d0295731](https://github.com/YunoHost/package_check/commit/d02957319a3c8d75f6a6548602678a7dd0bb069d))
[14:06:14]
<Yunohost Git/Infra notifications> [apps] @alexAubin approved [pull request #1695](https://github.com/YunoHost/apps/pull/1695#pullrequestreview-1520913220) Add Directorylister
[14:06:45]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1695](https://github.com/YunoHost/apps/pull/1695): Add Directorylister
[14:06:45]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 3 commits to master ([f183b2642690...90aeee0b3a38](https://github.com/YunoHost/apps/compare/f183b2642690...90aeee0b3a38))
[14:06:50]
<Yunohost Git/Infra notifications> [apps/master] Create directorylister.png - Éric Gaspar
[14:06:50]
<Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch directorylister
[14:06:51]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[14:06:53]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1695 from YunoHost/directorylister Add Directorylister - Éric Gaspar
[14:09:00]
<Aleks (he/him/il/lui)> yay we're down to less than 10% untested apps on https://dash.yunohost.org/appci/compare/stable...bookworm o/
[14:09:33]
<Aleks (he/him/il/lui)> will try to start a spreadsheet somewhere with the type of issues for each apps
[14:10:29]
<Aleks (he/him/il/lui)> > <@yunohostinfra:matrix.org> [package_check] @alexAubin pushed 1 commit to master: Yolo-add transversal test that config panel does work after install/upgrade/restore ([d0295731](https://github.com/YunoHost/package_check/commit/d02957319a3c8d75f6a6548602678a7dd0bb069d))
FIY I added this to test that config panel don't trigger an error right after install, will keep an eye on jobs in the coming hours
[14:13:16]
<Yunohost Git/Infra notifications> Job [#16957](https://ci-apps.yunohost.org/ci/job/16957) for galene failed miserably :(
[14:33:55]
<lapineige> Oh I forgot something important : I need to store this value. Should I store it as an app setting ?
[14:34:00]
<lapineige> > Oh yes perfect
Oh I forgot something important : I need to store this value. Should I store it as an app setting ?
[14:34:23]
<Aleks (he/him/il/lui)> yes
[14:34:37]
<lapineige> ok thanks
[14:40:02]
<lapineige> > @lapineige config management in `peertube-runner` seems not ideal: https://github.com/sindresorhus/env-paths/blob/f1729272888f45f6584e74dc4d0af3aecba9e7e8/index.js#L36, perhaps this needs to be patched and built from source?
I don't get it
[15:13:27]
<orhtej2> > I don't get it
There's the piece of code that tells `peertube-runner` where to store settings, temp files etc. It currently points to user running the process home dir, perhaps it should point to `$data_dir`?
[16:16:09]
<Aleks (he/him/il/lui)> > <@Alekswag:matrix.org> yay we're down to less than 10% untested apps on https://dash.yunohost.org/appci/compare/stable...bookworm o/
Soooo as promised I worked on a spreadsheet that sums up the various issues (that's maybe not all failing apps, not 100% clear to me) :
https://sans-nuage.fr/file/s/X2sHNAxGzbHTab9
[16:17:24]
<Aleks (he/him/il/lui)> There's 35~40ish apps on which we should add explicit PHP/mysql dependency, which is "easy" but just long and boring
[16:18:21]
<Aleks (he/him/il/lui)> all apps related to Synapse (~7ish) fail because of `openssl dhparam` CLI change I think
[16:18:56]
<Aleks (he/him/il/lui)> 20ish apps with APT dependencies that disappeared, typically `libssl1.1` doesn't exists anymore apparently
[16:19:46]
<Aleks (he/him/il/lui)> ah and the handful of Ruby apps failing, but there doesn't seem to be an explicit error message so it's not clear wtf is happening
[16:20:07]
<orhtej2> > <@Alekswag:matrix.org> Soooo as promised I worked on a spreadsheet that sums up the various issues (that's maybe not all failing apps, not 100% clear to me) :
> https://sans-nuage.fr/file/s/X2sHNAxGzbHTab9
> pip install explodes
Nice 😂
[16:56:17]
<Yunohost Git/Infra notifications> App archivist failed all tests in job [#16688](https://ci-apps.yunohost.org/ci/job/16688) :(
[16:59:20]
<Yunohost Git/Infra notifications> Job [#16959](https://ci-apps.yunohost.org/ci/job/16959) for qr failed miserably :(
[17:03:57]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Fix config panel test ? ([35294090](https://github.com/YunoHost/package_check/commit/352940900154dd5d91754b3e267ff6d8da71de17))
[17:04:08]
<Yunohost Git/Infra notifications> Job [#16688](https://ci-apps.yunohost.org/ci/job/16688) for archivist failed miserably :(
[17:08:35]
<Yunohost Git/Infra notifications> App archivist failed all tests in job [#16688](https://ci-apps.yunohost.org/ci/job/16688) :(
[17:57:48]
<Yunohost Git/Infra notifications> App soapbox rises from level 6 to 8 in job [#16962](https://ci-apps.yunohost.org/ci/job/16962) !
[18:46:26]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create pepettes.png ([c37d67f5](https://github.com/YunoHost/apps/commit/c37d67f5f24f397db655e19f34257dbc3fbb835b))
[19:21:55]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([c442cfbf](https://github.com/YunoHost/apps/commit/c442cfbf0778a3639f89b08510f93dde3c5faab4))
[19:22:01]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1693](https://github.com/YunoHost/apps/pull/1693): Update app levels according to CI results
[19:22:37]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1693](https://github.com/YunoHost/apps/pull/1693#pullrequestreview-1520948344) Update app levels according to CI results
[19:22:41]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1693](https://github.com/YunoHost/apps/pull/1693): Update app levels according to CI results
[19:22:42]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 4 commits to master ([c37d67f5f24f...b9511ceb8a0f](https://github.com/YunoHost/apps/compare/c37d67f5f24f...b9511ceb8a0f))
[19:22:45]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - ericgaspar
[19:22:46]
<Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch update_app_levels
[19:22:48]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[19:22:51]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1693 from YunoHost/update_app_levels Update app levels according to CI results - Éric Gaspar
[19:58:34]
<tituspijean> > try adding `-r` to this line: https://github.com/YunoHost-Apps/dendrite_ynh/blob/0b50d69473710013cb8d42103eb352ce9022b960/scripts/upgrade#L63
>
> file `internal/sqlutil/trace.go` no longer exists in upstream.
thanks! I think that was the issue 🙂
[20:00:00]
<orhtej2> I still find it strange, the file was removed in 0.13.0 and I had no problem (?) with the upgrade, at least the build part
[20:00:10]
<orhtej2> and I see the file is not there on my machine and I changed nothing
[20:00:19]
<orhtej2> I think 🤔
[20:01:08]
<orhtej2> it definitely worked on other server where I'm 100% everyting is 100% stock, the one I'm writing from is a ghetto hack as it has little RAM and little storage
[20:03:13]
<orhtej2> > There's the piece of code that tells `peertube-runner` where to store settings, temp files etc. It currently points to user running the process home dir, perhaps it should point to `$data_dir`?
after double-looking you can manipulate the result by setting `$HOME` env variable to something more reasonable
[21:12:06]
<orhtej2> IMO bookwyrm incompatibility with bookworm is a showstopper
[21:12:06]
<orhtej2> > <@Alekswag:matrix.org> Soooo as promised I worked on a spreadsheet that sums up the various issues (that's maybe not all failing apps, not 100% clear to me) :
> https://sans-nuage.fr/file/s/X2sHNAxGzbHTab9
Proposed speculative fix for `bookwyrm`: https://github.com/YunoHost-Apps/bookwyrm_ynh/pull/38, how does one request bookworm (lol what a coincidence!) build?
[21:19:19]
<Aleks (he/him/il/lui)> a showstopper for ?
[21:20:28]
<Aleks (he/him/il/lui)> unfortunately there's no ci-apps-dev for bookworm, i think i'd say : if you're confident with your changes, merge to master somehow and relaunch the test from https://ci-apps-bookworm.yunohost.org/ci/apps/bookwyrm/
[21:21:52]
<Aleks (he/him/il/lui)> like, adding `python3-dev` to the apt deps seems clearly a "trivial" change, personally I would yolomerge it
[21:22:02]
<Aleks (he/him/il/lui)> ¯\_(ツ)_/¯
[21:22:15]
<Aleks (he/him/il/lui)> however, you should be careful to merge this in `testing` first, then `master`
[21:22:34]
<Aleks (he/him/il/lui)> otherwise people testing not-so-trivial changes from the `testing` branch wont have the new stuff from master
[21:35:32]
<orhtej2> > <@Alekswag:matrix.org> a showstopper for ?
showstopper for YNH 12, stupid joke, nvm
[21:45:47]
<orhtej2> > <@Alekswag:matrix.org> however, you should be careful to merge this in `testing` first, then `master`
Whoops, changed target branch
[23:26:10]
<Yunohost Git/Infra notifications> App concrete5 rises from level 6 to 8 in job [#16967](https://ci-apps.yunohost.org/ci/job/16967) !