-
-
Notifications
You must be signed in to change notification settings - Fork 4.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
[Bug]: Unable to update background execution mode: conflict between new type (mixed) and old type (string)
#45083
Comments
If I toggle the background job type on my test system (Nextcloud 29, updated over years including beta and RC versions) using the CLI
then this error occurs:
This error does not occur in a productive system (version 29, updated over years excluding beta and RC versions, no fresh install). I assume that a database entry may be out of date. |
Can confirm the same issue here on my NC29.0.1 instance:
|
Thanks for reporting. Looks possibly related to changes made in PR #41755 that typed the values. Not a PR or area of code I'm at all familiar with, but some clues in case someone feels like taking a closer look. |
This comment was marked as duplicate.
This comment was marked as duplicate.
Same here too. Additional info: if I run
|
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
I updated to the latest version of Nextcloud yesterday and it seemed to solve my problem with cron not working and also not being able to switch cron types. |
Nextcloud up to date (stable version : Nextcloud Hub 8 (29.0.2)) and not solved. |
This comment was marked as duplicate.
This comment was marked as duplicate.
Same issue here:
just updated from 28.0.5 --> 29.0.2 |
And still happening on 29.0.3:( |
Rather than posting "me too" comments, upvote 👍 the initial report above, unless you have some new information to add. It just spams everybody and wastes developer time needed to fix bugs. Thanks! P.S. This is a known issue (reproduced). That's why I labeled it 1. to develop last week. :) |
As a workaround, for the time being to toggle the background mode, you should be able to change the value directly:
Where It's also possible to force it to be mixed, but I'm hesitant to suggest changing that since I'm not sure what's actually going on here and I still have yet to wrap my head around #41755, the migration path, what the expected behavior was here (other than obviously not breaking the ability to set the background mode). |
conflict between new type (mixed) and old type (string)
Lines 109 to 111 in 00aa8f5
We are using the new IAppConfig api in cron.php. That means if cron.php was called once, and the backgroundjobs_mode was not already set to cron, it's then changed to cron AND type is set from 2 (mixed) to 4 (string). When the value was typed once, it's not possible to set a new value without giving a type. Unfortunatly , the commands and provisioning_api are using the old IAppConfig api which uses type mixed. Pull request for the commands: #46185 The provisioning_api also needs an update to use the types. Needs some input from @ArtificialOwl. |
The final version should support all types, but the patch below made it work again.
|
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
We're blocked by this from upgrading our fleet from 28 to 29: we have a provisioning script which does a
so this bug makes the provisioning fail. Will this bug be fixed in the 29 series? I see that the MR has milestone set to 31, but as you can't skip major versions that would be unfortunate. |
I don't think the pull request will address the problem with the config import command. |
I have the same problem when setting the background image via OCC and also via the WebUI with Nextcloud 30.0.0 EDIT: Tracking in #48436 |
I'm pleasantly surprised to see the 18 people used the thumbs up reaction, thank you 🙏 It's definitely annoying that changing the background image is broken, sorry about that 😞 Could some please log a fresh issue for the problem? It's a different component that needs a different fix than the background mode. It's perfectly fine to omit most of the issue template (e.g. we don't need the configuration report, logs, etc). Just the basic steps to reproduce, your Nextcloud version and a screenshot would be nice. Thanks everyone 👍 EDIT: Tracking in #48436 |
Same issue here |
Bug description
When I want to change the background task execution mode, I have the message "Impossible de mettre à jour le mode d'exécution des tâches d'arrière-plan" (Unable to update background task execution mode)
Steps to reproduce
Expected behavior
Change accepted
Installation method
Community Manual Installation with Archive
Nextcloud Server version
29
Operating system
Debian
PHP engine version
8.3
Web server
Apache
Database engine version
MariaDB
Is this bug present after an update or on a fresh install?
Update from 28.0.5 to 29.0.0
Are you using the Nextcloud Server Encryption module?
No
What user-backends are you using?
Configuration report
List of activated Apps
Nextcloud Signing status
Nextcloud Logs
Additional info
No response
The text was updated successfully, but these errors were encountered: