feat: Update supported PostgreSQL versions

12 is now EOL but 17 was released and should be supported and tested.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
pull/49240/head
Ferdinand Thiessen 11 months ago
parent 397885adfa
commit a3e80e4195
No known key found for this signature in database
GPG Key ID: 45FAE7268762B400
  1. 8
      .github/workflows/phpunit-pgsql.yml
  2. 14
      apps/settings/lib/SetupChecks/SupportedDatabase.php

@ -56,14 +56,14 @@ jobs:
strategy:
matrix:
php-versions: ['8.1']
# To keep the matrix smaller we ignore PostgreSQL '13', '14', and '15' as we already test 12 and 16 as lower and upper bound
postgres-versions: ['12', '16']
# To keep the matrix smaller we ignore PostgreSQL versions in between as we already test the minimum and the maximum
postgres-versions: ['13', '17']
include:
- php-versions: '8.3'
postgres-versions: '16'
postgres-versions: '17'
coverage: ${{ github.event_name != 'pull_request' }}
- php-versions: '8.4'
postgres-versions: '16'
postgres-versions: '17'
name: PostgreSQL ${{ matrix.postgres-versions }} (PHP ${{ matrix.php-versions }}) - database tests

@ -24,6 +24,8 @@ class SupportedDatabase implements ISetupCheck {
private const MAX_MARIADB = '11.4';
private const MIN_MYSQL = '8.0';
private const MAX_MYSQL = '8.4';
private const MIN_POSTGRES = '13';
private const MAX_POSTGRES = '17';
public function __construct(
private IL10N $l10n,
@ -98,8 +100,16 @@ class SupportedDatabase implements ISetupCheck {
// we only care about X not X.Y or X.Y.Z differences
[$major, ] = explode('.', $versionlc);
$versionConcern = $major;
if (version_compare($versionConcern, '12', '<') || version_compare($versionConcern, '16', '>')) {
return SetupResult::warning($this->l10n->t('PostgreSQL version "%s" detected. PostgreSQL >=12 and <=16 is suggested for best performance, stability and functionality with this version of Nextcloud.', $version));
if (version_compare($versionConcern, self::MIN_POSTGRES, '<') || version_compare($versionConcern, self::MAX_POSTGRES, '>')) {
return SetupResult::warning(
$this->l10n->t(
'PostgreSQL version "%1$s" detected. PostgreSQL >=%2$s and <=%3$s is suggested for best performance, stability and functionality with this version of Nextcloud.',
[
$version,
self::MIN_POSTGRES,
self::MAX_POSTGRES,
])
);
}
} elseif ($databasePlatform instanceof OraclePlatform) {
$version = 'Oracle';

Loading…
Cancel
Save