Bump to NC 30

[blizzz]

based on #207

• requires add support for Keycloak / UCS 5.2 #207

• Waiting for docker build
• Testing fresh install
• Testing upgrade
• Request publish
• Merge

[blizzz]

Downstream found an update issue, hence the delay.

[palstek]

Anything I can do to accelerate the merge process?

Is there a beta app channel available to test this PR?

[jfk22]

Any ETA for the release of NC30 for UCS?

[blizzz]

Right now there is a failing test case on a re-installation on 5.0. That`s the blocker, and since last time my focus was unfortunately elsewhere.

The background is: when the same Nextcloud version was installed, removed and is installed again, then user_saml is not being fetched again from the appstore. It is in a Schroedinger state:

And then the joinscript fails, because it expects user_saml to be enabled.

Maybe this is a regression in 30 (or maybe up to 30.0.10, the last build i just investigated again). I suspect so. And it would also leave other apps uninstalled that were previously enabled. I'll continue the investigation here (but you can beat me to it and post results here but we may risk to duplicate work).

I also wished to get back to this earlier again, sorry for the delay.

[ateuber]

@blizzz Not sure if it relevant here, but I had a problem with missing apps in 2023 and fixed it like this:

univention-app shell nextcloud sudo -u www-data php /var/www/html/occ config:list | jq '.apps | with_entries(select(.value.enabled == "yes")) | keys' | tail -n +2 | head -n -1 | sed 's/ //g' | sed 's/"//g' | sed 's/,//g' > appinconfig
univention-app shell nextcloud sudo -u www-data php /var/www/html/occ app:list --output=json | jq '.enabled' | jq 'keys' | tail -n +2 | head -n -1 | sed 's/ //g' | sed 's/"//g' | sed 's/,//g' > appinappslist
comm -13 appinappslist appinconfig |  while read line ; do univention-app shell nextcloud sudo -u www-data php /var/www/html/occ app:disable $line ; univention-app shell nextcloud sudo -u www-data php /var/www/html/occ app:enable $line ; done

So maybe user_saml is disabled but still active in the config?

[blizzz]

@blizzz Not sure if it relevant here, but I had a problem with missing apps in 2023 and fixed it like this:

univention-app shell nextcloud sudo -u www-data php /var/www/html/occ config:list | jq '.apps | with_entries(select(.value.enabled == "yes")) | keys' | tail -n +2 | head -n -1 | sed 's/ //g' | sed 's/"//g' | sed 's/,//g' > appinconfig
univention-app shell nextcloud sudo -u www-data php /var/www/html/occ app:list --output=json | jq '.enabled' | jq 'keys' | tail -n +2 | head -n -1 | sed 's/ //g' | sed 's/"//g' | sed 's/,//g' > appinappslist
comm -13 appinappslist appinconfig |  while read line ; do univention-app shell nextcloud sudo -u www-data php /var/www/html/occ app:disable $line ; univention-app shell nextcloud sudo -u www-data php /var/www/html/occ app:enable $line ; done

So maybe user_saml is disabled but still active in the config?

The difference to NC 30 is nextcloud/server#44025 and with this the definition of whether an app is installed has changed. In 29, it was the presence of an app-path, in 30 it is the enabled state:

and in 30:

[blizzz]

To clarify on before, the issue that after an uninstall, and reinstall, apps are enabled by the old configuration, but not installed yet.

The code diverges further on master (nextcloud/server#49648, left a comment at https://github.com/nextcloud/server/pull/49648/files#r2341962849), and this is such an edge case, that it is very weird to consider it there again. I have a working patch against 30, considering to ship it here only and unlock the release, and think about something else for future releases.

[blizzz]

I bumped to latest 30 and applied most recent state of nextcloud/server#55065 as a patch (commit nextcloud/server@ac6653e).

Rebuilding takes today unfortunately, slow download server here, acting up github services there, … 😕

[blizzz]

… seems successful. Wrapping up and submitting…

Update
done. Now the ball is on Univention's side.