Saturday, August 05, 2023
apps@conference.yunohost.org
August
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
     
             

[01:06:28] <Yunohost Git/Infra notifications> App umami goes down from level 8 to 6 in job [#17473](https://ci-apps.yunohost.org/ci/job/17473)
[02:09:34] <Yunohost Git/Infra notifications> App civicrm_drupal7 rises from level 6 to 8 in job [#17558](https://ci-apps.yunohost.org/ci/job/17558) !
[03:07:32] <Yunohost Git/Infra notifications> App distbin goes down from level 8 to 6 in job [#17559](https://ci-apps.yunohost.org/ci/job/17559)
[05:13:24] <Yunohost Git/Infra notifications> App gemserv rises from level 6 to 8 in job [#17562](https://ci-apps.yunohost.org/ci/job/17562) !
[06:19:14] <Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1708](https://github.com/YunoHost/apps/pull/1708): Add streams to the catalog
[06:19:14] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 4 commits to master ([9c7971d08868...44d0983af315](https://github.com/YunoHost/apps/compare/9c7971d08868...44d0983af315))
[06:19:17] <Yunohost Git/Infra notifications> [apps/master] Update apps.toml Add streams - Papa Dragon
[06:19:19] <Yunohost Git/Infra notifications> [apps/master] Remove level properties for streams This will be added automatically later - Tagada
[06:19:23] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1708 from dragondaddy/apps - Éric Gaspar
[07:29:03] <Yunohost Git/Infra notifications> App my_capsule goes down from level 8 to 6 in job [#17564](https://ci-apps.yunohost.org/ci/job/17564)
[08:55:48] <Aleks (he/him/il/lui)> > https://github.com/YunoHost-Apps/rocketchat_ynh/issues/187

Could you elaborate on this apt/mongo thing ? I dont get why you'd need to call ynh_remove_mongo after apt unprovisioning
[09:05:58] <Tag> https://github.com/YunoHost-Apps/rocketchat_ynh/issues/187
The service is not removed by the helper if mongodb is installed. And after that, mondodb is uninstalled by the apt unprovisioning, leaving the yunohost service broken.I guess in packaging v1, we would call `ynh_remove_app_dependencies` and then `ynh_remove_mongo`
[09:06:06] <Tag> https://github.com/YunoHost-Apps/rocketchat_ynh/issues/187
The service is not removed by the helper if mongodb is installed. And after that, mondodb is uninstalled by the apt unprovisioning, leaving the yunohost service broken.
I guess in packaging v1, we would call `ynh_remove_app_dependencies` and then `ynh_remove_mongo`
[09:12:43] <PapaDragon> Quick question: after merging a pull request in master, should I delete the testing branch? I'm asking because github is telling me that testing is now 1 commit behind master
[09:19:19] <Tag> > testing is now 1 commit behind master

This is normal, because merging creates a new commit on the destination branch. You can continue working on the testing branch and merge it to master.
[09:31:18] <Aleks (he/him/il/lui)> > <@tag:lostpod.me> https://github.com/YunoHost-Apps/rocketchat_ynh/issues/187
> The service is not removed by the helper if mongodb is installed. And after that, mondodb is uninstalled by the apt unprovisioning, leaving the yunohost service broken.
> I guess in packaging v1, we would call `ynh_remove_app_dependencies` and then `ynh_remove_mongo`

Ah yes i see the helper code now ... To me i would say the issue is that remove_mongo is not a proper symetric of install_mongo, because install_mongo adds the extra apt dependency, but remove_mongo doesnt remove the dependency (+autoupdate)
[09:31:58] <Aleks (he/him/il/lui)> Alternatively we could tweak the behavior as in, instead of checking if the package is not installed anymore, looking if another app uses mongodb
[09:33:25] <Aleks (he/him/il/lui)> Or we could somewhat integrate mongodb officially and have something similar to php-fpm services which are automaticlly in the service list if the corresponding package exists
[09:34:22] <Aleks (he/him/il/lui)> Which may allow to just add the extra dep in the manifest.toml and not having to call the install/remove_mongodb helpers at all ?
[09:35:18] <Aleks (he/him/il/lui)> Ah but there's the logic about avx instruction ... Which as we witnessed is kinda broken and you do need avx anyway ...
[10:36:07] <appservice> The libera.chat bridge will be temporarily suspended at 14:00 UTC today. Please see https://matrix.org/blog/2023/08/libera-bridge-disabled/ [https://matrix.org/blog/2023/08/libera-bridge-disabled/] for more information.
[13:31:48] <Yunohost Git/Infra notifications> App hedgedoc failed all tests in job [#17575](https://ci-apps.yunohost.org/ci/job/17575) :(
[14:08:23] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 11 commits to Twitter---X ([1b46f97e5fa3...efe739529053](https://github.com/YunoHost/apps/compare/1b46f97e5fa3...efe739529053))
[14:08:26] <Yunohost Git/Infra notifications> [apps/Twitter---X] Remove level properties for streams This will be added automatically later - Tagada
[14:08:30] <Yunohost Git/Infra notifications> [apps/Twitter---X] Merge pull request #1708 from dragondaddy/apps - Éric Gaspar
[14:08:31] <Yunohost Git/Infra notifications> [apps/Twitter---X] Merge branch master into Twitter---X - Éric Gaspar
[14:08:35] <Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1704](https://github.com/YunoHost/apps/pull/1704): Replace Twitter by X
[14:08:37] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 3 commits to master ([44d0983af315...ac9f234e91c6](https://github.com/YunoHost/apps/compare/44d0983af315...ac9f234e91c6))
[14:08:39] <Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch Twitter---X
[14:08:40] <Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[14:08:44] <Yunohost Git/Infra notifications> [apps/master] Merge branch master into Twitter---X - Éric Gaspar
[14:08:50] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1704 from YunoHost/Twitter---X Replace Twitter by X - Éric Gaspar
[14:46:35] <lapineige> Can someone handle the latest akkoma release upgrade for me ?
https://meta.akkoma.dev/t/akkoma-stable-2023-08-secure-arms-are-bookworms/545
It's a security patch of high priority, and I have no access to my github for 8 days at least.
The bot should open a PR by itself, it's only the naming of the OTP version that needs to be changed (to do it : download the version, extract the archive, there is a file with the version number).
If the bot PR has good results in CI, you could merge it right away. That has to be someone from Yunohost team to have proper rights.

Thanks 🙏
(Same for Pleroma)
[14:47:19] <lapineige> Oh wait, there is an issue, I see it only handles Debian 12 now... That's bad, it means the patch can't be applied... :(
[14:51:39] <lapineige> I asked on the fediverse, I'll tell you what's their input on this
[16:05:28] <lapineige> The short anwser is: Akkoma OTP (=pre-built) releases are no longer built for Debian 11, it's Debian 12 only.
Eitheir we switch to manual install, or we build our own OTP release, or this security upgrade can't be made until Debian 12 compatibily
[20:31:41] <Yunohost Git/Infra notifications> App yunorunner failed all tests in job [#17586](https://ci-apps.yunohost.org/ci/job/17586) :(
[21:16:18] <lapineige> I have good news : there is a very simple workaround to adress the security issue. But I can't disclose it publicly. If someone is willing to do it please contact me directly