[08:54:20]
<Cippr> genma: Hello,
Regarding https://github.com/YunoHost/apps/pull/2041 is it relevant for Slip to PR an autoupdate about something like that :
autoupdate.strategy = "latest_webpage_link"
autoupdate.upstream = "https://files.spip.net/spip/archives/"
autoupdate.version_regex = ".*/spip-v([0-9]+\\.[0-9]+\\.[0-9]+)\\.zip$"
[08:55:08]
<Cippr> genma: Hello,
Regarding https://github.com/YunoHost/apps/pull/2041 is it relevant for Spip to PR an autoupdate about something like that :
autoupdate.strategy = "latest_webpage_link"
autoupdate.upstream = "https://files.spip.net/spip/archives/"
autoupdate.version_regex = ".*/spip-v([0-9]+\\.[0-9]+\\.[0-9]+)\\.zip$"
[13:43:20]
<Yunohost Git/Infra notifications> [piped_ynh] orhtej2 deleted branch ci-auto-update-sources-02c33c44
[13:43:20]
<ljf> thatoo i am testing the new euro-office package
[13:43:23]
<Yunohost Git/Infra notifications> [baikal_ynh] vdbhb59 [commented](https://github.com/YunoHost-Apps/baikal_ynh/issues/51#issuecomment-5392898905) on [issue #51](https://github.com/YunoHost-Apps/baikal_ynh/issues/51) Restore Baikal settings after upgrade: https://forum.yunohost.org/t/update-of-baikal-overwrites-auth-setting/42863
[13:43:23]
<Yunohost Git/Infra notifications> [immich_ynh] zamentur [commented](https://github.com/YunoHost-Apps/immich_ynh/issues/432#issuecomment-5393514993) on [issue #432](https://github.com/YunoHost-Apps/immich_ynh/issues/432) Postgresql migration nightmare: About pg_vector, for your information, discourse need it too.
We may include it in repo.yunohost.org ?
[13:43:24]
<Yunohost Git/Infra notifications> [immich_ynh] zamentur [commented](https://github.com/YunoHost-Apps/immich_ynh/issues/432#issuecomment-5393514993) on [issue #432](https://github.com/YunoHost-Apps/immich_ynh/issues/432) Postgresql migration nightmare: About pg_vector, for your information, discourse need it too.
We may include it in repo.yunohost.org ?
[13:43:25]
<Yunohost Git/Infra notifications> [searxng_ynh] ewilly merged [pull request #531](https://github.com/YunoHost-Apps/searxng_ynh/pull/531) : Upgrade to v2026.08.22
[13:43:26]
<Yunohost Git/Infra notifications> [searxng_ynh] github-actions[bot] opened [pull request #532](https://github.com/YunoHost-Apps/searxng_ynh/pull/532) : Upgrade master from testing
[13:43:26]
<Yunohost Git/Infra notifications> [searxng_ynh] ewilly deleted branch ci-auto-update-2026.08.22
[13:43:27]
<Yunohost Git/Infra notifications> [searxng_ynh] ewilly merged [pull request #532](https://github.com/YunoHost-Apps/searxng_ynh/pull/532) : Upgrade master from testing
[13:43:27]
<Yunohost Git/Infra notifications> [adguardhome_ynh] brimwats opened [issue #257](https://github.com/YunoHost-Apps/adguardhome_ynh/issues/257) : Permissions issue on YNH 13
[15:07:07]
<Yunohost Git/Infra notifications> [immich_ynh] oleole39 [commented](https://github.com/YunoHost-Apps/immich_ynh/issues/432#issuecomment-5389463936) on [issue #432](https://github.com/YunoHost-Apps/immich_ynh/issues/432) Postgresql migration nightmare: Thank you for the thorough explanation. So now I understand the reason for this is that there is no prebuilt deb package...
[15:07:17]
<otm33> Context : the app install script includes `ynh_exec_as_app ssh-keygen -t rsa -N "" -f "$data_dir/ssh_keys/ssh_host_key"
Help me understand this in CI test step 2 :
- Installation completed + url validation => OK
- Removing the app does output `"54804 INFO Deprovisioning data_dir..." and the app is uninstalled
- Reinstalling after removal => hangs. The dbug message after cancelling is :
`69592 INFO DEBUG - /home/yunohost.app/seaweedfs/ssh_keys/ssh_host_key already exists.`
Would this mean that `data_dir` is not really removed 🤔?
Of course, Step 2 passes if I add `ynh_safe_rm "$data_dir/ssh_keys"` to the remove script.
[15:07:20]
<otm33> I must be missing something...
[15:07:30]
<m606> > Removing the app does output `"54804 INFO Deprovisioning data_dir..."
any details below this step ?
could there be some permissions on the keyfile preventing it to be removed?
[15:07:40]
<otm33> ```
> Removing the app...
1327 INFO Removing seaweedfs…
1531 INFO [++++................] > Removing system configurations related to seaweedfs...
1537 INFO [####++++............] > Is /home/yunohost.app/seaweedfs/ssh_keys empty ?
1554 INFO [########++++........] > Removing system configurations related to seaweedfs...
37471 WARNING WARNING Port 2022 is already closed
39780 WARNING WARNING Port 2022 is already closed
53445 INFO [############++++....] > Is /home/yunohost.app/seaweedfs/ssh_keys empty ?
53454 INFO [####################] > Removal of seaweedfs completed
53570 INFO Deprovisioning ports...
53663 INFO Deprovisioning permissions...
54804 INFO Deprovisioning data_dir...
54811 INFO Deprovisioning install_dir...
54866 INFO Deprovisioning system_user...
55183 INFO Deprovisioning sources...
55188 SUCCESS seaweedfs uninstalled
```
> could there be some permissions on the keyfile preventing it to be removed?
[15:07:53]
<otm33> ```
ls -la /home/yunohost.app/seaweedfs/ssh_keys/
total 16
drwx------ 2 seaweedfs seaweedfs 4096 Aug 24 16:52 .
drwxr-x--- 6 seaweedfs seaweedfs 4096 Aug 24 16:52 ..
-rw------- 1 seaweedfs seaweedfs 2622 Aug 24 16:52 ssh_host_key
-rw-r--r-- 1 seaweedfs seaweedfs 586 Aug 24 16:52 ssh_host_key.pub
``
[15:08:17]
<m606> purge option is not selected I think?
https://github.com/YunoHost/yunohost/blob/3c339cc33a55a91005cbbcbb1eb43a88f554968b/src/utils/resources.py#L1161
[15:08:22]
<otm33> Dpes the CI remove script act as yunohost app remove (without purge) ?
[15:08:38]
<otm33> Many apps create folders in data_dir during their installation and there's no issue with the test script...
[15:09:58]
<orhtej2> > <@otm33:matrix.org> Dpes the CI remove script act as yunohost app remove (without purge) ?
Yep, no purge
[15:10:01]
<m606> maybe empty folder is not the same than file?
[15:41:46]
<m606> but yes you are right, e.g. https://github.com/YunoHost-Apps/overleaf_ynh/blob/a7054cdfb1a1355760060ddc6318cd2563029573/scripts/install#L64 installs a source in $data_dir without anything specific to remove it in the remove script and there is no such issue in the CI: https://ci-apps-dev.yunohost.org/ci/logs/22720.log
[16:06:46]
<otm33> This seems specific to the ssh-keys generated: I get the same behaviour with another app adding `ynh_exec_as_app ssh-keygen -t rsa -N "" -f "$data_dir/ssh_keys/ssh_host_key` to its install script. https://ci-apps-dev.yunohost.org/ci/job/23976
[17:23:07]
<otm33> This seems specific to the ssh-keys generated: I get the same behaviour with another app adding `ynh_exec_as_app ssh-keygen -t rsa -N "" -f "$data_dir/ssh_keys/ssh_host_key` to its install script. https://ci-apps-dev.yunohost.org/ci/job/23976
[17:23:07]
<ljf> thatoo we probably should alert users about this bug https://github.com/Euro-Office/eurooffice-nextcloud/issues/145
[17:23:08]
<otm33> <del>This seems specific to the ssh-keys generated</del>: I get the same behaviour with another app adding `ynh_exec_as_app ssh-keygen -t rsa -N "" -f "$data_dir/ssh_keys/ssh_host_key` to its install script. https://ci-apps-dev.yunohost.org/ci/job/23976
[17:23:08]
<otm33> The files/folders don't get removed (I added https://github.com/YunoHost-Apps/seaweedfs_ynh/blob/sftp_feat_rmissue/scripts/install#L14-L15) and here is the reinstall log : https://paste.yunohost.org/raw/kurepahoyu
[18:22:41]
<Yunohost Git/Infra notifications> [package_check] Thovi98 pushed to purge: --purge on remove so ther is no config left in data_dir after remove ([5457fa12](https://github.com/YunoHost/package_check/commit/5457fa1201e4def9f8b7dd4b9a0df981f6b5665e))
[18:22:41]
<Yunohost Git/Infra notifications> [package_check] Thovi98 opened [pull request #212](https://github.com/YunoHost/package_check/pull/212) : --purge on remove so ther is no config left in data_dir after remove
[18:22:42]
<Thomas> > <@yunohostinfra:matrix.org> [package_check] Thovi98 opened [pull request #212](https://github.com/YunoHost/package_check/pull/212) : --purge on remove so ther is no config left in data_dir after remove
m606 @m606:matrix.org: would this help?
[18:27:24]
<Yunohost Git/Infra notifications> [package_check] Josue-T [commented](https://github.com/YunoHost/package_check/pull/212#issuecomment-5399544305) on [issue #212](https://github.com/YunoHost/package_check/pull/212) --purge on remove so ther is no config left in data_dir after remove: Well in my opinion we would need to test both cases, because I means the case that an app is reinstalled and so that the...
[18:29:18]
<m606> yes, @otm33:matrix.org I made a test PR there (a source is unzipped in data_dir in install script), and same data is not removed, no surprise given the absence of --purge.
https://github.com/YunoHost-Apps/overleaf_ynh/blob/5b48d8006de6cd0b7fc955e1a1d518ae97f44a68/scripts/install#L13-L14
https://ci-apps-trixie-dev.yunohost.org/ci/logs/549.log
Now the question is how comes you get an error in your case ?!
[19:14:43]
<otm33> If files/folders do not get removed, in the same vein, mkdir -p and not only mkdir...
[19:24:18]
<otm33> m606 if the keys already exist, ssh-keygen prompts for confirmation. Should be a flag to <del>either overwrite existing keys or</del> leave existing keys...
[19:58:02]
<m606> or`ynh_hide_warnings`? 🙈
[20:02:47]
<m606> or`ynh_hide_warnings`? 🙈 (provided that it doesn't matter to reuse existing data)
[20:19:51]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly pushed to fix_migration: Try fixing psql migration issue ([227329b4](https://github.com/YunoHost-Apps/immich_ynh/commit/227329b4773c78421bc1042481d4b25478be478a))
[20:24:51]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly opened [pull request #434](https://github.com/YunoHost-Apps/immich_ynh/pull/434) : Fix migration
[20:26:18]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly assigned ewilly on [pull request #434](https://github.com/YunoHost-Apps/immich_ynh/pull/434) : Fix migration
[20:50:40]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly pushed to fix_migration: fix ([dcba15ac](https://github.com/YunoHost-Apps/immich_ynh/commit/dcba15acab6bca8bc9f50b371e6e6bdce0cfe49a))
[20:55:18]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly pushed to fix_migration: Merge branch fix_migration of github.com:YunoHost-Apps/immich_ynh into fix_migration ([fcc6ff8a](https://github.com/YunoHost-Apps/immich_ynh/commit/fcc6ff8ae31d3886692916cc359a32fa3089bfc9))
[21:14:20]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly pushed to fix_migration: fix ([de8b341f](https://github.com/YunoHost-Apps/immich_ynh/commit/de8b341f609c947828c25a9b9d709fe506862d92))
[21:23:58]
<otm33> I will add a simple check for wether the keys already exist.
[21:30:49]
<miro5001> > <@otm33:matrix.org> I will add a simple check for wether the keys already exist in the install script.
If folder exists, move it to ".bak". In case someone uninstalls keeping data then reinstalls
[21:31:07]
<otm33> I was thinking about something like if keys don't exist, generate them, else skip.
[21:33:43]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly edited [pull request #434](https://github.com/YunoHost-Apps/immich_ynh/pull/434) : Fix migration
[21:33:50]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly edited [pull request #434](https://github.com/YunoHost-Apps/immich_ynh/pull/434) : Fix migration
[21:33:55]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly [commented](https://github.com/YunoHost-Apps/immich_ynh/issues/432#issuecomment-5401549676) on [issue #432](https://github.com/YunoHost-Apps/immich_ynh/issues/432) Postgresql migration nightmare: lets try to roll back to v15 in bookworm -> https://github.com/YunoHost-Apps/immich_ynh/pull/434
CI sucess will not be e...
[21:34:03]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly edited [pull request #434](https://github.com/YunoHost-Apps/immich_ynh/pull/434) : Fix migration
[21:34:08]
<Yunohost Git/Infra notifications> [immich_ynh] ewilly [commented](https://github.com/YunoHost-Apps/immich_ynh/pull/434#issuecomment-5401583902) on [issue #434](https://github.com/YunoHost-Apps/immich_ynh/pull/434) Fix migration: trixietestme
[21:48:01]
<Yunohost Git/Infra notifications> [immich_ynh] fflorent [commented](https://github.com/YunoHost-Apps/immich_ynh/issues/432#issuecomment-5401808086) on [issue #432](https://github.com/YunoHost-Apps/immich_ynh/issues/432) Postgresql migration nightmare: Thanks @ewilly. Given the fact there also exist that the [openprojects package](https://github.com/YunoHost-Apps/openpro...
[22:37:43]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 77 pending update PRs
- 14 new apps PRs: bookstack, code-server, dokos, esphome, freescout, jackett, lasuite-docs, lasuite-meet, liberaforms, may, mitra, rallly, sparkyfitness, woodpecker
- 7 failed apps updates: automad, benevalibre, gamja, koinsight, loki, lute, projectsend
See the full log here: https://paste.yunohost.org/raw/janecobuso
Autoupdate dashboard: https://apps.yunohost.org/dash?filter=autoupdate