[01:32:48]
<Yunohost Git/Infra notifications> [gitlab_ynh] @leinadlime opened [issue #238](https://github.com/YunoHost-Apps/gitlab_ynh/issues/238): security issue in recent gitlab version
[09:03:04]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1963](https://github.com/YunoHost/apps/pull/1963): Update app levels according to CI results
[09:03:04]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([f12e1f7c](https://github.com/YunoHost/apps/commit/f12e1f7c2fb671895c7eae114b2307a585682db6))
[09:03:05]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([3409c8d4](https://github.com/YunoHost/apps/commit/3409c8d4de6db410dd5c92f489d7e538efec3fbd))
[09:03:05]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([4a5aaadb](https://github.com/YunoHost/apps/commit/4a5aaadba87446621c9c54ee1d5771e8c961dabc))
[09:03:43]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1963](https://github.com/YunoHost/apps/pull/1963): Update app levels according to CI results
[09:03:43]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([3aba8112](https://github.com/YunoHost/apps/commit/3aba811224263ae3df92825bc2c7f43f8cc3e45b))
[09:24:49]
<Yunohost Git/Infra notifications> App glitchsoc goes down from level 7 to 6 in job [#22175](https://ci-apps.yunohost.org/ci/job/22175)
[11:03:52]
<Yunohost Git/Infra notifications> App pleroma goes down from level 8 to 6 in job [#22200](https://ci-apps.yunohost.org/ci/job/22200)
[11:35:36]
<Yunohost Git/Infra notifications> @grenagit forked nextcloud_ynh to [grenagit/nextcloud_ynh](https://github.com/grenagit/nextcloud_ynh)
[11:46:45]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @grenagit opened [pull request #648](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/648): Fix Change PHP-FPM configuration
[13:14:40]
<Yunohost Git/Infra notifications> App pleroma goes down from level 8 to 6 in job [#22204](https://ci-apps.yunohost.org/ci/job/22204)
[13:25:35]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/648#issuecomment-1890458119) on [issue #648](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/648) Fix Change PHP-FPM configuration : Uuuuh but nowadays we use phpversion which indeeds seems to be the one used in scripts/config so that doesnt seem t...
[13:26:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/648#issuecomment-1890458440) on [issue #648](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/648) Fix Change PHP-FPM configuration : Ah I see, the actual issue is probably that unlike the other scripts, the settings are not auto-loaded in the scrips/co...
[13:27:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/648#issuecomment-1890458811) on [issue #648](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/648) Fix Change PHP-FPM configuration : Ah no, it *is* already loaded ... https://github.com/YunoHost-Apps/nextcloud_ynh/blob/testing/scripts/config#L18 ... im...
[13:33:33]
<Yunohost Git/Infra notifications> App leantime rises from level 6 to 7 in job [#22206](https://ci-apps.yunohost.org/ci/job/22206) !
[13:51:01]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 41 commits to fix-redirect-logout ([85dee20339d4...baa1725ba91a](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/85dee20339d4...baa1725ba91a))
[13:51:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/636#issuecomment-1890464065) on [issue #636](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/636) [fix] redirect logout: testme
[15:39:34]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to update_app_levels ([4a5aaadba874...523571b5d0c5](https://github.com/YunoHost/apps/compare/4a5aaadba874...523571b5d0c5))
[15:40:59]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1963](https://github.com/YunoHost/apps/pull/1963): Update app levels according to CI results
[15:56:22]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1963](https://github.com/YunoHost/apps/pull/1963#pullrequestreview-1820042052) Update app levels according to CI results
[15:57:50]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u pushed 1 commit to testing: 16.7.3 ([db00762a](https://github.com/YunoHost-Apps/gitlab_ynh/commit/db00762a3e02d51554aa7aed83bae458e5a8812f))
[15:57:51]
<Yunohost Git/Infra notifications> [gitlab_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README ([f9931f2d](https://github.com/YunoHost-Apps/gitlab_ynh/commit/f9931f2d31e23ae950d21c719054e2c074a4cac5))
[16:00:23]
<Yunohost Git/Infra notifications> [gitlab_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236#issuecomment-1890538868) on [issue #236](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236) Testing: May the CI gods be with you
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/...
[16:00:24]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236#issuecomment-1890539144) on [issue #236](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236) Testing: #fix #238
[16:00:24]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236#issuecomment-1890539144) on [issue #236](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236) Testing: #fix #238
[16:00:25]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u edited [pull request #236](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236): Testing
[16:00:25]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u edited [pull request #236](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236): Testing
[16:00:26]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236#issuecomment-1890538781) on [issue #236](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236) Testing: testme
[18:42:24]
<Yunohost Git/Infra notifications> [gitlab_ynh] @ericgaspar approved [pull request #236](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236#pullrequestreview-1820082471) Testing
[18:43:26]
<Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch update_app_levels
[18:43:27]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1963](https://github.com/YunoHost/apps/pull/1963): Update app levels according to CI results
[18:43:29]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 11 commits to master ([2d5cee23263e...5e12608fb1d9](https://github.com/YunoHost/apps/compare/2d5cee23263e...5e12608fb1d9))
[18:46:12]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u closed [issue #238](https://github.com/YunoHost-Apps/gitlab_ynh/issues/238): security issue in recent gitlab version
[18:46:12]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u merged [pull request #236](https://github.com/YunoHost-Apps/gitlab_ynh/pull/236): Testing
[18:46:13]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u pushed 5 commits to master ([d7196798124e...33847fe75e0b](https://github.com/YunoHost-Apps/gitlab_ynh/compare/d7196798124e...33847fe75e0b))
[18:46:38]
<Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u [commented](https://github.com/YunoHost-Apps/gitlab_ynh/issues/238#issuecomment-1890695555) on [issue #238](https://github.com/YunoHost-Apps/gitlab_ynh/issues/238) security issue in recent gitlab version: Youll be able to upgrade soon :+1: Thank you for the issue
[20:12:52]
<eric_G> https://github.com/YunoHost-Apps/nextcloud_ynh/blob/3e60bacfa5f4f3eaed7962a06a3880ffd6f2117e/scripts/upgrade#L225
[20:13:50]
<eric_G> updating the apps in Nextcloud seems to fail the update... https://paste.yunohost.org/raw/zerusalopa
[20:45:05]
<eric_G> too many apps available in Nextcloud...
[20:46:35]
<eric_G> they should go back to basics
[20:52:03]
<Aleks (he/him/il/lui)> Yup
[20:53:06]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> updating the apps in Nextcloud seems to fail the update... https://paste.yunohost.org/raw/zerusalopa
Thats because one app is not available in the new version, cf my PR from yesterday
[20:56:04]
<eric_G> yep, but I wonder why `exec_occ app:update --all` doesn't handle this case as it is highly expected...
[21:02:20]
<Aleks (he/him/il/lui)> yeah i don't know if it could just "not upgrade this one app and leave the code as it is" because the app could be super old and make assumptions about Nextcloud's API which are not true anymore and in fact totally break nextcloud
[21:02:43]
<eric_G> If I understand your script the upgrade just fails if an app in not yet compatible with the New version of Nextcloud?
[21:03:26]
<Aleks (he/him/il/lui)> it's kind of like having an app from the 3.x era .. maybe it was using helpers which don't exist anymore (except in YunoHost we kind of try to keep backward compatibility for a ~long time)
[21:04:16]
<Aleks (he/him/il/lui)> i mean we could have designed apps in the same way nextcloud does, imposing apps to declare the max version of yunohost they work on etc...
[21:04:24]
<Aleks (he/him/il/lui)> which would kind of help for bookworm for example
[21:04:39]
<Aleks (he/him/il/lui)> but at the same time be annoying for the kind of issues we are witnessing with nextcloud upgrades
[21:05:07]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> If I understand your script the upgrade just fails if an app in not yet compatible with the New version of Nextcloud?
yes it fails as early as possible and explain the issue instead of like in the middle of the upgrade
[21:05:21]
<eric_G> > <@Alekswag:matrix.org> i mean we could have designed apps in the same way nextcloud does, imposing apps to declare the max version of yunohost they work on etc...
or go back to basics 😅
[21:05:54]
<Aleks (he/him/il/lui)> but it's only 90% tested, we would need somebody from an old version with a not-yet-available-in-28 app installed and willing to run this branch
[21:07:14]
<eric_G> can't we deactivate unsupported apps so Nextcloud upgrade don't fail all the time for someone with a crappy unsupported app installed?
[21:10:20]
<eric_G> `exec_occ app:disable $="$(list_installed_apps_not_compatible_with_future_version $last_major_version)`
[21:10:22]
<eric_G> `exec_occ app:disable $(list_installed_apps_not_compatible_with_future_version $last_major_version)`
[21:11:47]
<Aleks (he/him/il/lui)> we could but that's kind of a "big" decision to automatically disable an app the user may be relying on and then they don't know when it'll be supported, maybe they'd like to stick to the current version with their app enabled rather than upgrading to the new version and discovering their important stuff doesnt work anymore and no way to downgrade
[21:14:17]
<eric_G> at least the app doesn't get removed and can be activates back in Nextcloud apps admin 🫠
[21:15:01]
<eric_G> are we updating Nextcloud to new versions too quickly?
[21:17:59]
<Aleks (he/him/il/lui)> dunno, on the one hand people are like "emaged Nextcloud released a new wersion 1 hour ago i want it naooow!11!!!1" and on the other hand people fail to realize that stability is also a good thing and experienced nextcloud admins do know that you should wait a good month (or like 28.1 maybe or idk) for new nextcloud versions to stabilize before upgrading (kind of like YunoHost lol)
[21:18:23]
<Aleks (he/him/il/lui)> but regarding apps, some apps may just go unmainained i guess, soo..
[21:18:23]
<eric_G> I am tempted to have one branch per major version
[21:18:46]
<Aleks (he/him/il/lui)> yeah that's kind of where we are headed with the current "oldstable" branch/PR
[21:19:10]
<eric_G> > <@Alekswag:matrix.org> but regarding apps, some apps may just go unmainained i guess, soo..
and it blocks Nextcloud upgrading for ever...
[21:19:33]
<Aleks (he/him/il/lui)> unless the admin is informed of which app is causing issue yeah 😬
[21:19:54]
<Aleks (he/him/il/lui)> it's not clear to me if the admin does display properly wether or not a nextcloud upgrade is available and wether or not all apps are compatible with the new version (what the script I wrote does)
[21:20:35]
<Aleks (he/him/il/lui)> this sounds like a big trou dans la raquette if it doesnt x_x
[21:20:58]
<eric_G> I think the Nextcloud warns that an app is not compatible...
[21:21:39]
<Aleks (he/him/il/lui)> well then people should read teh frakin admin interface then ;P
[21:23:36]
<eric_G> But I have the bare minimum app installed on my Nextcloud so am not really sure
[21:25:06]
<eric_G> But if we go with 1 branche per version we can't use the webadmin anymore to install / update
[21:25:27]
<eric_G> But if we go with 1 branche per version we can't use the webadmin anymore to install / update Nextcloud
[21:26:54]
<Aleks (he/him/il/lui)> yeah idk to me what's worth it is keeping track of older versions such as one may be able to manually upgrade say from 22 to 26 even though the current version of the package is 28
[21:27:17]
<Aleks (he/him/il/lui)> but we should keep having like an N-1 (oldstable?), N (master) and N+1 (testing)
[21:27:17]
<Aleks (he/him/il/lui)> and that's basically it
[21:32:00]
<miro5001> I remember having a nextcloud instance on a shared host. The web updater : check files and permissions, enable maintenance mode, backup, download, extract, replace, ask for disable maintenance mode. After that updates apps. See the end of this page
https://docs.nextcloud.com/server/latest/admin_manual/maintenance/update.html
[21:37:39]
<eric_G> ... or we could all level up to Kiteworks and meet the industry standards 😅 https://info.kiteworks.com/kiteworks-hardened-virtual-appliance-vanguard
[22:04:56]
<Yunohost Git/Infra notifications> App gitlab goes down from level 8 to 6 in job [#22211](https://ci-apps.yunohost.org/ci/job/22211)