-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update from 9.1.6 to 9.1.7 failing with postgresql DB #29997
Comments
GitMate.io thinks possibly related issues are #13100 (database connection error displays host, username and password details), #25590 (1091 Can't DROP 'PRIMARY' with upgrade 8.1.6->8.1.9), #26594 (Include app name in exceptions of /lib/private/Installer.php), #23637 (Error output when passing --data-dir during command line installation), and #22877 (Support importing arrays using occ config:import). |
Forgot to add the Schema of that failing table:
|
Hey, this issue has been closed because the label (This is an automated comment from GitMate.io.) |
Just because no one takes a look into that bug and fixes it, doesn't make it go away... |
Hi. I have the same problem when trying to migrate from 9.1.6 to 10.0.6 on Debian 9. Tarballs are from the Open Build Service. The underlying server is Apache with PHP 7.2, database is Postgres 10. Here is the status of the activity app at the moment the upgrade stops.
I tried to search if I could upgrade manually the activity app related table objects, but I'm quite ignorant about owncloud internals. I did not find any clue in the Migrations directories from any 10.0 versions and I did not find where to search in 9.1.7. |
this is the trouble - there is an issue in the used database library and/or our code which breaks on postgres 10. |
Thanks, you're right. |
On PostgreSQL 10 and onwards, you can't query min_value and increment_by directly from the sequence. You have to use the catalog:
The "offending" code is in the file ./lib/composer/doctrine/dbal/lib/Doctrine/DBAL/Schema/PostgreSqlSchemaManager.php. This SQL code unfortunately is incompatible with PostgreSQL version prior to 10. The PHP code will have to discriminate between the version, but that should be fairly easy. |
this is all handled in doctrine/dbal which we use as db abstraction layer. |
@DeepDiver1975 I think we can update dbal on master now ? |
seems master is already on dbal 2.8 in composer.lock despite composer.json saying 2.5. needs to be verified... |
we'll need to setup CI for Postgres 10 to verify that it works on master |
I've made a new issue for Postgres 10 support here: #33187 Closing this OC 9.1 issue then |
Steps to reproduce
Expected behaviour
Upgrade process should start and finish without error
Actual behaviour
Error at schema update check:
Server configuration
Operating system:
Debian GNU/Linux 8
Web server:
nginx
Database:
PostgreSQL 10.1
PHP version:
5.6.30-0+deb8u1
ownCloud version: (see ownCloud admin page)
9.1.7
Updated from an older ownCloud or fresh install:
updated
Version before failed upgrade, 9.1.6
Where did you install ownCloud from:
owncloud.org tarball
Signing status (ownCloud 9.0 and above):
Integrity checker has been disabled. Integrity cannot be verified.
The content of config/config.php:
List of activated apps:
Are you using external storage, if yes which one: local/smb/sftp/...
no
Are you using encryption: yes/no
no
Are you using an external user-backend, if yes which one: LDAP/ActiveDirectory/Webdav/...
no
Logs
Web server error log
ownCloud log (data/owncloud.log)
The text was updated successfully, but these errors were encountered: