-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[Bug]: After upgrade to NC 29.0.0.7 the upgrade mechanism seems to be broken #44157
Comments
It looks like a regression of nextcloud/updater#535 |
I can confirm that the changes of nextcloud/updater#535 are present in my instance: |
Cc @nextcloud/server-backend |
@stefan-niedermann What do you have in this apps.config.php file? |
This is the complete content of the file (I did not create, touch, modify or edit it manually): <?php
$CONFIG = array (
'apps_paths' => array (
0 => array (
'path' => OC::$SERVERROOT.'/apps',
'url' => '/apps',
'writable' => false,
),
1 => array (
'path' => OC::$SERVERROOT.'/custom_apps',
'url' => '/custom_apps',
'writable' => true,
),
),
'appsallowlist' => getenv('APPS_ALLOWLIST') ? explode(" ", getenv('APPS_ALLOWLIST')) : false,
); I can only guess that it has been created automatically by AIO to pass by default enabled apps via environment variables, but this is just a guess. |
Personally I would revert the PR as usually all settings in additional *.config.php should directly be written back to the normal config.php ... |
Strange, I always considered that a bug we hopefully address some time ;) It also does not work if you set the configuration file to read only and the configuration from the additional files have precedence over config.php 😢 I assume the apps_paths are also in config.php and OC::$SERVERROOT is resolved to the file system path @stefan-niedermann ? |
If so then please let us also add a command to write things out to config.php. |
This is a partial extract of the 'apps_paths' =>
array (
0 =>
array (
'path' => '/var/www/html/apps',
'url' => '/apps',
'writable' => false,
),
1 =>
array (
'path' => '/var/www/html/custom_apps',
'url' => '/custom_apps',
'writable' => true,
),
)
Not sure what exactly you mean by this? Given the paths are string literals in the |
That OC::$SERVERROOT from apps.config.php ends up in config.php as /var/www/html ;) |
Alright, I monkey patched the This seemed to work, fine for me. Please feel free to close this issue from my side. If you think this is an actual bug that should get fixed, feel free to keep it open. 🙂 |
Also used by the micro-services Docker images: |
Closed by nextcloud/updater#540 |
Bug description
After upgrading to NC 29.0.0.2 Beta via CLI, trying to further upgrade the instance fails:
Steps to reproduce
php updater/updater.phar --no-interaction --no-backup && php occ app:enable nextcloud-aio --force
twiceExpected behavior
Upgrading should work or simply detect no new versions. This looks more like a crash.
Installation method
Official All-in-One appliance
Nextcloud Server version
master
Operating system
Debian/Ubuntu
PHP engine version
PHP 8.1
Web server
Nginx
Database engine version
None
Is this bug present after an update or on a fresh install?
Upgraded to a MAJOR version (ex. 22 to 23)
Are you using the Nextcloud Server Encryption module?
None
What user-backends are you using?
Configuration report
List of activated Apps
Nextcloud Signing status
Nextcloud Logs
Additional info
I applied this patch without any effect.
I am absolutely aware that this version is tagged as
Beta
. However it might be an actual bug, therefore reporting it here 😉The text was updated successfully, but these errors were encountered: