30.0.3 RC2, 29.0.10 RC1 and 28.0.13 RC1

skjnldsv · November 28, 2024, 3:08pm

EDIT: a postgre issue have been caught in the 30 RC1
An RC2 have been published to fix this:

https://download.nextcloud.com/server/prereleases/nextcloud-30.0.3rc2.zip

Ho ho ho

Maintenance releases of 28.0.13, 29.0.10 and 30.0.3 are coming next Thursday and RCs for those are now available on our download server.

As always, help with testing is very much welcome!

We updated our servers, did our tests, and the release candidates seem pretty decent. Still, give it a whirl and report back here so we’re even more sure that it’s good to go! If you notice anything out of order, please report back on the appropriate Github repository!

next week, we’ll say goodbye to 28, which will receive its last final before going end-of-life

Download

Changelog

30.0.3 RC1

29.0.10 RC1

28.0.13 RC1

Paradox551 · November 28, 2024, 3:28pm

So anyway, time to restore from a backup (version 30):

Exception: Database error when running migration 5000Date20241120135411 for app app_api An exception occurred while executing a query: SQLSTATE[42804]: Datatype mismatch: 7 ERROR: argument of WHERE must be type boolean, not type integer LINE 1: …LECT “id”, “deploy_config” FROM “oc_ex_apps_daemons” WHERE 1 ^

spyro · November 28, 2024, 3:33pm

ouch. unlucky

Paradox551 · November 28, 2024, 3:46pm

Disabling and removing app_api fixes the issue and the upgrade goes flawlessly but yeah, kinda sucks that it happened.

Paradox551 · November 28, 2024, 4:00pm

Anyway, seeing as this RC literally breaks installs on upgrade perhaps it should be pulled until its fixed?

SysKeeper · November 28, 2024, 4:29pm

Looks like postgres?

Paradox551 · November 28, 2024, 4:41pm

Yes, postgresql.

SysKeeper · November 28, 2024, 4:44pm

Thank you for confirming, we are taking a look!

nickvergessen · November 28, 2024, 4:54pm

github.com

nextcloud/app_api/blob/51714417bc15d527aa29c0caa6c1c4e3c1658b5e/lib/Migration/Version5000Date20241120135411.php#L38


      
          	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
          	 * @param array $options
          	 *
          	 * @return null|ISchemaWrapper
          	 */
          	public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options) {
          		// encrypt "haproxy_password" in the "ex_apps_daemons" table
          		$qbSelect = $this->connection->getQueryBuilder();
          		$qbSelect->select(['id', 'deploy_config'])
          			->from('ex_apps_daemons')
          			->where(1);
          		$req = $qbSelect->executeQuery();
          
          		while ($row = $req->fetch()) {
          			$deployConfig = $row['deploy_config'];
          			$deployConfig = json_decode($deployConfig, true);
          			if (!empty($deployConfig['haproxy_password'])) {
          				$deployConfig['haproxy_password'] = $this->crypto->encrypt($deployConfig['haproxy_password']);
          				$encodedDeployConfig = json_encode($deployConfig);
          				$qbUpdate = $this->connection->getQueryBuilder();
          				$qbUpdate->update('ex_apps_daemons')

Replacing line 38 of apps/app_api/lib/Migration/Version5000Date20241120135411.php with:

should solve the issue

spyro · November 28, 2024, 5:59pm

Gennuine question - are the builds actually tested on postgresql?

I’m not trying to be funny, but the number of postgresql related issues I keep hitting with NC is getting alarming…

Eg. this one:

Paradox551 · November 28, 2024, 6:18pm

I think the core is, but the individual app developers may not test postgresql or may not test postgresql every release.

It’s also possible this bug only happens during upgrades and not during a fresh install, so they missed it.

Guillermo_Schimmel · November 28, 2024, 7:19pm

For the first time in years, this upgrade broke my install.

Paradox551 · November 28, 2024, 7:40pm

Can this upgrade be pulled and/or can it be updated so this is fixed?

I see people posting about this on nextclouds github, so its not an isolated issue.

bloo · November 29, 2024, 1:19pm

Fix at Fix migration for postgres support. by marcelklehr · Pull Request #456 · nextcloud/app_api · GitHub

This is app_api migration problem, not server

nickvergessen · November 29, 2024, 2:42pm

Exactly what happened here.

nickvergessen · November 29, 2024, 3:05pm

RC2 building has began and therefore it should solve in the next hours:

skjnldsv · November 29, 2024, 3:30pm

Thank you, 30.0.3 RC2 has just been released

rakekniven · December 2, 2024, 7:00am

This topic was automatically closed after 3 days. New replies are no longer allowed.