[00:09:33]
<orhtej2> > <@Alekswag:matrix.org> can you point to the corresponding log ?
https://ci-apps.yunohost.org/ci/job/25757
[00:10:37]
<orhtej2> If you look into raw log it installed litespeed to provide runtime which fails to start as it's a full on http server
[00:11:06]
<orhtej2> using fpm drops that wtf dependency
[00:16:27]
<Aleks (he/him/il/lui)> wtf
[00:16:28]
<Aleks (he/him/il/lui)> otherwise the phpX.Y-fpm dep is added here in the helpers: https://github.com/YunoHost/yunohost/blob/dev/helpers/helpers.v2.1.d/apt#L65
[00:16:28]
<Aleks (he/him/il/lui)> aaaaaah urhg i see it's the pre-install by package_check urgh
[00:16:29]
<Aleks (he/him/il/lui)> but not in the preinstall from CI but i guess we can add something similar such that this prevent this weird issue purely related to CI
[00:55:18]
<orhtej2> It's ci only?
[00:58:29]
<orhtej2> Autoupdate where?
[01:01:22]
<orhtej2> Still no harm in requiring php-fpm I guess 🤷
[01:02:38]
<trendless> Thanks for catching and fixing that, orhtej2 🙌
[06:30:40]
<Yunohost Git/Infra notifications> [calibreweb_ynh] RamanMalykhin [commented](https://github.com/YunoHost-Apps/calibreweb_ynh/issues/160#issuecomment-3691728189) on [issue #160](https://github.com/YunoHost-Apps/calibreweb_ynh/issues/160) OPDS Authentication Broken in Yunohost CalibreWeb Package: I did some digging on this issue, with the generous help of the people in Matrix support chat. It is caused by SSOWat (Y...
[06:31:08]
<Yunohost Git/Infra notifications> [calibreweb_ynh] RamanMalykhin [commented](https://github.com/YunoHost-Apps/calibreweb_ynh/issues/160#issuecomment-3691728189) on [issue #160](https://github.com/YunoHost-Apps/calibreweb_ynh/issues/160) OPDS Authentication Broken in Yunohost CalibreWeb Package: I did some digging on this issue, with the generous help of the people in Matrix support chat. It is caused by SSOWat (Y...
[06:32:59]
<Yunohost Git/Infra notifications> [calibreweb_ynh] RamanMalykhin [commented](https://github.com/YunoHost-Apps/calibreweb_ynh/issues/160#issuecomment-3691728189) on [issue #160](https://github.com/YunoHost-Apps/calibreweb_ynh/issues/160) OPDS Authentication Broken in Yunohost CalibreWeb Package: I did some digging on this issue, with the generous help of the people in Matrix. It is caused by SSOWat (Yunohosts SSO...
[08:22:25]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #3262](https://github.com/YunoHost/apps/pull/3262): Create textarea.png
[08:22:26]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed to main: Create textarea.png (#3262) ([44e1d5a3](https://github.com/YunoHost/apps/commit/44e1d5a38ecd75055950c66b849841226422ec34))
[08:22:27]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch textarea-logo
[09:29:20]
<eric_G> I can't find if Node is reinstalled in the restore script (after the `[resources.nodejs]` beeing added to the manifest)? -> https://github.com/YunoHost-Apps/tiddlywiki_ynh/issues/69
[10:21:24]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T merged [pull request #597](https://github.com/YunoHost-Apps/synapse_ynh/pull/597): Testing
[11:07:05]
<Yunohost Git/Infra notifications> [calibreweb_ynh] Krakinou [commented](https://github.com/YunoHost-Apps/calibreweb_ynh/issues/160#issuecomment-3692705320) on [issue #160](https://github.com/YunoHost-Apps/calibreweb_ynh/issues/160) OPDS Authentication Broken in Yunohost CalibreWeb Package: Thanks a lot for your research
A PR would be nice indeed
[11:45:03]
<Yunohost Git/Infra notifications> [freshrss_ynh] ericgaspar pushed to testing: Testing (#216) * Update manifest.toml * cleaning * Upgrade to v1.27.1 (#215) * Upgrade sources - main v1.27.1: htt... ([f869d3a0](https://github.com/YunoHost-Apps/freshrss_ynh/commit/f869d3a065d740b5b380e03a51237b07952f1624))
[11:45:07]
<Yunohost Git/Infra notifications> [freshrss_ynh] ericgaspar pushed to testing: Merge branch master into testing ([0f1f2c0d](https://github.com/YunoHost-Apps/freshrss_ynh/commit/0f1f2c0d9b5dc3ffec74caef36ccbb8bd078c169))
[11:53:28]
<m606> idk for sure, but at least in the current version of that package (where nodejs helper is used in install and restore scripts and not as a resource), it is being reinstalled during restore: https://paste.yunohost.org/yepimoligu.txt
and the backup archive contains settings.yml with `nodejs_version: 22.21.1` (or I guess whatever version was installed during backup).
[12:20:09]
<m606> the settings.yml doesn't seem to matter though - I've tried forging one with 22.18.0 as the OP and restoring, still I've got 22.21.1 installed (and in __PATH_WITH_NODEJS__) and properly executing - could not reproduce the issue on YNH 12.1.37
[12:20:15]
<Aleks (he/him/il/lui)> the restore script will reprovision the nodejs resource according to what's in the manifest yes
[12:22:58]
<m606> the settings.yml doesn't seem to matter though - I've tried forging one with 22.18.0 as the OP and restoring, still I've got 22.21.1 installed (and `__PATH_WITH_NODEJS__` in tiddlywiki.service is replaced with 22.21.1 too) and properly executing - could not reproduce the issue on YNH 12.1.37
[13:22:56]
<Yunohost Git/Infra notifications> [freshrss_ynh] ericgaspar pushed to master: Testing (#219) * Update manifest.toml * cleaning * Upgrade to v1.27.1 (#215) * Upgrade sources - main v1.27.1: htt... ([82f94674](https://github.com/YunoHost-Apps/freshrss_ynh/commit/82f946742e929a0706aa0d981a61a5414f92b525))
[13:22:57]
<Yunohost Git/Infra notifications> [freshrss_ynh] ericgaspar merged [pull request #219](https://github.com/YunoHost-Apps/freshrss_ynh/pull/219): Testing
[13:28:46]
<eric_G> I tested backing up and restoring TiddlyWiki, and encountered the same error as the above user... https://paste.yunohost.org/jicunuvese
[13:49:12]
<m606> hmm app originally installed from the catalog in (version 5.3.8~ynh1) ? on YNH 12.1.37? nothing interesting in the restoration log?
[14:08:05]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 just made [pull request #225](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/225) ready for review: Upgrade to v2.6.14
[14:08:16]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 merged [pull request #225](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/225): Upgrade to v2.6.14
[14:08:17]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 deleted branch ci-auto-update-2.6.10
[14:08:38]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 closed [pull request #243](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/243): Upgrade to v2.6.14
[14:08:41]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 deleted branch ci-auto-update-2.6.14
[14:09:50]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 opened [pull request #244](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/244): v2.6.14
[14:10:16]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 pushed to testing: Testing (#234) * Update nginx.conf * Auto-update READMEs * cleaning * Update manifest.toml * Auto-update READMEs -... ([c68245e7](https://github.com/YunoHost-Apps/wallabag2_ynh/commit/c68245e75c068a2d8d333f4de4f6f008829b655f))
[14:10:17]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 pushed to testing: Merge branch master into testing ([4085accb](https://github.com/YunoHost-Apps/wallabag2_ynh/commit/4085accbc678a77f933427e3416759b434f72c9a))
[14:12:13]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 [commented](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/244#issuecomment-3692926973) on [issue #244](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/244) v2.6.14: Manually tested scenarios (all passing): - Log in with SSO (user logged in, app goes to main screen) - Add new page to r...
[14:13:18]
<orhtej2> I'll take a second opinion but IMO Wallabag lives and breaths again! https://github.com/YunoHost-Apps/wallabag2_ynh/pull/244/
TY to all who worked on this update 😻
credits go to miro5001 I believe who fixed the patches one last time
[14:41:36]
<m606> i installed 2.5.4~ynh12 from catalog on YNH 12.1.37. I was able to login to wallabag and did nothing in it.
then I `sudo yunohost app upgrade wallabag2 -u https://github.com/YunoHost-Apps/wallabag2_ynh/tree/testing`, I got a strange warning message saying that I would have manually modified a file: https://paste.yunohost.org/usoruwonix.txt, but upgrade was successful - able to login to wallabag again and saw version 2.6.14
[15:17:14]
<orhtej2> > <@m606:matrix.org> i installed 2.5.4~ynh12 from catalog. I was able to login to wallabag and did nothing in it.
> then I `sudo yunohost app upgrade wallabag2 -u https://github.com/YunoHost-Apps/wallabag2_ynh/tree/testing`, I got a strange warning message saying that I would have manually modified a file: https://paste.yunohost.org/usoruwonix.txt, but upgrade was successful - able to login to wallabag again and saw version 2.6.14
Same, idk what that env change is about 🤷
[15:59:09]
<m606> the modifications which are mentioned on parameters.yml in the log when reapplying config seems to be matching the changes made in the PR to the default parameters.yml. I remember a mechanism that saves a checksum of the file to detect changes in conf file (checksum saved in the apps' settings.yml ?) . Here in my case at least (as it was a fresh install of Wallabag) it seems it is the upgrade script of the testing branch which changed the parameters.yml. Maybe the upgrade script should also change the checksum in the settings.
But in terms of migration, the case where someone would actually have manually modified that file before upgrade should be tested as well I assume.
[16:00:37]
<m606> the modifications which are mentioned on parameters.yml in the log when reapplying config seems to be matching the changes made in the PR to the default parameters.yml. I remember a mechanism that saves a checksum of the file to detect changes in conf file (checksum saved in the apps' settings.yml ?) . Here in my case at least (as it was a fresh install of Wallabag) it seems it is the upgrade script of the testing branch which changed the parameters.yml. Maybe the upgrade script should also change the checksum in the settings?
But in terms of migration, the case where someone would actually have manually modified that file before upgrade should be tested as well I assume.
[16:03:36]
<orhtej2> > <@m606:matrix.org> the changed which are mentioned in the log when reapplying config seems to be matching the changes made in the PR to the default parameters.yml. I remember a mechanism that saves a checksum of the file to detect changes in conf file (checksum saved in the apps' settings.yml ?) . Here in my case at least (as it was a fresh install of Wallabag) it seems it is the upgrade script of the testing branch which changed the parameters.yml. Maybe the upgrade script should also change the checksum in the settings.
> But in terms of migration, the case where someone would actually have manually modified that file before upgrade should be tested as well I assume.
I mean original install script must havenchanged the config after applying the template
[16:10:08]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #3263](https://github.com/YunoHost/apps/pull/3263): Update app levels according to CI results
[16:10:57]
<Aleks (he/him/il/lui)> holy cow we're friday
[16:27:45]
<Yunohost Git/Infra notifications> [mastodon_ynh] Thatoo closed [issue #531](https://github.com/YunoHost-Apps/mastodon_ynh/issues/531): Cant update from 4.3.8 to 4.4.8
[17:20:31]
<miro5001> The upgrade seems to alter the config file, need to find if there are new settings that have been added to it. And probably there is something to prevent composer from overwriting it : "This file is auto-generated during the composer install"
[17:24:15]
<miro5001> All the team worked on it, and that what makes yunoworld fantastic 😍
In fact, I have searched about integrating ldap to django and stumbled upon an article talking about integrating ldap to different frameworks, from link to link, got to that docs of ldaptools
[17:28:20]
<Yunohost Git/Infra notifications> [wallabag2_ynh] DeMiro5001 edited [pull request #242](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/242): Using the FOSUserBundle provider to find the users
[17:29:14]
<miro5001> I've renamed the PR so searching for it would be easy in the future
[17:42:17]
<miro5001> Found it here https://github.com/wallabag/wallabag/blob/dcba61ca5cdf940d6f6ffa39ae74d84f0ff953a0/composer.json#L235
[17:43:37]
<miro5001> I think we will also have to change composer.json to remove it from being updated
[18:21:20]
<miro5001> [@davidbgk:matrix.org](https://matrix.to/#/@davidbgk:matrix.org) can you please have a look at local.py here https://github.com/YunoHost-Apps/umap_ynh/blob/SSO/conf/local.py
Install is failing at `pip install ".[sync,yunohost]"` here https://ci-apps-dev.yunohost.org/ci/job/16020
Looks like local.py needs some fixing
[18:33:53]
<Yunohost Git/Infra notifications> [wallabag2_ynh] orhtej2 merged [pull request #244](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/244): v2.6.14
[19:13:27]
<Yunohost Git/Infra notifications> App wallabag2 goes down from level 8 to 6 in job [#26095](https://ci-apps.yunohost.org/ci/job/26095)
[19:32:21]
<miro5001> the only difference I see is 'wallabag_user_agent' has been removed, everything is ok
[19:35:03]
<Yunohost Git/Infra notifications> [wallabag2_ynh] DeMiro5001 pushed to patch-upgrade: Create composer.json ([f241b050](https://github.com/YunoHost-Apps/wallabag2_ynh/commit/f241b0505a6ba3ae9704cbb08a1da14967020cd5))
[19:35:07]
<Yunohost Git/Infra notifications> [wallabag2_ynh] DeMiro5001 opened [pull request #245](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/245): Create composer.json
[19:39:10]
<Yunohost Git/Infra notifications> [wallabag2_ynh] DeMiro5001 pushed to patch-upgrade: Update upgrade ([66717551](https://github.com/YunoHost-Apps/wallabag2_ynh/commit/66717551fc6373a9765c9871fe215cf1fa2be276))
[20:05:09]
<Yunohost Git/Infra notifications> [wallabag2_ynh] DeMiro5001 pushed to patch-upgrade: Update composer.json ([ec0ce579](https://github.com/YunoHost-Apps/wallabag2_ynh/commit/ec0ce579cfbf0dd1e36f913ee1aa7de907ff0eb5))
[20:29:55]
<Yunohost Git/Infra notifications> [wallabag2_ynh] DeMiro5001 closed [pull request #245](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/245): Fix upgrade
[20:29:56]
<Yunohost Git/Infra notifications> [wallabag2_ynh] DeMiro5001 [commented](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/245#issuecomment-3693334337) on [issue #245](https://github.com/YunoHost-Apps/wallabag2_ynh/pull/245) Fix upgrade: This not working as I expected. Im closing.
[21:36:12]
<orhtej2> how does remove everything except files 'a', 'b' and folder 'c' from $install_dir while NOT using ynh_setup_source?
[21:36:31]
<orhtej2> (other than copy paste the helper I guess 🤷)
[21:41:21]
<m606> "how can we" you mean?
[21:41:58]
<orhtej2> 'how does one', yes
[21:42:09]
<orhtej2> I just added 3 mv commands, screw it
[21:43:06]
<orhtej2> should we use `local keep_dir=/var/cache/yunohost/files_to_keep_during_setup_source/${YNH_APP_ID}` or `mktmp` is good enough?
[21:49:47]
<m606> I would guess that local the keep_dir is safer if it is there (no other software should mess with this temp dir dedicated to YNH), but I don't know the history of it.
What pushes you to be willing to evade the helper though?
[21:50:46]
<m606> still wallabag?
[21:52:18]
<orhtej2> no, Benevalibre
[21:53:01]
<orhtej2> it switched from sources-based install to pip-based one and I need to clean up the previous install except for uploaded assets
[21:53:22]
<orhtej2> upstream no longer allows downloading tarballs, probably to deter scrappers
[21:58:07]
<m606> in upgrade script, right ?
[21:58:29]
<orhtej2> yes I guess?
[21:59:03]
<orhtej2> I need to migrate from $install_dir full of stuff to $install_dir with `config.env`, `var` and `venv` dirs
[21:59:24]
<orhtej2> and existence of `benevalibre` dir inherited from old install breaks stuff
[22:13:00]
<m606> hmm i was checking the helper, but I don't have much to tell you sorry.
for what is to keep it uses that cache dir made in YNH, but at source files it also [uses mktemp](https://github.com/YunoHost/yunohost/blob/b91ab52deb47b942572790cc4b4b36b4b15ec365/helpers/helpers.v2.1.d/sources#L219)
I believe there is a reason, but I don't know much about how the cache dirs is managed in YNH, and mktemp should work fine...
[22:15:10]
<m606> hmm i was checking the helper, but I don't have much to tell you sorry.
for what is to keep it uses that cache dir made in YNH, but it unzips source files using [uses mktemp] (https://github.com/YunoHost/yunohost/blob/b91ab52deb47b942572790cc4b4b36b4b15ec365/helpers/helpers.v2.1.d/sources#L219) - as if precious vs. non-precious.
I believe there is a reason, but I don't know much about how the cache dirs is managed in YNH, and mktemp should work fine...
[22:15:32]
<m606> hmm i was checking the helper, but I don't have much to tell you sorry.
for what is to keep it uses that cache dir made in YNH, but it unzips source files using [uses mktemp](https://github.com/YunoHost/yunohost/blob/b91ab52deb47b942572790cc4b4b36b4b15ec365/helpers/helpers.v2.1.d/sources#L219) - as if precious vs. non-precious.
I believe there is a reason, but I don't know much about how the cache dirs is managed in YNH, and mktemp should work fine...
[22:26:44]
<orhtej2> > <@m606:matrix.org> hmm i was checking the helper, but I don't have much to tell you sorry.
> for what is to keep it uses that cache dir made in YNH, but at source files it also [uses mktemp](https://github.com/YunoHost/yunohost/blob/b91ab52deb47b942572790cc4b4b36b4b15ec365/helpers/helpers.v2.1.d/sources#L219)
> I believe there is a reason, but I don't know much about how the cache dirs is managed YNH, and mktemp should work fine...
The files are kept [somewhere else](https://github.com/YunoHost/yunohost/blob/b91ab52deb47b942572790cc4b4b36b4b15ec365/helpers/helpers.v2.1.d/sources#L189), tmp is for archives
[22:32:17]
<miro5001> Doesn't "pip install -u package" overwrite old files?
[22:39:29]
<orhtej2> > <@miro5001:matrix.org> Doesn't "pip install -u package" overwrite old files?
well yes but actually no 🤷
[23:13:33]
<miro5001> ynh_safe_rm doesn't accept exceptions
But while searching I found a lot of options :
1. GLOBIGNORE : https://blog.cclaude.rocks/post/2025/09/03.glob-bash.html
2. extglob : https://www.linuxjournal.com/content/bash-extended-globbing
[23:15:33]
<miro5001> ynh_safe_rm !(config.env|var|venv)
[23:25:54]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 37 pending update PRs
- 8 new apps PRs: filerise, homarr, homebox, jackett, minetest, psitransfer, trilium, wiki-go
- 5 failed apps updates: benevalibre, fluffychat, lasuite-meet, textarea, vert
See the full log here: https://paste.yunohost.org/raw/izotubexeh
Autoupdate dashboard: https://apps.yunohost.org/dash?filter=autoupdate